Skip to content

release: v7.0.0 - #538

Merged
davidgamero merged 3 commits into
Azure:mainfrom
bosesuneha:release/v7.0.0
Jul 16, 2026
Merged

release: v7.0.0#538
davidgamero merged 3 commits into
Azure:mainfrom
bosesuneha:release/v7.0.0

Conversation

@bosesuneha

Copy link
Copy Markdown
Member

Summary

Cuts v7.0.0 of Azure/k8s-deploy. Merging this PR will trigger release-pr.yml, which tags v7.0.0, moves the v7 major tag, and publishes the GitHub Release.

Breaking change

  • fix: confine manifest paths to workspace and harden URL fetcher #528manifests: inputs that resolve outside GITHUB_WORKSPACE (via ../, absolute paths, or symlinks) now fail the action with a clear error. Workflows that depend on reading manifests from outside the workspace (uncommon; mostly self-hosted runners) must move those files into the workspace before invoking this action. This is why the major version is bumped.

What's included since v6.0.0

14 commits — see CHANGELOG.md diff for the full list. Headlines:

PR Type Summary
#528 Breaking + Security Path containment to GITHUB_WORKSPACE, URL fetcher hardening
#527 Hardening Pin release workflow to commit SHA
#523–537 Security 12 dependabot updates (undici, esbuild, codeql, actions group, @types/node)

File changes

  • CHANGELOG.md — new [7.0.0] entry at top, formatted to match prior entries
  • package.json + package-lock.json6.0.07.0.0
  • README.md — bumped 11 usage examples from Azure/k8s-deploy@v5@v7 (note: @v5 was never updated to @v6 in docs: update action version references in README to latest majors #500, so this catches up to two majors at once)

Pre-merge checklist

  • npm ci clean
  • npm run typecheck clean
  • npm run build clean (esbuild bundle 1.1mb)
  • npm test — 271/271 passing (one cleanup-timing test from fix: confine manifest paths to workspace and harden URL fetcher #528 was flaky on first run; consistently passes on retry, worth a follow-up)
  • CHANGELOG entry matches existing format, version bumped in package.json + lockfile
  • README @v5@v7 (11 occurrences)
  • Reviewed by a second maintainer

Post-merge verification

  • Release Project workflow run is green
  • gh release view v7.0.0 --repo Azure/k8s-deploy exists with the changelog body
  • git ls-remote --tags https://github.com/Azure/k8s-deploy.git | grep -E 'v7(\.0\.0)?$' shows both v7 and v7.0.0 at the merge SHA
  • Trigger one downstream consumer pinned to @v7 and confirm it runs

Follow-ups (not blocking)

  • Stabilize the removes temp file on mid-stream response error test in fileUtils.test.ts — likely needs a deterministic wait for the cleanup promise rather than waitForCleanup().

@bosesuneha
bosesuneha requested a review from a team as a code owner June 23, 2026 18:37
@ejaifeobuks ejaifeobuks mentioned this pull request Jun 29, 2026
Comment thread README.md
bosesuneha and others added 2 commits June 30, 2026 10:43
Addresses review feedback on PR Azure#538 by documenting the recommended
practice of pinning the action to a full commit SHA for supply-chain
security, in line with GitHub's hardening guidance.
@bosesuneha
bosesuneha requested a review from Tatsinnit June 30, 2026 18:01

@davidgamero davidgamero left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@davidgamero
davidgamero merged commit c56f17f into Azure:main Jul 16, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants