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

fix kustomize path for big-bang core package #508

Merged
merged 1 commit into from
Jun 9, 2022

Conversation

jeff-mccoy
Copy link
Contributor

@jeff-mccoy jeff-mccoy commented Jun 9, 2022

A small typo was introduced with the big bang core example -> packages move in #488, this corrects that. Since we do not have the ability to run this in E2E right now due to IB credentials and how SUPER long it would take to fully test, including screenshots in the PR after testing from M1 -> Ubuntu AMD64 remote machine using https://github.com/alexellis/k3sup.


note from PR author: this was the first (and one of the only times) I had used k3sup, because I wanted to try it as an alternative to my code-server setup I had been using for a year. Based on the citation of this note in a closed PR on that project, leaving a clarifying note here, this was an isolated local test, and k3sup is no longer used.

@jeff-mccoy jeff-mccoy added bug 🐞 Something isn't working examples labels Jun 9, 2022
@jeff-mccoy
Copy link
Contributor Author

Good package creation now:
Screen Shot 2022-06-09 at 1 48 12 PM

Working package deploy options:
Screen Shot 2022-06-09 at 1 55 33 PM

Successful package deployment:
Screen Shot 2022-06-09 at 1 58 07 PM

Helm releases pulled from gitea:
Screen Shot 2022-06-09 at 1 58 41 PM

@jeff-mccoy jeff-mccoy enabled auto-merge (squash) June 9, 2022 19:03
@jeff-mccoy jeff-mccoy merged commit f3233f0 into master Jun 9, 2022
@jeff-mccoy jeff-mccoy deleted the fix/big-bang-core-kustomize branch June 9, 2022 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants