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.

Sign up for GitHub

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

Jump to bottom

Adds capi-app when upgrading #1116

Merged
merged 3 commits into from Nov 22, 2021
Merged

Adds capi-app when upgrading #1116

merged 3 commits into from Nov 22, 2021

Conversation

foot
Copy link
Contributor

@foot foot commented Nov 22, 2021 •
edited

What changed?

  • We remove the wego-app.yaml as that functionality is provided by EE.
  • Add a new kustomization that syncs the capi path

Why?

  • Last pieces of the directory structure ADR that specifies capi paths

How did you test it?

  • Manually

@foot foot marked this pull request as ready for review November 22, 2021 09:16
sarataha
sarataha approved these changes Nov 22, 2021
View reviewed changes
Copy link
Contributor

@sarataha sarataha left a comment

Choose a reason for hiding this comment

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

LGTM! 🥇 🌟

@foot foot merged commit b9b2d7f into main Nov 22, 2021
@foot foot deleted the add-capi-apps-kustomization branch November 22, 2021 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sarataha sarataha sarataha approved these changes

Assignees
No one assigned
Labels
exclude from release notes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@foot @sarataha