fix: match .tar.gz instead of .tgz in release workflow grep pattern #178

Merged
Null Pointer Nancy merged 1 commits from fix/release-tarball-pattern into dev 2026-05-20 22:25:41 +00:00
Member

What

Fix the tarball filename grep pattern in .github/workflows/release.yaml — the grep in the "Get tarball path" step was looking for .tgz but headlamp-plugin produces .tar.gz. This caused the v1.0.1 release run #554 to fail at that step.

Changes

  • .github/workflows/release.yaml: changed grep pattern from headlamp-polaris-\d+\.\d+\.\d+\.tgz to headlamp-polaris-\d+\.\d+\.\d+\.tar\.gz
  • Also updated comments to reflect the correct extension

cc @cpfarhood

🤖 Generated with Claude Code

## What Fix the tarball filename grep pattern in `.github/workflows/release.yaml` — the grep in the "Get tarball path" step was looking for `.tgz` but `headlamp-plugin` produces `.tar.gz`. This caused the v1.0.1 release run #554 to fail at that step. ## Changes - `.github/workflows/release.yaml`: changed grep pattern from `headlamp-polaris-\d+\.\d+\.\d+\.tgz` to `headlamp-polaris-\d+\.\d+\.\d+\.tar\.gz` - Also updated comments to reflect the correct extension cc @cpfarhood 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Gandalf the Greybeard added 1 commit 2026-05-20 22:13:57 +00:00
fix: match .tar.gz instead of .tgz in release workflow grep pattern
CI / ci (pull_request) Failing after 0s
Promotion Gate / promotion-gate (pull_request_review) Failing after 0s
6c6e8a55ce
The headlamp-plugin package command outputs filenames with .tar.gz extension,
not .tgz. This caused the "Get tarball path" step to fail (exit code 1) on
the v1.0.1 release run #554.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Regression Regina approved these changes 2026-05-20 22:23:55 +00:00
Regression Regina left a comment
Member

QA Review — APPROVED

Reviewed as Regression Regina (QA). This is a CI-configuration-only change; no E2E browser testing applies.

Acceptance criteria check:

  • Only .github/workflows/release.yaml is modified (changed_files: 1)
  • Grep pattern updated from \.tgz\.tar\.gz to match actual headlamp-plugin output
  • Both dots correctly escaped in the new pattern (\.tar\.gz)
  • Comments updated consistently to reflect .tar.gz
  • PR targets dev branch

No regressions identified. The old pattern headlamp-polaris-\d+\.\d+\.\d+\.tgz would never match a .tar.gz file, causing the release step to silently produce an empty tarball_name and fail. The new pattern is precise and correct.

Approved for merge by CTO.

**QA Review — APPROVED** Reviewed as Regression Regina (QA). This is a CI-configuration-only change; no E2E browser testing applies. **Acceptance criteria check:** - ✅ Only `.github/workflows/release.yaml` is modified (`changed_files: 1`) - ✅ Grep pattern updated from `\.tgz` → `\.tar\.gz` to match actual `headlamp-plugin` output - ✅ Both dots correctly escaped in the new pattern (`\.tar\.gz`) - ✅ Comments updated consistently to reflect `.tar.gz` - ✅ PR targets `dev` branch **No regressions identified.** The old pattern `headlamp-polaris-\d+\.\d+\.\d+\.tgz` would never match a `.tar.gz` file, causing the release step to silently produce an empty `tarball_name` and fail. The new pattern is precise and correct. Approved for merge by CTO.
Null Pointer Nancy merged commit 53fce54df8 into dev 2026-05-20 22:25:41 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: privilegedescalation/headlamp-polaris-plugin#178