Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Store public key in k8s secret #3411

Merged
merged 7 commits into from
Mar 16, 2021

Conversation

alaa
Copy link
Contributor

@alaa alaa commented Jan 28, 2021

This PR is a clone of the old #2916 for the reason that the upstream fork of the #2916 was deleted therefore opening this new PR.
For more details about the purpose of this change, please check the conversation here: #2916

@alaa alaa force-pushed the feat/store_pub_key_in_k8s_secret branch 2 times, most recently from 0b06322 to c11e416 Compare January 28, 2021 19:30
@alaa alaa mentioned this pull request Jan 28, 2021
Kingdon Barrett added 2 commits February 26, 2021 07:22
go.mod - reset k8s machinery to 1.17.17

This is the latest release that we can update to
-- see fluxcd#3378

Try to remove when distribution/distribution#2905 is out

docker/distribution patch still needed for now

(Put it back for now.) On 2021-02-25 this has been merged, 2.7.2 of
docker/distribution should include it hopefully relatively soon!

Signed-off-by: Kingdon Barrett <[email protected]>
@alaa
Copy link
Contributor Author

alaa commented Feb 27, 2021

Hi @kingdonb, just a friendly reminder.

@kingdonb kingdonb self-assigned this Mar 1, 2021
@kingdonb kingdonb added this to the 1.22.0 milestone Mar 1, 2021
@kingdonb
Copy link
Member

kingdonb commented Mar 1, 2021

Thanks, this looks straightforward and I don't think it has any potential to cause harm or break things.

I will see that it gets included in the reviews for 1.22.0, which should be coming out this week. 🙌

@kingdonb kingdonb force-pushed the feat/store_pub_key_in_k8s_secret branch from c11e416 to cb61b26 Compare March 9, 2021 18:03
@kingdonb
Copy link
Member

kingdonb commented Mar 9, 2021

I rebased this to include it in the omnibus branch that I've composed for Flux v1.22.0 release.

Copy link
Member

@kingdonb kingdonb left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@kingdonb kingdonb merged commit c86defe into fluxcd:master Mar 16, 2021
@kingdonb kingdonb removed their assignment Mar 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants