Skip to content

Commit

Permalink
Merge #86693 #86694
Browse files Browse the repository at this point in the history
86693: release: use `stable` tag for preflight image r=celiala a=rail

Previously, we used a fixed version of the prflight docker image to
submit certification results to Red Hat Connect.

Unfortunately, their API is not stable yet, so we ended up with failures
submitting the certification result.

This PR changes the preflight docker image tag to `stable` with hope we
don't need to update the pinned image version on the release branches.

Release justification: Blocks the release process

Release note: None

86694: colexec: remove a redundant assertion r=yuzefovich a=yuzefovich

This commit removes a redundant assertion from the sort code where we
previously enforced that we had at least one partition to sort (which is
always the case). The assertion seems useless since the code works just
fine even if zero partitions are given (which will never be the case
since the sort operator short-circuits on a zero-length batch).

Release justification: low-risk cleanup.

Release note: None

Co-authored-by: Rail Aliiev <[email protected]>
Co-authored-by: Yahor Yuzefovich <[email protected]>
  • Loading branch information
3 people committed Aug 25, 2022
3 parents eaa7eaf + 9580d03 + b2cbc0d commit 1b2ebf8
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 136 deletions.
2 changes: 1 addition & 1 deletion build/release/teamcity-publish-redhat-release.sh
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,6 @@ docker run \
-v $PWD/artifacts:/artifacts \
-v ~/.docker/config.json:/temp-authfile.json:ro \
-v ~/.docker/config.json:/tmp/docker/config.json:ro \
quay.io/opdev/preflight:1.1.0 check container \
quay.io/opdev/preflight:stable check container \
"${rhel_repository}:${build_name}" --submit
tc_end_block "Run preflight"
Loading

0 comments on commit 1b2ebf8

Please sign in to comment.