FDN-4400 Update generic-charts to v2.0.23

[team-foundation-git]

This PR contains the following updates:

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

⚠️ 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.23

Compare Source

---

This PR has been generated by Renovate Bot.

Summary by CodeRabbit

• Chores
  • Updated Helm chart versions for deployment configurations from v2.0.22 to v2.0.23.

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR updates Helm chart source references in two Argo CD Application manifests, bumping the generic-charts targetRevision from v2.0.22 to v2.0.23 for both dependency-api and dependency-www deployments.

Changes

Cohort / File(s)	Summary
Helm Chart Version Updates deploy/dependency-api/app.yaml, deploy/dependency-www/app.yaml	Updated targetRevision for charts/flow-generic from v2.0.22 to v2.0.23 in both Argo CD Application specifications.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• flowcommerce/dependency#891 — Modifies the same app.yaml files by bumping generic-charts targetRevision (v2.0.20→v2.0.22 vs. this PR's v2.0.22→v2.0.23).
• flowcommerce/dependency#889 — Updates identical Argo CD Application sources in deploy/dependency-api/app.yaml and deploy/dependency-www/app.yaml by bumping generic-charts targetRevision.
• flowcommerce/dependency#876 — Modifies the same Argo CD Application source entries in deploy/dependency-api/app.yaml and deploy/dependency-www/app.yaml with a different chart version bump.

Suggested reviewers

• arias-m
• 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 concisely describes the main change: updating the generic-charts Helm dependency from v2.0.22 to v2.0.23, which matches the primary objective of the PR.
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.}

[flow-tech]

Argo diff output for dependency-api:

===== datadoghq.com/DatadogMonitor datadog/dependency-api-errorrate ======
144d143
<   - generated:kubernetes

[flow-tech]

Argo diff output for dependency-www:

===== datadoghq.com/DatadogMonitor datadog/dependency-www-errorrate ======
144d143
<   - generated:kubernetes

[coderabbitai]

♻️ Duplicate comments (1)

deploy/dependency-api/app.yaml (1)

45-47: Same chart bump as dependency-www; ensure verification covers this app too.

🤖 Prompt for AI Agents

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

In `@deploy/dependency-api/app.yaml` around lines 45 - 47, The chart version for
the app at path "charts/flow-generic" was bumped (targetRevision: v2.0.23) like
dependency-www but verification wasn't updated; update the deployment
verification to include this app by adding or extending the existing check for
dependency-www to also validate the chart at path "charts/flow-generic"
(targetRevision) — ensure your CI/build/test job verifies the new targetRevision
v2.0.23 is deployed and that any smoke/integration tests used for dependency-www
also run against this app.

🤖 Prompt for all review comments with AI agents

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

Duplicate comments:
In `@deploy/dependency-api/app.yaml`:
- Around line 45-47: The chart version for the app at path "charts/flow-generic"
was bumped (targetRevision: v2.0.23) like dependency-www but verification wasn't
updated; update the deployment verification to include this app by adding or
extending the existing check for dependency-www to also validate the chart at
path "charts/flow-generic" (targetRevision) — ensure your CI/build/test job
verifies the new targetRevision v2.0.23 is deployed and that any
smoke/integration tests used for dependency-www also run against this app.

---

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f830e60 and 896c6e2.

📒 Files selected for processing (2)

• deploy/dependency-api/app.yaml
• deploy/dependency-www/app.yaml

[team-foundation-git]

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.