chore(model-catalog-api): pin image to the CKAN autocomplete fix - #106
Merged
Conversation
Deploys mintproject/model-catalog-api#9, which restores the standard variable autocomplete on the CKAN dataset form. That widget has been returning no suggestions since the catalog moved off the v1.8.0 FastAPI service: `label` filtered with `_eq` so partial terms matched nothing, and `enable_ckan` did not exist so CKAN could not read the response shape. Pin the tag to the commit SHA rather than leaving it on `latest`. This is what most images in values.yaml already do, and it makes the rollout explicit instead of depending on when a pod happens to restart. Chart version bumped so `ct lint` accepts the change. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vh9z6U8JosPJnGqPNQf3fk
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Deploys mintproject/model-catalog-api#9.
Why
The standard-variable autocomplete on the CKAN dataset form returns no suggestions. It has been broken since the model catalog moved off the v1.8.0 FastAPI service to the current Hasura-backed one. Two behaviours the widget relies on were never ported:
labelfilters with_eq, so a partial term matches nothing.enable_ckandoes not exist, so the response is a JSON-LD array that CKAN'sparseCompletionscannot read.model-catalog-api#9 adds
label_containsandenable_ckan. CI on that PR is green and the image is published.Change
components.model_catalog_api.image.taglatestf711a6b49d4ed523f6fd365d1da488669b127735charts/mint/Chart.yamlversion9.0.0-beta.39.0.0-beta.4Pinning the SHA rather than staying on
latestfollows what most images invalues.yamlalready do (cromo, mic-web, mic-api, data-catalog, model-catalog-endpoint, model-catalog-explorer, mint-ui-react). It also makes the rollout explicit: withlatestandpullPolicy: Alwaysthe new image lands whenever a pod next restarts, which is not a deploy anyone chose.The chart version bump is required for
ct lint --check-version-increment.charts/mint/README.mdis updated to match; the helm-docs action will confirm it.Verified
helm lint charts/mint— 0 charts failed.ghcr.io/mintproject/model-catalog-api:f711a6b4…returns 200 from the registry manifest API.Two things to check before merging
main.9.0.0-beta.4. There is an unpushed local branch (fix/config-checksum-annotations) that also bumps to9.0.0-beta.4. Whichever merges second needsbeta.5.🤖 Generated with Claude Code
https://claude.ai/code/session_01Vh9z6U8JosPJnGqPNQf3fk