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

refactor: migrated to upstream chart and did a lot of other things #34

Merged
merged 8 commits into from
Aug 2, 2024

Conversation

@Michael-Kruggel Michael-Kruggel marked this pull request as ready for review August 2, 2024 17:19
@JoeHCQ1
Copy link
Contributor

JoeHCQ1 commented Aug 2, 2024

Confluence has a createContainerConfigError - @Michael-Kruggel looks like you may have run into the same problem I had.

@Michael-Kruggel
Copy link
Collaborator Author

Michael-Kruggel commented Aug 2, 2024

Yeah @JoeHCQ1, its due to the awkward nature of the upgrade test. Instead of checking out the latest tag and deploying that, it takes the most recent package and then tries to deploy that using your changes.

Upgrade upstream would never work because there is not an upstream flavor to upgrade from.

Note: I checked-out main, deployed the cluster and package using uds run default. I then checked-out this branch, and deployed just the package using uds run dev, and it came up healthy.

@corang corang disabled auto-merge August 2, 2024 18:30
@corang
Copy link
Collaborator

corang commented Aug 2, 2024

confirmed test from @Michael-Kruggel

@corang corang merged commit dce7d82 into main Aug 2, 2024
8 of 10 checks passed
@corang corang deleted the refactor branch August 2, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment