Skip to content

Vault: re-encrypt files as values#468

Open
mcint wants to merge 4 commits intomasterfrom
mcint/vault-reencrypt
Open

Vault: re-encrypt files as values#468
mcint wants to merge 4 commits intomasterfrom
mcint/vault-reencrypt

Conversation

@mcint
Copy link
Copy Markdown
Contributor

@mcint mcint commented Mar 30, 2026

Answering calls to make the secret names searchable, both on github and locally, and to see more meaningful and well-scoped diffs

@mcint mcint requested review from ElanHR, SuperQ, jetpham and jof March 30, 2026 18:53
@mcint
Copy link
Copy Markdown
Contributor Author

mcint commented Mar 30, 2026

I have unpushed drafts to decrypt value-string-based secrets, supporting round-tripping back out of this format. I need to test that more that ansible-playbook in use decrypts even each-item-in-a-list formulation that's caused the most code headache so far, not just values and dicts, and lists as a whole.

@SuperQ
Copy link
Copy Markdown
Collaborator

SuperQ commented Mar 30, 2026

I'm not a fan of these inline secrets vars. Unless the tooling has improved, I found it a pain to deal with the vars this way.

I prefer the "vault" style in #458. It's easy enough to ansible-vault view group_vars/foo/vault.

If we do want to switch to this style, can we ad a bit of documentation about how to use these?

@mcint mcint mentioned this pull request Mar 30, 2026
Copy link
Copy Markdown
Collaborator

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

No, really, I don't like doing it this way. Please address my comments.

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.

2 participants