scripts/build_utils.sh: quote $PROFILES

otherwise the following "--use-network" would be taken as a profile,
and "yes" is used as the path to package's dsc file.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1873 from tchaikov/wip-profiles-nocheck

scripts/build_utils: do not include nocheck in $PROFILES

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>

Merge pull request #1874 from tchaikov/wip-ceph-dev-new-trigger-escape-in-grep

ceph-dev-new-trigger:  escape ) by adding \ before it

Reviewed-by: Deepika Upadhyay <dupadhya@redhat.com>

ceph-dev-new-trigger: add escape ) by adding \ before it

this should address the regression introduced by
99ac2b60d3e9c236802daf752d9924076151bf82

Signed-off-by: Kefu Chai <kchai@redhat.com>

scripts/build_utils: do not include nocheck in $PROFILES

we don't use nocheck for disabling the "make check" dependencies,
instead, <pkg.ceph.check> is used to enable "make check" dependencies.
so they are not enabled by default.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1870 from ideepika/crimson-pkg-build

build_utils: specify buildprofiles for crimson build

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1862 from ideepika/backup

build_utils: set extra args for jaeger based on branch name

Reviewed-by: Kefu Chai <kchai@redhat.com>

build_utils: check if PROFILE for pbuilder exists

since not sure if --profile accepts comma or not, added this check

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>

ceph-dev-new-trigger: add jaeger-only trigger

if branch will have `jaeger` in branch name, it shall only build
      focal, centos8 x86_64

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>

build_utils: export jaeger build args if FLAVOR == jaeger

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>

Merge pull request #1871 from ceph/wip-cleanupcephio

Prune old ceph.io builds

update extrapkg needed for crimson, to be added by debian buildprofiles

cleanup: add/ remove some missing go to have comments

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>

Merge pull request #1872 from ceph/wip-arm64-focal

ceph-build: arm64 focal builds too

ceph-build: arm64 focal builds too

https://lists.ceph.io/hyperkitty/list/ceph-users@ceph.io/message/GAIDYRWBOXC5FQD4H4UHUHZ6NLNSEKRC/

Signed-off-by: David Galloway <dgallowa@redhat.com>

ceph-website*: Remove TODO

The README isn't going to be updated all that often.  It's not a big deal to run the job anyway.

Signed-off-by: David Galloway <dgallowa@redhat.com>

ceph-website-prs: Clean up old builds

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1869 from ceph/wip-split-cephio

ceph-website: Split job into two (PRs and main)

ceph-website: Split job into two (PRs and main)

The PRs job will build PRs.

The non-PRs job will build the production website.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1868 from tchaikov/wip-build_utils-arrow-repo

scripts/build_utils: use apache repo hosted by apache.jfrog.io

Reviewed-by: Gal Salomon <gsalomon@redhat.com>

scripts/build_utils: use apache repo hosted by apache.jfrog.io

the http://dl.bintray.com/apache/arrow/ mirror returns 403 "Forbidden"
now.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1867 from ceph/ceph_ansible_prs_purge_dashboard

ceph-ansible-prs: add purge-dashboard jobs

ceph-ansible-prs: add purge-dashboard jobs

This adds the purge-dashboards jobs for both container and non container
scenarios.
Those scenarios aren't triggered by default.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>

Merge pull request #1865 from ceph/wip-wipeout

ceph-website: Build PR checkouts

ceph-website: Build PR checkouts

The previous way this was configured would only build branches from this repo; no forks.  When a PR is submitted from a fork, the job would just build main.  Oops.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1864 from ceph/ceph_ansible_prs_cephadm_adopt_hosts

ceph-ansible-prs: don't run cephadm jobs on smithi

ceph-ansible-prs: don't run cephadm jobs on smithi

smithi nodes aren't enough powerful for that job, let's run it on either
braggi or amadi.

Signed-off-by: Dimitri Savineau <dsavinea@redhat.com>

Merge pull request #1861 from ceph/wip-normaldistro

build_utils.sh: Fix variable

build_utils.sh: Fix variable

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1860 from ceph/auth_cephadm_ansible

cephadm-ansible: use dedicated credential-id

cephadm-ansible: use dedicated credential-id

This makes the cephadm-ansible job use the right credential id

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #1850 from galsalomon66/update-repo-for-arrow-parquet-installation

scripts/build_utils.sh: add package repo for apache arrow

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1859 from ceph/drop_syntax_check

ceph-ansible: drop pr-syntax-check

ceph-ansible: drop pr-syntax-check

This is now done using GitHub workflows.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

script/build_utils: update repo for arrow and parquet installation;

Signed-off-by: gal salomon <gal.salomon@gmail.com>

Merge pull request #1858 from ceph/wip-modules

ceph-pr-submodules: This job can run anywhere

Reviewed-by: Kefu Chai <kchai@redhat.com>

ceph-pr-submodules: This job can run anywhere

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1856 from ceph/wip-orirvingi

ceph-volume-*: Run on irvingi too.

Merge pull request #1855 from ceph/c-v-test-suite-cleanup

ceph-volume-test: cleanup

ceph-volume-*: Run on irvingi too.

I added irvingi01 and irvingi02 as CentOS7 Vagrant builders today.  We very briefly tried to use DHC for builders but that crashed and burned.

Signed-off-by: David Galloway <dgallowa@redhat.com>

ceph-volume-test: cleanup

Those jobs don't exist anymore in tox.ini of ceph-volume functional
tests. Let's remove them.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #1854 from ceph/fix_missing_subcommand

ceph-volume-test: add missing SUBCOMMAND var

ceph-volume-test: add missing SUBCOMMAND var

SUBCOMMAND is missing for `centos7-bluestore-mixed_type` test.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #1853 from ceph/fix_cv_test_suite

ceph-volume-test: add missing SUBCOMMAND definition

ceph-volume-test: add missing SUBCOMMAND definition

This is making some tests fail in the CI like following:

```
+ /tmp/venv.LBZdiW0Nx4/bin/tox --workdir=/tmp/tox.pQhNDR2lrs -vre
  xenial-bluestore-mixed_type -- --provider=libvirt
  using tox.ini:
  /home/jenkins-build/build/workspace/ceph-volume-scenario/src/ceph-volume/ceph_volume/tests/functional/lvm/tox.ini
  (pid 1864)
  ERROR: unknown environment 'xenial-bluestore-mixed_type'
```

it's picking up the wrong tox.ini file because SUBCOMMAND isn't injected
as it should be.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

Merge pull request #1852 from ceph/wip-naginator

ansible: Install naginator on Jenkins master

ceph-ansible-prs: Retry failed builds w/ naginator

Signed-off-by: David Galloway <dgallowa@redhat.com>

ansible: Install naginator on Jenkins master

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1851 from ceph/wip-website-live

ceph-website: Update build script for go-live

ceph-website: Update build script for go-live

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1848 from ceph/wip-website-dev

ceph-website: Use proper build env per branch

Merge pull request #1840 from ceph/wip-tags

ansible: Give registration tasks a tag

Merge pull request #1849 from ceph/wip-cleaner-args

ansible: Set tmpwatch/tmpcleaner args correctly

ceph-website: Use proper build env per branch

Fixes: https://github.com/ceph/ceph.io/issues/161
Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1847 from ceph/wip-cic8

ceph-iscsi-*: Support CentOS8

ansible: Set tmpwatch/tmpcleaner args correctly

https://github.com/ceph/ceph-build/pull/1838#pullrequestreview-686910470

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1838 from ceph/wip-suse-cleaner

ansible: It's called tmpwatch in OpenSUSE

ceph-iscsi-*: Support CentOS8

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1845 from ceph/wip-debugmore

ceph-ansible-prs: Debug vagrant problems

Merge pull request #1846 from ceph/ca_split_jobs

ceph-ansible-prs: don't run all_in_one on smithi

ceph-ansible-prs: don't run all_in_one on smithi

smithi nodes aren't enough powerful for that job, let's run it on either
braggi or amadi

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

ceph-ansible-prs: Debug vagrant problems

Occasionally having VMs come up but not get an IP.  Or vagrant thinks they got an IP but didn't.  This is impossible to debug after the VMs in the job are torn down so let's spit out some debug info before that.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1844 from tchaikov/wip-revert-92b5739f271a263d58bd7fe6ef8c8000f3cb4204

Revert "add repo update for arrow installation (ubunto/centos)."

Revert "add repo update for arrow installation (ubunto/centos)."

curl and sudo are missing.

This reverts commit 92b5739f271a263d58bd7fe6ef8c8000f3cb4204.

Merge pull request #1843 from ceph/wip-c7-vl

ansible: Pin CentOS7 vagrant-libvirt to 0.3.0

ansible: Pin CentOS7 vagrant-libvirt to 0.3.0

So the vagrant-libvirt project itself admits that it does not support CentOS8.

Jobs /usually/ pass on the smithi builders (CentOS7.9, Vagrant 2.2.3, and vagrant-libvirt 0.0.40 or 0.3.0) so let's try running that everywhere.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1827 from galsalomon66/wip-test-repo-update-for-arrow

scripts/build_utils.sh: add package repo for apache arrow (enables parquet objects processing)

Reviewed-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1841 from ceph/wip-pin-vagrant-libvirt

ansible: Pin vagrant-libvirt to 0.3.0

ansible: Pin vagrant-libvirt to 0.3.0

The latest version (and possibly older) 0.5.3 is giving us tons of issues.

- VMs hanging on `vagrant up`
- `vagrant ssh-config` failing

Not to mention we have to manually build libssh as well as krb5.

Let's not mess with all that.

With this PR, we also don't /need/ to run centos8-vagrant.yml.  It can be used to RE-install vagrant but the separate playbook is no longer required.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1836 from ceph/wip-c8-v2210

ansible: Pin centos8 vagrant builders to 2.2.10

Merge pull request #1839 from ceph/split_smithi_braggi

ceph-ansible-prs: use braggi for biggest jobs

ansible: Give registration tasks a tag

This is so I can skip it when I'm setting up a new CentOS 8 Vagrant builder.

https://wiki.sepia.ceph.com/doku.php?id=production:jenkins.ceph.com&#setting_up_new_jenkins_builder

Signed-off-by: David Galloway <dgallowa@redhat.com>

ceph-ansible-prs: use braggi for biggest jobs

smithi and ovh workers aren't enough powerful, let's use braggi and
adami instead for those jobs.

Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>

ansible: It's called tmpwatch in OpenSUSE

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1837 from ceph/wip-remove-saved

scripts/build_utils.sh: Remove virsh save images

scripts/build_utils.sh: Remove virsh save images

Fixes

```
+ sudo virsh undefine container_rgw0_1623696405_fb24199d9194c338f3d6
error: Refusing to undefine while domain managed save image exists
```
and subsequent
```
+ sudo service libvirt-guests restart
Redirecting to /bin/systemctl restart libvirt-guests.service
Job for libvirt-guests.service failed because the control process exited with error code.
See "systemctl status libvirt-guests.service" and "journalctl -xe" for details.
Build step 'Conditional step (single)' marked build as failure

Jun 15 15:40:10 braggi19 libvirt-guests.sh[44513]: Resuming guest container_rgw0_1623696405_fb24199d9194c338f3d6: error: Failed to start domain container_rgw0_1623696405_fb24199d9194c338f3d6
Jun 15 15:40:10 braggi19 libvirt-guests.sh[44513]: error: Network not found: no network with matching name 'vagrant-libvirt'
Jun 15 15:40:11 braggi19 systemd[1]: libvirt-guests.service: Main process exited, code=exited, status=1/FAILURE
Jun 15 15:40:11 braggi19 systemd[1]: libvirt-guests.service: Failed with result 'exit-code'.
Jun 15 15:40:11 braggi19 systemd[1]: Failed to start Suspend/Resume Running libvirt Guests.
```

Signed-off-by: David Galloway <dgallowa@redhat.com>

ansible: Pin centos8 vagrant builders to 2.2.10

Seeing weird random failures with 2.2.16.

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1835 from ceph/wip-vagrantdestroy

ceph-{ansible,container}*: Globally destroy all stale VMs

ceph-{ansible,container}*: Globally destroy all stale VMs

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1834 from ceph/wip-c8-vagrant

ansible: Update centos8-vagrant playbook to work for CentOS8.4

Merge pull request #1833 from ceph/wip-feature_branches-master

pr-api: trigger on feature branches

ansible: Update centos8-vagrant playbook to work for CentOS8.4

Signed-off-by: David Galloway <dgallowa@redhat.com>

pr-api: trigger on feature branches

While other Jenkins jobs don't filter by specific branches, the API
tests do, so we need to support 'feature-*' branch syntax.

Signed-off-by: Ernesto Puerta <epuertat@redhat.com>

add repo update for arrow installation (ubunto/centos).

Signed-off-by: gal salomon <gal.salomon@gmail.com>

Merge pull request #1831 from tchaikov/wip-build-deb-mem

scripts/build_utils: increase mem per job to 3000 MiB

Merge pull request #1832 from tchaikov/wip-no-master-on-bionic

ceph-dev-cron: remove leap15 from comment for job building master

ceph-dev-cron: remove leap15 from comment for job building master

as we don't build master on leap15 anymore

Signed-off-by: Kefu Chai <tchaikov@gmail.com>

ceph-dev-cron: do not build master on bionic

see also 2cbeb00308051188337773b587ea5e2913a42b56

Signed-off-by: Kefu Chai <tchaikov@gmail.com>

scripts/build_utils: increase mem per job to 3000 MiB

see also https://github.com/ceph/ceph/pull/41677

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1830 from tchaikov/wip-no-bionic-on-master

ceph-dev-*-trigger: stop building master on bionic

Reviewed-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: David Galloway <dgallowa@redhat.com>
Reviewed-by: Josh Durgin <jdurgin@redhat.com>

ceph-dev-trigger: remove leap15 from comment for job building master

as we don't build master on leap15 anymore

Signed-off-by: Kefu Chai <kchai@redhat.com>

scripts/build_utils: always pass -DALLOCATOR

this change reverts 901fe2e097548707350bc89c7650e0f72c45d994 and
17b40974b1cb10d1d8ba0373dbf578fb0e82f036.

since we don't need to buld master (quincy and up) on bionic anymore,
there is no need to worry about the FTBFS when compiling with a buggy
gperftools (<2.6.2) shipped along with bionic.

Signed-off-by: Kefu Chai <kchai@redhat.com>

ceph-dev-*-trigger: stop building master on bionic

in short, we don't need to test on bionic anymore for couple reasons:

- for accessing newer GCC
- for accessing more packages

see the discussion at
https://lists.ceph.io/hyperkitty/list/dev@ceph.io/thread/BW5NKQ4KTOPV7OYUBUHVDVCSH72LVIEO/

the only qa suite referencing ubuntu 18.04 is the qa/suites/orch/rook/smoke,
but it deploys ceph using Rook, and the container image of Ceph is
based on CentOS, so stop building on ubuntu/bionic does not
break it, despite the fact that 18.04 is included by
qa/suites/orch/rook/smoke/0-distro.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1829 from tchaikov/wip-viva-la-venv

*/build/teardown: create venv for running teardown_vagrant_tests()

ceph-volume-nightly: do not set $scenarios

teardown_vagrant_tests() takes care of this already.

Signed-off-by: Kefu Chai <kchai@redhat.com>

*/build/teardown: create venv for running teardown_vagrant_tests()

to address the regression introduced by
e302eaabb977f3e93b1a7005c879f4b2eba581fc, which failed to create venv
after removing the global `$(mktemp -td venv.XXXXXXXXXX)` call.

Signed-off-by: Kefu Chai <kchai@redhat.com>

Merge pull request #1826 from dmick/wip-ceph-grafana

Add ceph-grafana job

Add ceph-grafana job to build ceph-grafana container.

This job builds from the ceph workspace based on a ceph branch and an
architecture (x86_64 or arm64).  A parent job calls this job for
branches or architectures.  More versions etc. could be parameterized,
but for now they're hardcoded in the ceph branch in
monitoring/grafana/build/Makefile.

Merge pull request #1825 from ceph/wip-kernel-sha

kernel: Export sha1 as an env variable so shaman build URLs work

kernel: Export sha1 as an env variable so shaman build URLs work

Signed-off-by: David Galloway <dgallowa@redhat.com>

Merge pull request #1824 from ceph/wip-oklist

ceph-website: Allow Pete-Robelou to build beta.ceph.io