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

refactor: refer to ansible facts through ansible_facts.* namespace #187

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

Normo
Copy link
Member

@Normo Normo commented Feb 26, 2024

Related to #185

@Normo Normo self-assigned this Feb 26, 2024
@Normo Normo requested a review from a team as a code owner February 26, 2024 09:35
@Normo Normo enabled auto-merge February 26, 2024 10:08
@Normo Normo added this to the v3.3.0 milestone Feb 26, 2024
@Normo Normo requested review from christianhueserhzdr and removed request for christianhueserhzdr February 26, 2024 10:16
Signed-off-by: Norman Ziegner <norman.ziegner@ufz.de>
Copy link
Member

@christianhueserhzdr christianhueserhzdr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Norman @Normo. I am fine with this PR and will approve it now.

@Normo Normo merged commit 9f50c72 into hifis-net:main Feb 29, 2024
7 checks passed
@Normo Normo deleted the ansible-fact-namespacing branch March 1, 2024 09:45
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 this pull request may close these issues.

Allow roles to run with INJECT_FACTS_AS_VARS set to false
2 participants