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

feat(kustomize): add kustomizeFilePath to support gitrepo artifact type #3260

Merged
merged 7 commits into from
Oct 29, 2019

Conversation

mrampton
Copy link
Contributor

@mrampton mrampton commented Oct 25, 2019

This adds an additional property (kustomizeFilePath) to BakeManifestContext and KustomizeBakeManifestRequest. This property is required by Kustomize when using the git/repo artifact type.

This PR is associated with another in Rosco (spinnaker/rosco#449) and another (yet to be created) in Deck. The git/repo artifact type was added to Clouddriver with the following PR: spinnaker/clouddriver#4049

@spinnakerbot
Copy link
Contributor

The following commits need their title changed:

  • d3265cd: Merge remote-tracking branch 'origin' into gitrepo-kustomize

Please format your commit title into the form:

<type>(<scope>): <subject>, e.g. fix(kubernetes): address NPE in status check

This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here.

@mrampton mrampton changed the title feat(kustomize): add kustomizeFilePath to support git/repo artifact type feat(kustomize): add kustomizeFilePath to support gitrepo artifact type Oct 25, 2019
@ethanfrogers
Copy link
Contributor

@spinnakerbot assign @ezimanyi

Copy link
Contributor

@ezimanyi ezimanyi left a comment

Choose a reason for hiding this comment

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

Looks reasonable to me!

@ethanfrogers ethanfrogers added the ready to merge Approved and ready for merge label Oct 29, 2019
@mergify mergify bot merged commit 2136674 into spinnaker:master Oct 29, 2019
@mergify mergify bot added the auto merged Merged automatically by a bot label Oct 29, 2019
sergiopena pushed a commit to sergiopena/orca that referenced this pull request Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Approved and ready for merge target-release/1.17
Projects
None yet
4 participants