Skip to content

egress: bump version to v2.3.2 for SigNoz cardinality fix#366

Merged
myleshorton merged 1 commit into
mainfrom
fisk/bump-egress-v2.3.2
May 21, 2026
Merged

egress: bump version to v2.3.2 for SigNoz cardinality fix#366
myleshorton merged 1 commit into
mainfrom
fisk/bump-egress-v2.3.2

Conversation

@myleshorton

Copy link
Copy Markdown
Contributor

Why

The auto-bump step in release-egress.yml failed on the #365 merge — the workflow's GITHUB_TOKEN can't push directly to `main` (branch protection requires PRs):

```
auto patch-bump: v2.3.1 -> v2.3.2
[main 5e5a271] chore: bump egress version to v2.3.2 [skip ci]
remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: - Changes must be made through a pull request.
```

Fix

The workflow already handles manual bumps via PR (takes the "manual bump detected" branch, skips auto-commit/push). Bumping via this PR is the path of least resistance.

When merged: the next workflow run reads `current=v2.3.2 != last_tag=v2.3.1`, picks the manual-bump branch, skips the protected-branch fight, builds linux/amd64+arm64, publishes the v2.3.2 GitHub Release.

Then the running egress can be updated to pick up the otelhttp metric suppression from #365 and the SigNoz cardinality spike subsides.

Permanent fix (separate PR)

Branch protection will keep biting future releases. Options: allowlist github-actions[bot] for direct push, switch the workflow to open a PR instead, or use a GitHub App with bypass permission.

🤖 Generated with Claude Code

The auto-bump in release-egress.yml failed on the #365 merge because
the GITHUB_TOKEN cannot push directly to main (branch protection requires
PRs). Bumping via a PR lands in the "manual bump detected" branch of the
workflow, which skips the auto-push and proceeds to build+release.

Once this merges, the workflow will publish v2.3.2 with the
otelhttp /ws cardinality fix from #365.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 21, 2026 17:17
@myleshorton myleshorton merged commit 7e17969 into main May 21, 2026
1 check failed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository-wide version constant to v2.3.2 so the egress release workflow detects a manual bump (avoiding protected-branch direct pushes) and can publish the v2.3.2 release that includes the SigNoz cardinality fix from #365.

Changes:

  • Bump common.Version from v2.3.1 to v2.3.2.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to 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.

2 participants