Skip to content

Examples: use ghcr helloworld image instead of gcr - #7414

Merged
khanhtc1202 merged 3 commits into
pipe-cd:masterfrom
rahulshendre:examples-gcr-to-ghcr
Sep 25, 2026
Merged

khanhtc1202 merged 3 commits into
pipe-cd:masterfrom
rahulshendre:examples-gcr-to-ghcr

Conversation

@rahulshendre

Copy link
Copy Markdown
Contributor

What this PR does:
Switches the examples from gcr.io/pipecd/helloworld to ghcr.io/pipe-cd/helloworld.

The old tags used here (v0.5.0, v0.6.0, v0.27.4) are only available on gcr, so they are updated to v0.48.0, which is on ghcr and supports both amd64 and arm64. Cloud Run can pull public ghcr images now, so the cloudrun examples work as well.

v0/kubernetes/kustomize-remote-base is left unchanged since it depends on an external base repo

Why we need it:

gcr.io/pipecd is being removed.

Which issue(s) this PR fixes:

Part of #7385

Does this PR introduce a user-facing change?:

  • How are users affected by this change: examples pull helloworld from ghcr.io
  • Is this breaking change: No
  • How to migrate (if breaking change): N/A

Screenshots/Videos (for documentation or website changes):
Attach a screenshot or screen recording if your PR modifies any files under docs/ or any .md/.mdx files. Run the site locally using make run/site and verify your changes before submitting.

Signed-off-by: rahulshendre <rahulshendre789@gmail.com>
Copilot AI lite review requested due to automatic review settings September 25, 2026 08:31
@rahulshendre
rahulshendre requested a review from a team as a code owner September 25, 2026 08:31
@netlify

netlify Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for pipecd-site canceled.

Name Link
🔨 Latest commit 01c8763
🔍 Latest deploy log https://app.netlify.com/projects/pipecd-site/deploys/6ab641cedcb6f500082f16d2

@codecov

codecov Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 30.36%. Comparing base (46dccc8) to head (01c8763).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #7414   +/-   ##
=======================================
  Coverage   30.35%   30.36%           
=======================================
  Files         601      601           
  Lines       52622    52622           
=======================================
+ Hits        15975    15977    +2     
+ Misses      35128    35125    -3     
- Partials     1519     1520    +1     
Flag Coverage Δ
. 24.34% <ø> (+0.01%) ⬆️
.-pkg-app-pipedv1-plugin-analysis 29.19% <ø> (ø)
.-pkg-app-pipedv1-plugin-ecs 33.75% <ø> (ø)
.-pkg-app-pipedv1-plugin-kubernetes 59.85% <ø> (ø)
.-pkg-app-pipedv1-plugin-kubernetes_multicluster 60.48% <ø> (ø)
.-pkg-app-pipedv1-plugin-scriptrun 58.22% <ø> (ø)
.-pkg-app-pipedv1-plugin-terraform 39.88% <ø> (ø)
.-pkg-app-pipedv1-plugin-wait 32.22% <ø> (ø)
.-pkg-app-pipedv1-plugin-waitapproval 55.14% <ø> (ø)
.-pkg-plugin-sdk 51.89% <ø> (-0.18%) ⬇️
.-tool-actions-gh-release 19.94% <ø> (ø)
.-tool-actions-plan-preview 28.04% <ø> (ø)
.-tool-codegen-protoc-gen-auth 0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Copilot review overview

🟡 Changes recommended

Two Helm values files omit the required image.tag, causing invalid rendered image references.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 2 Medium severity

Open (2)
What changed in this PR

Updates examples from the deprecated GCR Hellworld image to the GHCR image, using compatible tags.

Changes:

  • Migrates image references to ghcr.io/pipe-cd/helloworld.
  • Updates selected tags to v0.48.0.
  • Updates the event-watcher image replacement regex.
File Summary
examples/​v1/​local-modules/​helm-charts/​helloworld/​values.yaml GHCR repository updated; image.tag is still missing.
examples/​v1/​kubernetes/​helm-local-chart/​values.yaml GHCR repository and tag updated.
examples/​v0/​local-modules/​kustomize-bases/​helloworld/​deployment.yaml GHCR image updated.
examples/​v0/​local-modules/​helm-charts/​helloworld/​values.yaml GHCR repository updated; image.tag is still missing.
examples/​v0/​kubernetes/​script-run/​deployment.yaml GHCR image and tag updated.
examples/​v0/​kubernetes/​kustomize-local-base/​kustomization.yaml GHCR image and tag updated.
examples/​v0/​kubernetes/​helm-local-chart/​values.yaml GHCR repository and tag updated.
examples/​v0/​cloudrun/​wait-approval/​service.yaml GHCR image and tag updated.
examples/​v0/​cloudrun/​simple/​service.yaml GHCR image and tag updated.
examples/​v0/​cloudrun/​secret-management/​service.yaml GHCR image and tag updated.
examples/​v0/​cloudrun/​canary/​service.yaml GHCR image and tag updated.
examples/​v0/​cloudrun/​analysis/​service.yaml GHCR image and tag updated.
examples/​.pipe/​event-watcher-using-regex.yaml Image replacement regex updated for GHCR.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread examples/v0/local-modules/helm-charts/helloworld/values.yaml
Comment thread examples/v1/local-modules/helm-charts/helloworld/values.yaml

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.

Copilot review overview

🔵 Needs a closer look

The event-watcher regex does not match the multi-digit versions used by the examples.

Review effort: Lite
Findings: None

Resolved since last review (2)

@khanhtc1202 khanhtc1202 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@khanhtc1202
khanhtc1202 enabled auto-merge (squash) September 25, 2026 09:46
@khanhtc1202
khanhtc1202 merged commit 9bf5403 into pipe-cd:master Sep 25, 2026
66 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for contributing to PipeCD, @rahulshendre! The changes in this pull request will be part of the upcoming release!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants