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

ci: deploy with kustomize #18

Merged
merged 1 commit into from
Jan 3, 2023
Merged

Conversation

NilsOveTen
Copy link
Contributor

@NilsOveTen NilsOveTen commented Dec 23, 2022

prerequisite for #15, #16 & #17

@NilsOveTen NilsOveTen force-pushed the chore/not/deploy-with-kustomize branch 4 times, most recently from 32de04e to 3058da4 Compare January 2, 2023 10:45
@NilsOveTen NilsOveTen changed the title chore: deploy with kustomize ci: deploy with kustomize Jan 2, 2023
@NilsOveTen
Copy link
Contributor Author

Burde egentlig klare å få en versjon av deployment-fdk-dataset-event-publisher.yaml i base-mappa, men blir noe knot når namespace er inkludert i navnet til secretKeyRef.

Sikkert flere måter å fikse det på, men tror det beste hadde vært en refaktorering av navnene til secretKeyRef i GCP, som fort blir en kjempejobb...

Så lagde #19 og utsetter det til Hans Olav er tilbake

@NilsOveTen NilsOveTen force-pushed the chore/not/deploy-with-kustomize branch from 3058da4 to 408375d Compare January 2, 2023 11:49
@tenstad
Copy link
Contributor

tenstad commented Jan 2, 2023

+1 til å endre (secret) names til å ikke inneholde namespace (generelt, og for alle type ressurser)

@NilsOveTen NilsOveTen merged commit 2f6d811 into main Jan 3, 2023
@NilsOveTen NilsOveTen deleted the chore/not/deploy-with-kustomize branch March 1, 2023 06:54
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.

3 participants