Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion agent-chart/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ dependencies:
version: "0.0.1386"
repository: https://flanksource.github.io/charts
- name: canary-checker
version: "1.1.3-beta.149"
version: "1.1.3-beta.150"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Dependency bump is not realized while lockfile is stale

Line 14 updates canary-checker to 1.1.3-beta.150, but agent-chart/Chart.lock still pins canary-checker to 1.1.3-beta.63. This breaks the chart/lock contract and can deploy the old dependency despite this PR.

Suggested fix
# Regenerate and commit agent-chart/Chart.lock so canary-checker resolves to 1.1.3-beta.150
# (e.g., run Helm dependency update for agent-chart, then commit the lockfile change)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@agent-chart/Chart.yaml` at line 14, The Chart.yaml file has been updated to
pin canary-checker to version 1.1.3-beta.150, but the corresponding Chart.lock
file still references the old version 1.1.3-beta.63 for canary-checker. Update
the canary-checker entry in agent-chart/Chart.lock to match the new version
1.1.3-beta.150 specified in Chart.yaml to maintain consistency between the chart
manifest and its lockfile, ensuring the correct dependency version will be
deployed.

repository: https://flanksource.github.io/charts
- name: mission-control-watchtower
version: "0.1.35-beta.4"
Expand Down
2 changes: 1 addition & 1 deletion chart/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ dependencies:
version: "0.0.1386"
repository: https://flanksource.github.io/charts
- name: canary-checker
version: "1.1.3-beta.149"
version: "1.1.3-beta.150"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Chart dependency declaration and lockfile are inconsistent

Line 19 bumps canary-checker to 1.1.3-beta.150, but chart/Chart.lock still pins 1.1.3-beta.107. This can cause builds/releases to resolve a different version than declared in Chart.yaml.

Suggested fix
# Regenerate and commit chart/Chart.lock so canary-checker resolves to 1.1.3-beta.150
# (e.g., run Helm dependency update for chart, then commit the lockfile change)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@chart/Chart.yaml` at line 19, Update the Chart.lock file to match the
canary-checker dependency version declared in Chart.yaml. The Chart.lock file
currently pins canary-checker to version 1.1.3-beta.107, but Chart.yaml has been
updated to 1.1.3-beta.150. Regenerate or manually update the Chart.lock file to
ensure the lockfile reflects the dependency version specified in Chart.yaml,
which will ensure consistent builds and releases.

repository: https://flanksource.github.io/charts
- name: flanksource-ui
version: "1.4.278"
Expand Down
Loading