-
Notifications
You must be signed in to change notification settings - Fork 62
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
install nginx out of band #83
Conversation
nimakaviani
commented
Nov 15, 2023
- remove embedded nginx installation
- add a new reconciler to deploy nginx resources
- use the same resources previously installed via argo
- wait for it to successfully deploy before proceeding
- fix minor issues
lining up the PRs to reduce review overload. This is required for the gitea server's ingress to be installed out of band and for the domain names to become available to the gitea client |
- remove embedded nginx installation - add a new reconciler to deploy nginx resources - use the same resources previously installed via argo - wait for it to successfully deploy before proceeding - fix minor issues Signed-off-by: Nima Kaviani <[email protected]>
592b636
to
1440d25
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some style points. Take them as you will.
Signed-off-by: Nima Kaviani <[email protected]>
03bbce0
to
631ffbe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM