Skip to content

Store ArtifactHub metadata by version#440

Open
AkashKumar7902 wants to merge 1 commit into
kubewarden:mainfrom
AkashKumar7902:issue-160-artifacthub-version-history
Open

Store ArtifactHub metadata by version#440
AkashKumar7902 wants to merge 1 commit into
kubewarden:mainfrom
AkashKumar7902:issue-160-artifacthub-version-history

Conversation

@AkashKumar7902

Copy link
Copy Markdown
Contributor

Summary

  • write each released policy ArtifactHub metadata under <policy>/<version>/
  • copy the release README into the same version directory
  • remove the old flat artifacthub-pkg.yml and README.md files during migration so Artifact Hub indexes the multi-version layout

Fixes #160

Testing

  • yq e . .github/workflows/release.yml >/dev/null
  • extracted the edited push-artifacthub shell block and ran bash -n after replacing GitHub expressions
  • git diff --check

Reference: Artifact Hub documents multi-version Kubewarden policy packages as <package>/<version>/{README.md,artifacthub-pkg.yml}.

Signed-off-by: Akash Kumar <meakash7902@gmail.com>

@viccuad viccuad 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.

This is a change with the potential to cause huge disruptions.

I would like to see a test where a repo with these changes is used in https://staging.artifacthub.io/ as source prior to merging.

@flavio flavio moved this from Pending review to Blocked in Kubewarden Admission Controller Apr 28, 2026
@flavio flavio moved this from Blocked to In Progress in Kubewarden Admission Controller Apr 28, 2026
@flavio flavio moved this from In Progress to Todo in Kubewarden Admission Controller Apr 28, 2026
@AkashKumar7902

Copy link
Copy Markdown
Contributor Author

Agreed. This changes the layout Artifact Hub indexes, so a staging import is the right gate before merging.

I cannot honestly complete that staging.artifacthub.io validation from this environment because it needs a repo/source registered in Artifact Hub staging. I will keep this PR blocked until we have that test result. Once the staging source is configured and indexed, I will update the PR with the result and any changes needed from the import logs.

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.

Policy version history

3 participants