Replace error message of missing credential item by user prompt if tty available

[davidferlay]

Current

• When an action requires a credential item which is missing from keyring, an error is thrown:

ERROR   error on processing parameter "vault-pass" with "keyring.GetKeyValue": vaultpass not found in keyring. Use `plasmactl keyring:set` to add it.

• Message is not very user friendly and may affray user

Expected

• If TTY is available (ie action is not running in CI), it would be great to:

  • directly prompt user for missing credentials and add them to keyring, instead of raising an error message

• If TTY is not available (ie action is running in CI), it would be great to:

  • raise a more user friendly error message

[davidferlay]

Would be great to print some text to let user know which credentials is being added:

➜ plasmactlx platform:vault edit platform/group_vars/platform/vault.yaml                                  
Value: 

➜ plasmactlx platform:var-get erp_odoo_ldap_company_name --keyring-passphrase-file /tmp/kp   
Value:  

[davidferlay]

Also would be great to have a more informative error message when credential item is missing from keyring and no TTY available (like in CI) :

$ time plasmactl bump --sync --hide-progress
 INFO  Processing propagation...
  ERROR   the keyring is malformed or wrong passphrase provided

Atm error message can be misleading

I'm wondering how to implement it only once in keyring / for all usage ? rather than having to implement a specific message everytime keyring is used

[davidferlay]

Reminder to self: test with meta command

[davidferlay]

➜ plx platform:vault edit platform/group_vars/platform/vault.yaml --keyring-passphrase-file /tmp/kp                    
Enter value of 'vaultpass':  

👌