Skip to content

Update Helm chart version to 2.0.1 for tracebloc, - #19

Merged
saadqbal merged 1 commit into
mainfrom
develop
Mar 4, 2026
Merged

Update Helm chart version to 2.0.1 for tracebloc, #19
saadqbal merged 1 commit into
mainfrom
develop

Conversation

@saadqbal

@saadqbalsaadqbal commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

reflecting minor enhancements and improvements in deployment configurations.


Note

Low Risk
Low risk: only a chart version bump plus a new opt-in release script that tags/pushes Git tags; no runtime application logic changes.

Overview
Updates the tracebloc Helm chart version from 2.0.0 to 2.0.1.

Adds scripts/tag-from-chart.sh to derive a Git tag from tracebloc/Chart.yaml and create/push it, with options for --tracebloc-prefix and --dry-run, and a guard against tagging an existing version.

Written by Cursor Bugbot for commit f007c90. This will update automatically on new commits. Configure here.

…hancements and improvements in deployment configurations.
@saadqbal
saadqbal merged commit 3d72109 into mainMar 4, 2026
11 checks passed

@cursorcursorBot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

if git rev-parse "$TAG" &>/dev/null; then
echo "Error: Tag ${TAG} already exists." >&2
exit 1
fi

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Tag existence check resolves any ref, not just tags

Medium Severity

git rev-parse "$TAG" resolves any git ref — branches, tags, and even commit-like strings — not just tags. If a branch happens to share the same name (e.g., v2.0.1), the script will incorrectly refuse to create the tag, reporting it already exists. The check needs to be scoped to tags specifically, e.g. via git rev-parse "refs/tags/$TAG" or git show-ref --verify "refs/tags/$TAG".

Fix in CursorFix in Web

exit 0
;;
esac
done

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Unknown arguments silently ignored risking unintended tag push

Medium Severity

The case statement has no * default branch, so unrecognized flags are silently ignored. A misspelled --dry-run (e.g., --dryrun, --dry_run, --dry-Run) would be silently skipped and the script would proceed to actually create and push a git tag — the opposite of what the user intended. For a script with irreversible side effects, unknown arguments warrant an error exit.

Fix in CursorFix in Web

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@saadqbal