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

supplement provisioning instructions #10

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davgordo
Copy link
Contributor

@davgordo davgordo commented Jan 10, 2023

I had a little trouble here and there with some gotchas in provisioning the demo, so adding the following clarifications:

  • All branches of the shared workflow repo should be forked in order to include gh-pages for helm chart publishing.
  • Documented some additional required steps to get the GitHub Actions for the Backstage image to build and publish properly. Also see the following PR that removes Kosli steps.
  • Fixed Quay permissions image link
  • Included additional required manual step to enable self-hosted runners for all repos
  • Provided the user a workaround in case the Vault chart results in a TLS secret for the wrong service
  • Instructed the user to initialize Vault, which must be done manually as far as I know

If I misinterpreted any of these additional provisioning requirements, please comment, I am happy to adjust the PR.

@raffaelespazzoli
Copy link
Owner

the "dummy" value is correct. that way the path operator can patch that value based on the actual cluster the to which those apps are being deployed.
Also let's remove kosli completely instead of documenting how to set it up.

@davgordo davgordo force-pushed the readme branch 2 times, most recently from 8261d19 to f0df145 Compare January 10, 2023 01:41
@davgordo
Copy link
Contributor Author

@raffaelespazzoli thanks for the review, I've modified the PR and submitted one for the backstage-app-demo repo.

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.

2 participants