Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No possibility to ignore dynaconf vault #12816

Closed
dosas opened this issue Oct 4, 2023 · 1 comment · Fixed by #12822
Closed

No possibility to ignore dynaconf vault #12816

dosas opened this issue Oct 4, 2023 · 1 comment · Fixed by #12822

Comments

@dosas
Copy link
Collaborator

dosas commented Oct 4, 2023

Since this PR I have no option to run robottelo tests without configuring the vault.

  File "robottelo/robottelo/utils/vault.py", line 31, in export_vault_addr
    vaulturl = re.findall('VAULT_URL_FOR_DYNACONF=(.*)', envdata)[0]
               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

or

File "robottelo/robottelo/utils/vault.py", line 37, in export_vault_addr
    if re.findall('VAULT_ENABLED_FOR_DYNACONF=(.*)', envdata)[0] == 'true':
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range

I have tried the following without success:

  1. Using env var form pull request template export VAULT_SECRET_ID_FOR_DYNACONF=somesecret
  2. Trying to disable the plugin pytest -p no:auto_vault
  3. Putting some nonsense values into .env file
echo VAULT_URL_FOR_DYNACONF=asdf >> .env
echo VAULT_ENABLED_FOR_DYNACONF=true/false >> .env

yields

ERROR - Error! Vault CLI in not installed in your system, refer link https://learn.hashicorp.com/tutorials/vault/getting-started-install to install vault CLI as per your system spec!

The only workaround for me so far is deleting auto_vault from conftest.py

@jyejare any help would be greatly appreciated

@ogajduse
Copy link
Member

ogajduse commented Oct 5, 2023

@dosas #12822 aims to address this issue. Please feel free to comment on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants