FDN-4400 Update generic-charts to v2.0.24

[team-foundation-git]

This PR contains the following updates:

Package	Update	Change
git@github.com:flowcommerce/generic-charts.git	patch	v2.0.20 -> v2.0.24

⚠️ This PR updates the shared Helm chart version.

Before merging:

1. Ensure your service is compatible with the new chart version

2. Review diff in PR with changes

3. Merge when ready to deploy this change

After merging, ArgoCD will automatically sync the new chart version.

---

Release Notes

flowcommerce/generic-charts (git@github.com:flowcommerce/generic-charts.git)

v2.0.24

Compare Source

v2.0.23

Compare Source

v2.0.22

Compare Source

v2.0.21

Compare Source

---

This PR has been generated by Renovate Bot.

Summary by CodeRabbit

• Chores
  • Updated deployment configuration for the registry service.

[flow-tech]

Argo diff output for registry:

===== /Service production/registry ======
11c11
<     helm.sh/chart: flow-generic-1.6.62
---
>     helm.sh/chart: flow-generic-1.6.64

===== apps/Deployment production/registry-live ======
16c16
<     helm.sh/chart: flow-generic-1.6.62
---
>     helm.sh/chart: flow-generic-1.6.64

===== datadoghq.com/DatadogMonitor datadog/registry-argorollout ======
162d161
<   - generated:kubernetes

===== datadoghq.com/DatadogMonitor datadog/registry-errorrate ======
147d146
<   - generated:kubernetes

===== networking.istio.io/Gateway production/registry-flow-io ======
17c17
<     helm.sh/chart: flow-generic-1.6.62
---
>     helm.sh/chart: flow-generic-1.6.64

===== networking.istio.io/VirtualService production/registry-rollout-live ======
12c12
<     helm.sh/chart: flow-generic-1.6.62
---
>     helm.sh/chart: flow-generic-1.6.64

[coderabbitai]

📝 Walkthrough

Walkthrough

Updates the Helm chart dependency version for charts/flow-generic from v2.0.20 to v2.0.24 in the ArgoCD Application manifest for the registry deployment.

Changes

Cohort / File(s)	Summary
Helm Chart Version Update deploy/registry/app.yaml	Bumps targetRevision of charts/flow-generic from v2.0.20 to v2.0.24.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• FDN-4400 Update generic-charts to v2.0.20 #662: Updates the same ArgoCD Application manifest with a prior version bump of charts/flow-generic (v2.0.19→v2.0.20), establishing a pattern of dependency updates in this file.

Suggested reviewers

• gheine

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically summarizes the main change: updating the generic-charts Helm dependency from v2.0.20 to v2.0.24.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@deploy/registry/app.yaml`:
- Line 47: Confirm with chart maintainers whether removing the
"generated:kubernetes" tag in targetRevision v2.0.24 is intentional; if it is,
update the ArgoCD application's ignoreDifferences block (the ignoreDifferences
stanza in app.yaml) to ignore the DatadogMonitor CRD tag drift by adding an
entry for group "datadoghq.com", kind "DatadogMonitor" that uses a
jqPathExpressions filter to ignore '.spec.tags[] | select(. ==
"generated:kubernetes")'; ensure the datadogMonitors configuration in
deploy/registry/values.yaml is referenced when coordinating this change.

---

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 93e710a and e30bc70.

📒 Files selected for processing (1)

• deploy/registry/app.yaml