-
Notifications
You must be signed in to change notification settings - Fork 0
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
RE-279 Updates showcase services to use the pod init runtime container #10
Draft
mikeycmccarthy
wants to merge
35
commits into
main
Choose a base branch
from
RE-279
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixed typo Revert webconfig Update backend URL in react config Update url in cors policy Update API url Update API_URL Open egress on networkpolicy.yaml Update corsmapping for local testing first Update API_URL for local testing Update corsmapping Add OPTIONS as allowed method to cors policy Hash passwords in db changelog Tenmp fix - drob liquibase schema Remove password hashing functionality Add debugging Update debug logging Set logging level Restore changeset passwords back to plain text Remove logging level Add registration-service ingress.yaml Update frontend networkpolicy label Add frontend-service ingress UpdateAPI_URL to servicename Add frontend domain to cors allowedorigins Revert values back to localhost Update api_url to registration-service svc name Added ingresscontroller namespace to networkpolicy Add frontend custom domain to cors policy Revert API_URL back to localhost Removed backend ingress.yaml Changed cors policy back to localhost Update API_URL Updated corspolicy Temp allow all origins for testing Update CORSmapping to allow all (temp) Add custom domain to cors policy Restore cors and API_URL back to localhost Configure networking for frontend service
Change API_URL to svc name Add port to API_URL Add FQDN to cors Allow ingress from nginx controller to backend Add ingress for registration-service Add backend domain to API_URL Add both custom domains to cors Pass only backend to cors policy Add both domains to cors Amend for one ingress only Update API_URL with FE domain Disable cors - test Amending frontend-service ingress
Remove webconfig.java (CORS) Update backend protocols to HTTPS Remove webconfig
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Updates the showcase services to use the runtime container in order to be self-documenting and to 'eat our own dogfood'.
PR in draft status as the following things are required: