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

sops: ensure proper wiring to default server #513

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

souleb
Copy link
Member

@souleb souleb commented Dec 9, 2021

make sure we use the right service to encrypt.

@souleb souleb force-pushed the fix-sops-decrypt-defaultserver branch from e1f8d5d to 6d6e4a9 Compare December 9, 2021 13:34
@stefanprodan stefanprodan added the area/sops SOPS related issues and pull requests label Dec 9, 2021
@stefanprodan
Copy link
Member

make sure we use the right service to encrypt

But we don't encrypt anything

@hiddeco
Copy link
Member

hiddeco commented Dec 9, 2021

That does not really matter, as the intent of the full implementation is that it should work anyway for e.g. potential further integration in the Flux CLI (or something).

This is technically a bug, and is right to be addressed :-).

@hiddeco hiddeco changed the title fix decryption using the defaultServer sops: ensure proper decryption wiring to server fallback Dec 9, 2021
@hiddeco hiddeco changed the title sops: ensure proper decryption wiring to server fallback sops: ensure proper wiring to server fallback Dec 9, 2021
@hiddeco
Copy link
Member

hiddeco commented Dec 9, 2021

@souleb the commit takes care of encryption but mentions "decryption".

@hiddeco hiddeco changed the title sops: ensure proper wiring to server fallback sops: ensure proper wiring to default server Dec 9, 2021
Signed-off-by: Soule BA <bah.soule@gmail.com>
@souleb souleb force-pushed the fix-sops-decrypt-defaultserver branch from 6d6e4a9 to 3191f49 Compare December 9, 2021 14:04
@souleb
Copy link
Member Author

souleb commented Dec 9, 2021

@hiddeco sorry 🙏 , I have rewritten the commit message.

@stefanprodan stefanprodan merged commit 3782765 into fluxcd:main Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/sops SOPS related issues and pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants