Skip to content

Use local Updatecli CLI reference sync#736

Open
AkashKumar7902 wants to merge 1 commit intokubewarden:mainfrom
AkashKumar7902:issue-677-local-cli-ref-updatecli
Open

Use local Updatecli CLI reference sync#736
AkashKumar7902 wants to merge 1 commit intokubewarden:mainfrom
AkashKumar7902:issue-677-local-cli-ref-updatecli

Conversation

@AkashKumar7902
Copy link
Copy Markdown

@AkashKumar7902 AkashKumar7902 commented Apr 26, 2026

Summary

  • replace the two remote ghcr.io/updatecli/policies/file:0.2.1 compose entries with a local Updatecli manifest
  • copy cli-docs.md from kubewarden/kwctl and kubewarden/policy-server before the existing header/PR step runs
  • remove the now-unused values files for the defunct remote policy

Fixes #677

Checklist

AI usage

OpenAI Codex assisted with this contribution. I reviewed the generated changes and am responsible for them.

Testing

  • rg -n "ghcr\.io/updatecli/policies/file|update-kwctl-cli-ref-docs|update-policy-server-cli-ref-docs" updatecli .github || true
  • yq e . updatecli/update-cli-ref-docs.yaml >/dev/null
  • yq e . updatecli/updatecli.d/copy-cli-ref-docs.yaml >/dev/null
  • verified cli-docs.md exists on kubewarden/kwctl and kubewarden/policy-server main branches with gh api
  • git diff --cached --check

I could not run the Updatecli compose locally because updatecli is not installed, and pulling ghcr.io/updatecli/updatecli:v0.94.0 with Docker is denied by GHCR in this environment.

Signed-off-by: Akash Kumar <meakash7902@gmail.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 26, 2026

Deploy Preview for docs-kubewarden-io ready!

Name Link
🔨 Latest commit 9b091b1
🔍 Latest deploy log https://app.netlify.com/projects/docs-kubewarden-io/deploys/69ee1028cc018300087f9257
😎 Deploy Preview https://deploy-preview-736--docs-kubewarden-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jhkrug
Copy link
Copy Markdown
Contributor

jhkrug commented Apr 28, 2026

Hello @AkashKumar7902,

You appear not to have used our docs PR template, which included the following:

Please reinstate this, and answer accordingly. Feel free to reach out to me to discuss in person, if necessary.

Thank you

@jvanz jvanz moved this from Pending review to Blocked in Kubewarden Admission Controller Apr 28, 2026
@AkashKumar7902
Copy link
Copy Markdown
Author

Thanks, I missed that. I restored the checklist in the PR description and confirmed the AI Policy acknowledgement.

@AkashKumar7902
Copy link
Copy Markdown
Author

Following up on the merge conflict. Looking at main, jvanz already addressed the underlying breakage in #677 via 8de45b0 / cbe9c02 / a48d707 (commits on main) — those switched the updatecli sources to point at the kubewarden-controller monorepo (crates/kwctl/cli-docs.md, crates/policy-server/cli-docs.md) using the existing ghcr.io/updatecli/policies/file:0.2.1 policy.

This PR took a different angle (defining the copy logic locally with updatecli's git scm + file kind so we no longer depend on the external updatecli policy image). Now that main has a working sync via the upstream policy, the local approach is no longer required to fix the failing job from #677.

Happy to either:

  1. Close this as superseded by the main-branch fix.
  2. Rebase and rework on top of main if you still want the local-policy independence (the conflict is in updatecli/values.d/ so I'd be replacing the new files rather than deleting them).

@viccuad @jvanz @flavio — what would you prefer? Defaulting to (1) unless you'd like (2).

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

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Workflow for automatically updating CLI reference docs is failing

3 participants