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

Add sourcesecret and kustomization manifestgen #1001

Merged
merged 1 commit into from
Feb 26, 2021

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Feb 25, 2021

This includes a change to the sync generator to make the deploy
secret name configurable for future implementation by deploy refactor in #968.

@hiddeco hiddeco force-pushed the manifestgen-deploysecret-kustomization branch 2 times, most recently from 054d1d0 to c49ee87 Compare February 26, 2021 10:19
@stefanprodan
Copy link
Member

I find the deploysecret pkg name confusing, can we rename it to gitsecret or sourcesecret?

@hiddeco hiddeco changed the title Add deploysecret and kustomization manifestgen Add gitsecret and kustomization manifestgen Feb 26, 2021
@hiddeco hiddeco force-pushed the manifestgen-deploysecret-kustomization branch 8 times, most recently from e239978 to 183aaf8 Compare February 26, 2021 14:06
@hiddeco hiddeco changed the title Add gitsecret and kustomization manifestgen Add sourcesecret and kustomization manifestgen Feb 26, 2021
@hiddeco hiddeco force-pushed the manifestgen-deploysecret-kustomization branch 5 times, most recently from 7700aa2 to c78a0be Compare February 26, 2021 15:35
This includes a change to the `sync` generator to make the deploy
secret name configurable.

Signed-off-by: Hidde Beydals <[email protected]>
@hiddeco hiddeco force-pushed the manifestgen-deploysecret-kustomization branch from c78a0be to 8a5bba8 Compare February 26, 2021 15:59
@hiddeco hiddeco marked this pull request as ready for review February 26, 2021 15:59
@hiddeco hiddeco added area/bootstrap Bootstrap related issues and pull requests area/git Git related issues and pull requests area/helm Helm related issues and pull requests enhancement New feature or request area/kustomization Kustomization related issues and pull requests labels Feb 26, 2021
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Nice refactoring! Thanks @hiddeco 🍻

@hiddeco hiddeco merged commit bb35624 into main Feb 26, 2021
@hiddeco hiddeco deleted the manifestgen-deploysecret-kustomization branch February 26, 2021 16:16
hiddeco added a commit that referenced this pull request Mar 16, 2021
Regression bug introduced in #1001.

Signed-off-by: Hidde Beydals <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/bootstrap Bootstrap related issues and pull requests area/git Git related issues and pull requests area/helm Helm related issues and pull requests area/kustomization Kustomization related issues and pull requests enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants