Loki: mTLS STRICT for metrics

Package Merge Request

Package Changes

Configures the servicemonitor for TLS scraping

Package MR

https://repo1.dso.mil/platform-one/big-bang/apps/sandbox/loki/-/merge_requests/47

https://repo1.dso.mil/platform-one/big-bang/apps/sandbox/loki/-/merge_requests/26

Includes this Loki bump as well: https://repo1.dso.mil/platform-one/big-bang/bigbang/-/merge_requests/2023

For Issue

Closes https://repo1.dso.mil/platform-one/big-bang/apps/sandbox/loki/-/issues/25

Closes https://repo1.dso.mil/platform-one/big-bang/apps/sandbox/loki/-/issues/24

changed milestone to %1.41.0

added monitoring statusdoing teamcore/security labels

assigned to @brettcharrier

added 1 commit

• 08f62e4d - wrong if flag for istio

Compare with previous version

added 1 commit

• ab11feb4 - moved tlsConfig from Loki to Monitoring

Compare with previous version

changed milestone to %1.42.0

removed statusdoing label

added statusreview label

requested review from @ryan.j.garcia

added statusdoing label and removed statusreview label

changed the description

Moving this to doing for now since it's tied to the package MR in review. Once that is approved/merged you can update this one with the new Loki tag to point to.

changed title from added serviceMonitor for istioInjection to Loki: mTLS STRICT for metrics

@brettcharrier This merge request is not marked as draft, if it is ready for review please add the label, statusreview.

added needs-labels label

removed needs-labels label

resolved all threads

added 1 commit

• 412c7954 - it works

Compare with previous version

added 1 commit

• 3478c20a - monolith too

Compare with previous version

resolved all threads

Can we also add the bump to the Loki chart into this MR instead of this opening a new one

added 1 commit

• 637a992c - added updated Loki tag

Compare with previous version

resolved all threads

added 47 commits

• 637a992c...03aeed22 - 40 commits from branch master
• c3db6064 - added serviceMonitor for istioInjection
• 29bc97b7 - wrong if flag for istio
• a1f3a9e5 - moved tlsConfig from Loki to Monitoring
• 1d0d4c55 - it works
• 97e3d372 - monolith too
• dda9aeb7 - added updated Loki tag
• 393bf55e - Merge branch 'enable-mTLS-for-loki' of...

Compare with previous version

added 1 commit

• d0fd9416 - added comment

Compare with previous version

resolved all threads

resolved all threads

@brettcharrier can you have this MR close this issue as well please https://repo1.dso.mil/platform-one/big-bang/apps/sandbox/loki/-/issues/24

changed the description

mentioned in merge request !2023 (closed)

changed the description

@ryan.j.garcia good to approve and merge?

removed statusdoing label

added statusreview label

resolved all threads

approved this merge request

approved this merge request

unapproved this merge request

Helm tests are failing...this is technically due to these changes but also because Gluon helm test pods are not currently istio injected. I'm working on an MR to istio inject those pods which should resolve this issue...

resolved all threads

resolved all threads

approved this merge request

merged

mentioned in commit 68664931