Repository navigation
NGSOK-1746 PySpark sdist without hadoop-cloud, and a correct CI guard - #62
Merged
Merged
Conversation
giggsoff
marked this pull request as draft
September 3, 2026 06:43
giggsoff
force-pushed
the
feature/NGSOK-1746-followup
branch
2 times, most recently
from
September 3, 2026 07:09
99662e4 to
19f2d3d
Compare
giggsoff
marked this pull request as ready for review
September 4, 2026 09:00
…-cloud Packages published by a release-triggered run are linked to that release and listed among its assets, so the Maven deploy moves to the tag push and the release event keeps only the tarballs. The PySpark sdist moves to the plain hadoop3 leg, since it packages the assembly jars and was shipping the AWS SDK bundle, and the CI guard now reports a still-running run as running rather than as missing. A new job attaches the CycloneDX SBOM and its grype vulnerability report, as the kyuubi release workflow does.
The install lifecycle passes through pre-integration-test, so setup-integration-test-env.sh ran with the default image tag N/A and rebuilt the images through minikube's Docker daemon, which is containerd-backed and fails to boot buildkit. Point that step at the images already built and loaded so the script skips the rebuild.
giggsoff
force-pushed
the
feature/NGSOK-1746-followup
branch
from
September 4, 2026 09:04
945649f to
b5ea645
Compare
werzerbb
approved these changes
Sep 4, 2026
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.
Two follow-ups to #43. The Maven deploy moves from the release event to the tag push, because packages published by a release-triggered run are linked to that release and listed among its assets — 32
org.apache.spark.spark-*_2.13 (maven)lines on4.2.0.2-4.4.0-0, where arenadata/kyuubi, which deploys from a tag-push workflow, has none. The PySpark sdist moves to the plainhadoop3leg so it stops shipping the AWS SDK bundle, and the CI guard that reported a still-running run as missing is fixed here, since it landed on the branch after #43 merged.Also attaches the CycloneDX SBOM and its grype vulnerability report to the release, in a job of its own.