Skip to content

fix: paginate all tags to build olm catalog#221

Merged
GrigoryPervakov merged 1 commit into
mainfrom
fix/catalog-template-pagination
Jun 9, 2026
Merged

fix: paginate all tags to build olm catalog#221
GrigoryPervakov merged 1 commit into
mainfrom
fix/catalog-template-pagination

Conversation

@GrigoryPervakov

Copy link
Copy Markdown
Member

Why

Currently, the catalog is built on the latest 30 images

What

Use pagination to read all tags

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.

Pull request overview

This PR updates the OLM catalog template generation script to fetch all bundle tags from the GitHub Packages (GHCR) API instead of being limited to the default first page of results, ensuring the catalog includes more than the latest ~30 images.

Changes:

  • Request up to 100 versions per page from the GitHub API (per_page=100).
  • Add pagination by following the Link: ... rel="next" response header and aggregating tags across pages.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"${next_url}" 2>/dev/null \
| jq -r '.[].metadata.container.tags[]' 2>/dev/null
# Follow `Link: <url>; rel="next"` until exhausted; multi-arch+attestation rows overflow page size.
next_url=$(grep -i '^link:' "${headers_file}" | sed -n 's/.*<\([^>]*\)>; rel="next".*/\1/p' | tr -d '\r')
Comment on lines +18 to +19
headers_file=$(mktemp)
next_url="${gh_api_url}"
@GrigoryPervakov GrigoryPervakov marked this pull request as ready for review June 9, 2026 21:53
@GrigoryPervakov GrigoryPervakov merged commit 30368f9 into main Jun 9, 2026
21 checks passed
@GrigoryPervakov GrigoryPervakov deleted the fix/catalog-template-pagination branch June 9, 2026 21:53
@mintlify

mintlify Bot commented Jun 9, 2026

Copy link
Copy Markdown

Docs PR opened: ClickHouse/mintlify-docs-dev#161

Synced upstream Kubernetes Operator docs, adding a monitoring guide, an External Secret section, and the AdditionalPort API reference.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants