-
Notifications
You must be signed in to change notification settings - Fork 166
Cuts over review app deployment to kustomize overlay/base instead of helm chart #11125
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
Merged
Merged
Changes from all commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
46bb4b7
Swapping to argo application and kustomize
stephencshelton 1e179e4
Initial stab at using kustomize
stephencshelton 2cf5951
Fixing var references
stephencshelton 923cad6
Changes to indentsg
stephencshelton 934465b
Adding missing targets
stephencshelton fc6a5e4
More testing
stephencshelton 42a800c
Fixing variable substitution
stephencshelton c858146
More testing
stephencshelton 13f54cc
More testing
stephencshelton fb1ce03
Adding namespace to apply command
stephencshelton 7d9c4b2
Not validating our application manifest
stephencshelton 8540a02
It was KAS all along
stephencshelton e66aeaf
Removing components key
stephencshelton c1c97e1
Seeing if this helps
stephencshelton 118f9f8
Starting simple and adding more
stephencshelton 8d65c49
Updating to not use patchesStrategicMerge
stephencshelton 4c24702
Swapping to use configmaps and pull environment from them instead
stephencshelton 10314ee
More testing
stephencshelton e8f4c20
Swapping out container images in jobs as well, also removing duplicat…
stephencshelton c1a9a00
Updating application.yaml
stephencshelton a138fd8
Adding in missing service_providers.yml
stephencshelton ec04192
Adding missing DASHBOARD_URL
stephencshelton 43577c5
Add in override for idp initContainer
stephencshelton a9190bf
Adding in missing environment vars for readonly filesystem
stephencshelton 1dd4247
Lowering min/max replicas in HPA
stephencshelton b505979
Adding in missing redis urls
stephencshelton 268c44a
Starting to cleanup some of the overrides
stephencshelton 57e5284
Testing out more nameReference transformers
stephencshelton 2561bbf
Adding in canary for idp override
stephencshelton e1fb1b1
Adding in pivcac ingress
stephencshelton c210883
Adding more image overrides
stephencshelton 6f7ccd1
Adding missing environment var
stephencshelton 01a45b2
More overrides
stephencshelton b3d3f8d
Updating reviewapp image for pivcac
stephencshelton 4beaf57
Fix more pivcac references
stephencshelton f614e9b
Swapping from review to pivcac
stephencshelton 1f8edf6
Seeing if this helps out
stephencshelton 5f657eb
Removing canary
stephencshelton 74a5333
Adding in pull policy overrides
stephencshelton e65224a
Adding in more missing imagepullpolicies
stephencshelton 26843f5
Fixing CronJob imagepullpolicy reference
stephencshelton c49f479
Making sure rollouts is playing nice
stephencshelton ee3c289
Testing out some things
stephencshelton 4c7b460
Seeing if this fixes the routing issue
stephencshelton a8352ad
Swapping to main now that the other is landed
stephencshelton 01d8959
Adding echo statement to help find the application deployment in argo
stephencshelton File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Oops, something went wrong.
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.
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.
can we add a thing that's like "watch your app here" to link to argo? would that be useful? i know we probably can't easily hook it up to do
argocd app waitlike for the other envs as easily, because the argo api token is currently protected to only run on mainThere 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.
I mean nothing stopping us from adding that given we know the environment name, so we can generate the url based on that. Happy to add that if people think it would be good, would also like to throw the URL's as a comment on the MR eventually down the line so people don't have to go digging for them
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.
Alright added a line to help folks find their application in ArgoCD and check the progress