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

cml_lab_facts module does not have fallback to ENV var CML_LAB #40

Open
tvarohohlavy opened this issue Nov 27, 2024 · 0 comments
Open

Comments

@tvarohohlavy
Copy link

There is an inconsistency between modules regarding the CML_LAB environmental variable usage.

cml_lab_facts does not have fallback to this ENV var:

argument_spec.update(lab=dict(type='str', required=True), )

but others do have it:

lab=dict(type='str', required=True, fallback=(env_fallback, ['CML_LAB'])),

lab=dict(type='str', required=True, fallback=(env_fallback, ['CML_LAB'])),

It would make sense to have the same behavior when using ENV vars across all modules.

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

No branches or pull requests

1 participant