← Back to team overview

sts-sponsors team mailing list archive

Re: [Merge] ~troyanov/maas:fix-cli-imports into maas:master

 


Diff comments:

> diff --git a/src/maascli/snap.py b/src/maascli/snap.py
> index a8b8068..58a6747 100644
> --- a/src/maascli/snap.py
> +++ b/src/maascli/snap.py
> @@ -568,6 +567,8 @@ def get_vault_settings(options) -> dict:
>      if not options.vault_uri:
>          return {}
>  
> +    from maasserver.vault import prepare_wrapped_approle, VaultError

If I understand correctly, this particular command can be executed only on MAAS Region Controller, because it updates MAAS config and checks that Vault is available from the Region Controller.

> +
>      required_arguments = [
>          "vault-approle-id",
>          "vault-wrapped-token",


-- 
https://code.launchpad.net/~troyanov/maas/+git/maas/+merge/439218
Your team MAAS Committers is subscribed to branch maas:master.



References