Repository navigation
ci: build Quest from a pinned source commit - #152
Merged
Merged
Conversation
The floating quay.io/parseablehq/quest:main image now expects 202 from stream deletion. This server still deletes synchronously and returns 200, so TestSmokeRbacBasic and TestSmokeRoles fail on main and on every pull request. The previous image digest is no longer available from the registry, so build Quest from the last source commit before that change. The compose files take the image and pull policy from QUEST_IMAGE and QUEST_PULL_POLICY and keep their previous defaults. Refs #151 Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
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
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.
Summary
CI runs the upstream Quest integration suite from the floating
quay.io/parseablehq/quest:mainimage. Since 2026-11-11 that image expects202from stream deletion. This server still deletes synchronously and returns200, soTestSmokeRbacBasicandTestSmokeRolesfail onmainand on every pull request, whatever the pull request changes.The previous image digest has already been removed from the registry, so it cannot be pinned. This change builds Quest from the last source commit before the
202expectation instead.Changes
.github/workflows/integration-test.yaml: each Quest job fetches Quest at a fixed commit, checks that the checkout is that commit, builds the image locally, and runs it.docker-compose-*test*.yaml: the Quest service takes its image and pull policy fromQUEST_IMAGEandQUEST_PULL_POLICY. The defaults are unchanged, so local runs still pullquay.io/parseablehq/quest:main.Cost
Each Quest job now builds the Quest image (a Go test build plus tool downloads) instead of pulling it.
Follow-up
This is a stopgap. Issue #151 tracks porting asynchronous stream deletion, after which the pin should be removed.
Test plan
integration / Quest standalone (smoke)passes on this pull requestintegration / Quest distributed (smoke)passes on this pull requestRefs #151
🤖 Generated with Claude Code
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.