enable istio on the argocd namespace

Package Owner Merge Request

Package Changes

Summary

Enables istio on the argocd namespace.

This enables istio on the argocd namespace. I tested installing an application through the argocd web interface with and without SSO enabled and also performed an additional sync on the application. The application installed correctly. All the pods deployed ok with and without SSO enabled. It appears that no package-level changes are required.

argocd-argocd-application-controller-57498d646d-67fsw   2/2     Running   0          11m
argocd-argocd-dex-server-75c4878f47-dgq55               2/2     Running   0          11m
argocd-argocd-redis-bb-master-0                         2/2     Running   0          11m
argocd-argocd-redis-bb-replicas-0                       2/2     Running   0          11m
argocd-argocd-redis-bb-replicas-1                       2/2     Running   0          10m
argocd-argocd-repo-server-54cd6647cd-q94q2              2/2     Running   0          11m
argocd-argocd-server-58896fc755-svb4m                   2/2     Running   0          11m

https://repo1.dso.mil/platform-one/big-bang/apps/core/argocd/-/merge_requests/47

https://repo1.dso.mil/platform-one/big-bang/apps/core/argocd/-/releases/3.6.8-bb.9

No known issues

Closes https://repo1.dso.mil/platform-one/big-bang/apps/core/argocd/-/issues/16

added statusreview label

added statusdoing label and removed statusreview label

Throwing to doing pending resolution of https://repo1.dso.mil/platform-one/big-bang/apps/core/argocd/-/issues/16#note_424262

requested review from @BrandenCobb

added argocd statusreview labels and removed statusdoing label

removed statusreview label

added statusdoing label

added 1 commit

• 2b336f6f - argocd tag bump

Compare with previous version

marked this merge request as draft

added 17 commits

• 2b336f6f...f68b2fdb - 16 commits from branch master
• ee354b92 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

marked this merge request as ready

changed the description

added statusreview label and removed statusdoing label

resolved all threads

LGTM.

added 7 commits

• ee354b92...63082bd3 - 6 commits from branch master
• f65b1db3 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

Changes LGTM, doing a quick test to make sure all is as expected.

approved this merge request

@ryan.j.garcia want to test this sucker and validate everything happy?

changed milestone to %1.18.0

unapproved this merge request

Per https://repo1.dso.mil/groups/platform-one/big-bang/-/epics/111#note_425871 we may not actually need this (i.e. more potential harm/problems than benefit)...

Next steps...? - evaluate the TLS docs of argo, make sure it meets what we're looking for - write up some justification for why injection is disabled for argo and create a gatekeeper exception in BB

removed statusreview label

added 1 commit

• b8ffa488 - allow istio-injection enable or disable

Compare with previous version

added 3 commits

• b8ffa488...3e40cecc - 2 commits from branch master
• a069edd2 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

added 3 commits

• a069edd2...39e14cee - 2 commits from branch master
• c1399cba - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

added statusreview label

LGTM. We will want to include a release note to mention that you will need to restart argo pods to get the sidecars on an upgrade.

changed milestone to %1.19.0

mentioned in issue #636 (closed)

resolved all threads

added 6 commits

• c1399cba...86b566d2 - 4 commits from branch master
• 2812ff13 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration
• b8107759 - obfuscate istio.injection values

Compare with previous version

added 9 commits

• b8107759...be675cda - 8 commits from branch master
• 729105e5 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

resolved all threads

approved this merge request

added 7 commits

• 729105e5...6cb67e96 - 6 commits from branch master
• c3d0a541 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

approved this merge request

approved this merge request

added 3 commits

• c3d0a541...12abe975 - 2 commits from branch master
• 18aa6545 - Merge remote-tracking branch 'origin/master' into argocd-istio-integration

Compare with previous version

approved this merge request

added statusdoing label and removed statusreview label

Testing with istio-injection enabled on NS, and 2 replicas for each application within the chart, plus redis enabled. Seeing this error after an initial successful sync on Application:

unapproved this merge request

added 1 commit

• 1b14ac23 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

added statusreview label and removed statusdoing label

resolved all threads

approved this merge request