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

[Bug]: Molecule tests are not working without GitHub pipelines #304

Open
1 task done
tbauriedel opened this issue Jan 17, 2024 · 3 comments
Open
1 task done

[Bug]: Molecule tests are not working without GitHub pipelines #304

tbauriedel opened this issue Jan 17, 2024 · 3 comments
Labels
bug Something isn't working documentation Improvements or additions to documentation

Comments

@tbauriedel
Copy link
Member

Please try to fill out as much of the information below as you can. Thank you!

  • Yes, I've searched similar issues on GitHub and didn't find any.

Which version contains the bug?

No response

Describe the bug

Molecule tests have a reference to environment variables. Example

However, the environments are only defined within the GitHub workflow. If you try to run local molecule tests (molecule test -s <scenario>), they fail due to the missing environments.
So you will always have to run the whole pipeline local.

How to recreate the bug?

No response

@tbauriedel tbauriedel added bug Something isn't working needs-triage Needs to be triaged labels Jan 17, 2024
@widhalmt
Copy link
Member

Did you try to set the environment variables locally?

export MYENVIRONMENTVARIABLE=myvalue

I guess, we could set some automatically, but I think it helps to have some control about some parts of the checks.

@widhalmt widhalmt added documentation Improvements or additions to documentation and removed needs-triage Needs to be triaged labels Feb 13, 2024
@widhalmt
Copy link
Member

If that fixes your problem, we need to put it into the documentation. It shouldn't be misleading.

@tbauriedel
Copy link
Member Author

I know. But I expected all molecule tests should work out of the box.

As far as I know, elasticsearch 7 will rotate out of the collection soon. After that, we could adjust the variables in the scenarios to work with 8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants