Merge pull request #2346 from Matan-B/wip-matanb-clean-with-seastar

ceph-dev*: final transition to WITH_CRIMSON

Reviewed-by: Radosław Zarzyński <rzarzyns@redhat.com>

Merge pull request #2345 from dmick/ceph-release-containers-defaults

ceph-release-containers: tweak the parameter defaults

ceph-dev*: final transition to WITH_CRIMSON

Follow up to: https://github.com/ceph/ceph-build/pull/2329
Now that https://github.com/ceph/ceph/pull/62325 is merged.

Signed-off-by: Matan Breizman <mbreizma@redhat.com>

ceph-release-containers: tweak the parameter defaults

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2344 from dmick/fix-include-raw

PR 2315 added back in an 'include-raw' which is deprecated

PR 2315 added 'include-raw' which is deprecated

should use include-raw-verbatim.  Also, {} must be doubled.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2342 from ceph/add-ceph-release-containers

Add ceph-release-containers job

Merge pull request #2315 from batrick/tentacle-kickoff

Tentacle kickoff!

Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

Add ceph-release-containers job

Builds prerelease versions of ceph container images: two arch-specific
images and one 'manifest-list' image.  Pushed all to quay.ceph.io/ceph,
in a private set of repos (prerelease-amd64, prerelease-arm64, and
prerelease) for testing and signoff.  Once moved, the script
container/make-manifest-list.py can be used with the --promote flag to
promote them from quay.ceph.io to quay.io for public release.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2341 from dmick/remove-container-build-from-ceph-build

ceph-build: remove container build

*: update relevant scripts and triggers for tentacle

Signed-off-by: Patrick Donnelly <batrick@batbytes.com>

scripts: add tentacle

Signed-off-by: Patrick Donnelly <batrick@batbytes.com>

Merge pull request #2340 from ceph/sepia-fog-fixes

sepia-fog-images: Mark down systems we borrow instead of lock

ceph-build: remove container build

The container must be built from signed packages, and signing
is not part of the Jenkins build.  So we'll create two new jobs
to build the prerelease containers (so that builders don't have
to have access to both amd64 and arm64 build hosts).

Signed-off-by: Dan Mick <dan.mick@redhat.com>

sepia-fog-images: Mark down systems we borrow instead of lock

Signed-off-by: David Galloway <david.galloway@ibm.com>

Merge pull request #2333 from ceph/no-setup-arm64

ceph*setup: Do not run on arm64 builders

Merge pull request #2329 from Matan-B/wip-matanb-add-with-crimson

ceph-dev*: intermediate transition to WITH_CRIMSON

ceph*setup: Do not run on arm64 builders

We're already low on ARM builders.  This is a waste of resources.

Signed-off-by: David Galloway <david.galloway@ibm.com>

Merge pull request #2332 from ceph/tf-fixes

ansible: Fixes to podman subuid/subgid checks

ansible: Fixes to podman subuid/subgid checks

I think this was working during my testing because sub{u,g}id_check.rc /wasn't/ 0.  I had manually modified the system so it wasn't in the same state as a fresh cloud instance.

I manually ran this playbook on an instance and confirmed it gets past the podman setup now.

Signed-off-by: David Galloway <david.galloway@ibm.com>

Merge pull request #2331 from ceph/2284-followup

Install and configure podman on Ubuntu builders

ansible: Install and configure podman on Ubuntu builders

Signed-off-by: David Galloway <david.galloway@ibm.com>

ansible: Remove ceph-docs deps from builder playbook

We're building docs in readthedocs.org infra now.

Signed-off-by: David Galloway <david.galloway@ibm.com>

Merge pull request #2328 from phlogistonjohn/jjm-bwc-container_only_pr

scripts: add build-with-container.py and friends to container_only_pr

ceph-dev*: intermediate transition to WITH_CRIMSON

See: https://github.com/ceph/ceph/pull/61672
This PR changes any WITH_SEASTAR / --with-seastar to WITH_CRIMSON /
--with crimson respectively. We should let our builders add the new flags
and once the PR is merged we could clean up the unused WITH_SEASTAR.

Signed-off-by: Matan Breizman <mbreizma@redhat.com>

scripts: add build-with-container.py and friends to container_only_pr

Add the build-with-container.py script and the helper shell script and
Dockerfile that support the builder image to the container_only_pr
patterns in build_utils.sh.  This should save time and resources when
contributing patches to only these files in the ceph/ceph repo.

Signed-off-by: John Mulligan <jmulligan@redhat.com>

Merge pull request #2326 from dmick/pr-only-for

Rework changed-files test (doc-only, container-only)

Merge pull request #2327 from ceph/wip-skip-interp

ansible: Add tag to python interpreter var setting tasks

ansible: Add tag to python interpreter var setting tasks

I'm working on a combination of terraform and ansible to quickly
spin up instances in IBM Cloud so they can join Jenkins as builders.

Part of this process involves creating a virtualenv and running ansible
from it.  python-jenkins needs to be installed in the virtualenv and
setting the python interpreters inside the playbook breaks that.

This PR will let me `--skip-tags interpreter`

Signed-off-by: David Galloway <david.galloway@ibm.com>

Rework changed-files test (doc-only, container-only)

Take an array of patterns to compare each changed file against.
For docs, use the current suggestions from
https://tracker.ceph.com/issues/69650

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2325 from dmick/ceph-dev-new-trigger-7-days

ceph-dev-new-trigger: use new feature, 7-day-old or newer branches

ceph-dev-new-trigger: use new feature, 7-day-old or newer branches

Actually modify the trigger job to use choosing-strategy: ancestry and
maximum-age: 7, so that only branches newer than 7 days old get built.

Also set build-name to the branch being triggered, for clarity.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2324 from dmick/jjb6

update all jobs to jenkins-job-builder v6; add 'choose from 7 days old' to ceph-dev-new-trigger

jenkins-job-builder,ceph-build-pull-requests: fetch and use jjb 6

currently it's actually a custom version of jjb6, to add
new choosing-strategy: ancestry, to be able to use
maximum-age in days to choose a branch to build.  This will
change to 6.5 once that branch is released.

Signed-off-by: Dan Mick <dan.mick@redhat.com>
sq

Update all jobs with ${VAR} syntax to ${{VAR}} for jjb6 compatibility

jenkins-job-builder 6 changed the interpretation of {var} to *always*
refer to a jjb macro; this is true even for ${VAR}, which we use in a
lot of place for Jenkins variable expansion (although not all; it's
unclear whether the {} are or ever were necessary, or in what exact
contexts).  So far, we've pinned jjb to <6, but we need new
functionality.

In any event, rather than changing all of them to the same {}-less style
that might work, change the ${} style to ${{}} and leave $ alone.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Update all jobs for jjb6: !include-raw -> !include-raw-verbatim

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2323 from ceph/pipeline-branch

ceph-dev-pipeline: Add CEPH_BUILD_BRANCH param

ceph-dev-pipeline: Add CEPH_BUILD_BRANCH param

So that we can easily test in-development changes to this pipeline.

Signed-off-by: Zack Cerza <zack@cerza.org>

Merge pull request #2320 from rhcs-dashboard/node-install-for-windows-test

ceph-website: setup nodejs for building the website

ceph-website: setup nodejs for building the website

Signed-off-by: Nizamudeen A <nia@redhat.com>

Merge pull request #2322 from dmick/wip-quiet-submodule-update

ceph-{dev-,}setup: Make git submodule operations much quieter.

ceph-{dev-,}setup: Make git submodule operations much quieter.

After f381e3f05fcb08a7b1a3a1aa8f86e066bd4718ba, which is quite the
win for ceph-dev-new-setup, credit to zack@redhat.com

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2321 from ceph/pipeline-branch

Correct ceph-dev-pipeline branch

Correct ceph-dev-pipeline branch

Signed-off-by: Zack Cerza <zack@cerza.org>

Merge pull request #2310 from ceph/jenkinsfile

Add ceph-dev-pipeline

Merge pull request #2316 from dmick/wip-remove-quincy

Remove quincy builds/mentions from most builds/build configs.

ceph-dev-pipeline: Add more info to build desc

BRANCH, DISTROS, ARCHS, SETUP_BUILD_ID with a link to the setup build.

Signed-off-by: Zack Cerza <zack@cerza.org>

Add ceph-dev-pipeline-trigger

This job serves to trigger ceph-dev-pipeline via webhook payloads, but is
opt-in. This implementation initially targets only GitHub push event payloads.
On each push, it will inspect the head commit for [git trailers][0]. The trailer
required to opt in is "CI-PIPELINE: true". If that trailer is found, the job
looks for these others:

ARCHS, CI_COMPILE, CI_CONTAINER, CI_PIPELINE, DISTROS, DWZ, FLAVOR, SCCACHE

Any that are found are passed as parameters.

[0] https://git-scm.com/docs/git-interpret-trailers

Signed-off-by: Zack Cerza <zack@cerza.org>

ceph-dev-pipeline: optionally skip package build

Signed-off-by: Zack Cerza <zack@cerza.org>

ceph-dev-pipeline: Report build status to shaman

Signed-off-by: Zack Cerza <zack@cerza.org>

ceph-dev-pipeline: build container separately

Signed-off-by: Zack Cerza <zack@cerza.org>

ceph-*build: move container build to separate script

Signed-off-by: Zack Cerza <zack@cerza.org>

Merge pull request #2319 from ceph/wip-update-java-alternatives

Updating java openjdk to version 17 on rhel and debian builders, I ha…

Updating java openjdk to version 17 on rhel and debian builders, I have added the update-alternatives command to make sure we set the OS to use the new java version

Signed-off-by: Adam Kraitman <akraitma@li-8b09b2cc-35b7-11b2-a85c-cd1dbade58f9.ibm.com>

ceph-*build: Update status in separate script

Signed-off-by: Zack Cerza <zack@redhat.com>

ceph-dev-new-setup: allow cd-pipeline to copy artifacts

Signed-off-by: Zack Cerza <zack@redhat.com>

Add ceph-dev-pipeline

This may ultimately replace the existing ceph-*build jobs; for now, it is a
work in-progress.

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #2318 from dmick/wip-ceph-build-quay-creds

ceph-build.yml: fix the name of the Jenkins cred

ceph-build.yml: fix the name of the Jenkins cred

(this is why it's a bad idea to switch between . and _)

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2317 from dmick/wip-ceph-build-quay-creds

ceph-build: use correct quay.ceph.io credentials

ceph-build: use correct quay.ceph.io credentials

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2314 from ceph/prs-no-smithi

Do not run make check or API tests on smithi due to OOM

Remove quincy builds/mentions from most builds/build configs.

Intentionally leaving the map from number to name in build_utils.sh for
historical purposes.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Do not run make check or API tests on smithi due to OOM

Saw many jobs fail on smithi004.  dmesg shows lots of OOM kills.

Signed-off-by: David Galloway <david.galloway@ibm.com>

Merge pull request #2311 from ceph/fix-cdnb

ceph-dev-new-build: Fix parameter name

Merge pull request #2312 from dmick/wip-jenkins-builder

ansible/examples/builder.yml: "vars" should be a dict

it always should have been, but this syntax is now an error.  Make it
a nested dict (rather than a list of dicts).

Signed-off-by: Dan Mick <dan.mick@redhat.com>

ceph-dev-new-build: Fix parameter name

Signed-off-by: Zack Cerza <zack@redhat.com>

Merge pull request #2295 from ceph/keep-more

Merge pull request #2309 from ljflores/wip-revert-pr-2306

Revert "ceph-website-prs/config/definitions: remove org filter for site compile check

ceph-website-prs/config/definitions: remove whitelist section

This section allows three people to bypass the org restriction.
However, zdover23 is already part of the org, and the other two
were originally contractors who are no longer involved with
the project.

Signed-off-by: Laura Flores <lflores@ibm.com>

Revert "ceph-website-prs/config/definitions: remove org filter for site compile check"

This reverts commit a227cbd01c628e9d98cf5cb3d96f0a299c4d12c3.
We decided that this is too much of a security risk and are looking
into better options.

Signed-off-by: Laura Flores <lflores@ibm.com>

Merge pull request #2307 from dmick/wip-sepia-fog-images

Change default distroversions, improve task-detection code

sepia-fog-images: look for deploy tasks only of the image being captured

It doesn't matter if there are deploy tasks for other images, only for
the one we're about to capture

Signed-off-by: Dan Mick <dmick@redhat.com>

sepia-fog-images: change defaults for distrovers to capture

Signed-off-by: Dan Mick <dmick@redhat.com>

Merge pull request #2306 from ljflores/wip-ceph-website

ceph-website-prs/config/definitions: remove org filter for site compile check

ceph-website-prs/config/definitions: remove org filter for site compile check

These filters block anyone outside the ceph org from having the site compile
check run on their PRs. We do want people from outside the org to be able to
raise PRs, so removing these filters will allow that check to run.

Signed-off-by: Laura Flores <lflores@ibm.com>

Merge pull request #2305 from dmick/wip-remove-ceph-container-jobs

remove all jobs referring to ceph-container.

Merge pull request #2304 from dmick/ceph-use-container

ceph,ceph-build: build prerelease containers (using container/)

remove all jobs referring to ceph-container.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

ceph,ceph-build: build prerelease containers (using container/)

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2273 from ceph/dvanders_allow_zac_to_render

ceph-website: zac is the docs maintainer

Merge pull request #2299 from dmick/wip-attic

move disabled jobs to 'attic/' subdirectory

Merge pull request #2302 from dmick/wip-fix-ceph-dev-build

ceph-dev-build: fix another error (stray '+' in build/build_rpm)

ceph-dev-build: fix another error (stray '+' in build/build_rpm)

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2301 from dmick/wip-fix-ceph-dev-build

ceph-dev-build/build/setup_rpm: fix syntax error

ceph-dev-build/build/setup_rpm: fix syntax error

Reversed the =~ operator when porting change from ceph-dev-new-build

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2300 from dmick/wip-containerfile-ceph-dev-build

ceph-dev-build: change to use container/Containerfile

ceph-dev-build: change to use container/Containerfile

This is the same change as done by 6361c5c4bac7467893b1f629090712c8c0d736a8
and 435515fe481bd1752b630d23986249e90c0fa4f8 in ceph-dev-new-build, but
for ceph-dev-build.  ceph-container is no longer used.

Also remove CONTAINER_BRANCH parameter from ceph-dev

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2298 from dmick/wip-skip-testing-container-prs

build_utils.sh, various PR build scripts: skip if container/

Merge pull request #2292 from dmick/wip-builder-ansible

Clean up problematic rpm macro, install skopeo package for centos9

move ceph-medic* and diamond* to attic/

long-obsolete jobs

Signed-off-by: Dan Mick <dan.mick@redhat.com>

move disabled jobs to 'attic/' subdirectory

Reduce clutter, avoid mistaken interpretations; it's easy to miss
'disabled: true' when you're searching for relevant jobs

Signed-off-by: Dan Mick <dan.mick@redhat.com>

build_utils.sh, various PR build scripts: skip if container/

Parallel to the docs-only test, skip PR checks (api, make check, etc.)
if the only changes are to the container/ subdirectory

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2297 from dmick/get-tarballs-prerelease

scripts/get-tarballs.sh: pull to prerelease area

scripts/get-tarballs.sh: pull to prerelease area

I've got a bunch of doc work to do to the release process;
this is one of them.  Maybe we should also add this, and
its copy, to the ceph-cm-ansible role for d.c.c

Signed-off-by: Dan Mick <dan.mick@redhat.com>

Merge pull request #2296 from dmick/add-get-tarballs.sh

Keep track of extra script get-tarballs.sh

Keep track of extra script get-tarballs.sh

This is used by sync-pull to fetch the tarballs when
constructing a release, but needs modification for the
prerelease/ directory structure.  It's invoked by ssh
to download.ceph.com to run from /home/signer/bin, because
it needs to copy large files from chacra to download.ceph.com
and doing that on download.ceph.com is much more efficient.

Signed-off-by: Dan Mick <dan.mick@redhat.com>

ceph-dev-new-build: Keep more builds

Builds of this job take up 30KB; we can keep far more.

Signed-off-by: Zack Cerza <zack@redhat.com>

ceph-dev-new-trigger: Keep more builds

Builds of this job take up 20MB.

Signed-off-by: Zack Cerza <zack@redhat.com>