Skip to content
New issue

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

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

Already on GitHub? # to your account

B #99: Correctly handle vault-encrypted values in inventory #119

Merged
merged 1 commit into from
Feb 17, 2025
Merged

Conversation

sk4zuzu
Copy link
Contributor

@sk4zuzu sk4zuzu commented Feb 13, 2025

  • Make sure precheck role doesn't fail
  • Extend Makefile so it detects if --ask-vault-pass is required

- Make sure precheck role doesn't fail
- Extend Makefile so it detects if --ask-vault-pass is required
@sk4zuzu sk4zuzu requested review from rsmontero and jaimecb February 13, 2025 17:19
Copy link
Contributor

@jaimecb jaimecb left a comment

Choose a reason for hiding this comment

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

LGTM

@sk4zuzu sk4zuzu added this to the Release 1.2.1 milestone Feb 17, 2025
@rsmontero rsmontero merged commit 4a3588e into master Feb 17, 2025
@rsmontero rsmontero deleted the b-99 branch February 17, 2025 12:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants