feat: add Pulp upload scripts and pipeline integration#2570
Open
vamahaja wants to merge 1 commit into
Open
Conversation
djgalloway
reviewed
Apr 23, 2026
Contributor
|
jenkins retest |
Contributor
|
My comment is a nit. I'd like to push this to Jenkins to make sure we don't run over the character limit we saw a few months ago but otherwise okay with this. Exciting stuff! 👍 |
zmc
reviewed
Apr 23, 2026
Member
zmc
left a comment
zmc
left a comment
There was a problem hiding this comment.
Generally looks great! I agree with David's suggestion. I think we should likely have PULP_URL be a parameter defaulting to the host we have in openshift.
vamahaja
force-pushed
the
upload-to-pulp
branch
3 times, most recently
from
210595f to
5cee478
Compare
vamahaja
commented
Apr 27, 2026
vamahaja
force-pushed
the
upload-to-pulp
branch
6 times, most recently
from
93eb758 to
4bb903f
Compare
Contributor
|
@vamahaja is this ready now? |
Member
Author
@djgalloway testing couple of scenarios, will complete it ASAP |
vamahaja
force-pushed
the
upload-to-pulp
branch
12 times, most recently
from
58c7717 to
e3299fc
Compare
Add an optional path to publish built RPM/DEB artifacts to Pulp alongside the
existing Chacra flow.
- Introduce a `PULP_UPLOAD` boolean parameter (default false) on the `ceph-dev` job.
- Add `scripts/setup_pulp.sh` to install `pulp-cli` (with pulp-cli-deb) via uv and
write client config from Jenkins credentials.
- Add `scripts/pulp_upload.sh` to upload discovered packages, attach them to the
target repository, create a publication, and publish a distribution with
project/version/ref/arch/sha labels.
- Update the Jenkinsfile to install/configure the Pulp client when forcing a
rebuild with Pulp upload while Chacra already has artifacts, and to invoke
`pulp_upload.sh` after `chacra_upload.sh` when `THROWAWAY` is not `true` and
`PULP_UPLOAD` is `true`.
Signed-off-by: Vaibhav Mahajan <vamahaja@redhat.com>
Member
Author
djgalloway
reviewed
May 11, 2026
Comment on lines
+13
to
+33
| resolve_branch_from_ceph_version() { | ||
| local branch="" | ||
| if [[ $CEPH_VERSION =~ ^21.* ]]; then | ||
| branch="main" | ||
| elif [[ $CEPH_VERSION =~ ^20.* ]]; then | ||
| branch="tentacle" | ||
| elif [[ $CEPH_VERSION =~ ^19.* ]]; then | ||
| branch="squid" | ||
| elif [[ $CEPH_VERSION =~ ^18.* ]]; then | ||
| branch="reef" | ||
| elif [[ $CEPH_VERSION =~ ^17.* ]]; then | ||
| branch="quincy" | ||
| elif [[ $CEPH_VERSION =~ ^16.* ]]; then | ||
| branch="pacific" | ||
| else | ||
| log "ERROR: Unsupported Ceph version '${CEPH_VERSION}'" \ | ||
| "(expected 16.x–20.x for branch mapping)" | ||
| exit 1 | ||
| fi | ||
| printf '%s\n' "$branch" | ||
| } |
Contributor
There was a problem hiding this comment.
Is it possible to reuse https://github.com/ceph/ceph-build/blob/main/scripts/build_utils.sh#L13-L78? Having another distro map hard coded somewhere is not scalable.
Member
There was a problem hiding this comment.
Maybe we ought to drop a yaml or json file in, say, the repo root for this
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
3 participants
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.
This PR will introduce an optional path to publish built RPM/DEB artifacts to Pulp alongside the existing Chacra flow.
PULP_UPLOADboolean parameter (default false) on theceph-devjob.scripts/setup_pulp.shto installpulp-cli(with pulp-cli-deb) via uv and write client config from Jenkins credentials.scripts/pulp_upload.shto upload discovered packages, attach them to the target repository, create a publication, and publish a distribution with project/version/ref/arch/sha labels.pulp_upload.shafterchacra_upload.shwhenTHROWAWAYis nottrueandPULP_UPLOADistrue.