Skip to content

ceph-dev-pipeline/build/Jenkinsfile: fix crimson-debug#2494

Merged
zmc merged 1 commit intoceph:mainfrom
Matan-B:wip-matanb-crimson-debug-fix
Nov 10, 2025
Merged

ceph-dev-pipeline/build/Jenkinsfile: fix crimson-debug#2494
zmc merged 1 commit intoceph:mainfrom
Matan-B:wip-matanb-crimson-debug-fix

Conversation

@Matan-B
Copy link
Contributor

@Matan-B Matan-B commented Nov 10, 2025

When ceph-dev-pipeline was introduced, crimson-debug cmake flag was left out. The corresponding ceph_build_args_from_flavor from scrips/build_utils.sh is defined correctly.

When ceph-dev-pipeline was introduced, crimson-debug cmake flag
was left out. The corresponding ceph_build_args_from_flavor from
scrips/build_utils.sh is defined correctly.

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
@Matan-B Matan-B requested review from athanatos, dmick and zmc November 10, 2025 13:18
@Matan-B Matan-B added this to Crimson Nov 10, 2025
@Matan-B Matan-B moved this to Tested in Crimson Nov 10, 2025
@Matan-B Matan-B requested a review from djgalloway November 10, 2025 13:19
@Matan-B
Copy link
Contributor Author

Matan-B commented Nov 10, 2025

Can you please help me understand better the new build pipeline:

  • when is build_utils.sh used in our CI? Since it seems builds published at shaman/quay use the Jenkinsfile updated here. However, we include the script under !include-raw: in ceph-dev-trigger.yml
  • build_utils.sh uses ceph_build_args_from_flavor to pass CEPH_EXTRA_RPMBUILD_ARGS (--with tcmalloc or --with crimson) - where is the corresponding extra rpm flag set in the Jenkinsfile? Found rpmbuild_args

Thank you!

@dmick
Copy link
Member

dmick commented Nov 10, 2025

build_utils.sh has a lot of stuff in it (@zmc has been breaking it up but it's still quite the catchall). The intent was to use relevant pieces in ceph-dev-pipeline, or recode them where applicable. This one got missed.

Copy link
Member

@zmc zmc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this, and sorry that got lost as we switched to ceph-dev-pipeline right around the time of the crimson flavor split

@zmc zmc merged commit 1fbe1d4 into ceph:main Nov 10, 2025
1 check passed
@Matan-B Matan-B moved this from Tested to Merged (Main) in Crimson Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Merged (Main)

Development

Successfully merging this pull request may close these issues.

3 participants