]>
git.apps.os.sepia.ceph.com Git - ceph-build.git/log
Guillaume Abrioux [Tue, 4 Dec 2018 09:26:17 +0000 (10:26 +0100)]
ceph-ansible-pipeline: update pipeline
- Reintroduce the purge_cluster_non_container and purge_cluster_container
scenario for stable-3.2 and master (somehow they got removed...),
- Remove purge_lvm_osds_* scenarios.
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Andrew Schoen [Mon, 3 Dec 2018 20:28:06 +0000 (14:28 -0600)]
Merge pull request #1214 from ceph/vagrant-prune
vagrant: prune stale machines before a run
Alfredo Deza [Mon, 3 Dec 2018 13:28:13 +0000 (08:28 -0500)]
ceph-volume jobs: prune stable vagrant vms
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Alfredo Deza [Mon, 3 Dec 2018 13:24:52 +0000 (08:24 -0500)]
scripts: pruning from global status is needed once per teardown
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Alfredo Deza [Mon, 3 Dec 2018 13:19:19 +0000 (08:19 -0500)]
scripts: create a prune_stale_vagrant_vms function to clear before jobs start
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Alfredo Deza [Fri, 30 Nov 2018 20:07:30 +0000 (15:07 -0500)]
Merge pull request #1213 from ceph/wip-adminme
teuthology-PRs: Add trigger phrase
David Galloway [Fri, 30 Nov 2018 18:45:13 +0000 (13:45 -0500)]
teuthology-PRs: Add trigger phrase
Not sure if there was a default that used to work in the past but this
field was blank in the Jenkins web UI and I had to set it before commenting
'jenkins test' triggered a build.
Signed-off-by: David Galloway <dgallowa@redhat.com>
Alfredo Deza [Fri, 30 Nov 2018 17:33:56 +0000 (12:33 -0500)]
Merge pull request #1212 from ceph/wip-moar-cleanup
build_utils: Manually clean up vagrant old domain IDs
David Galloway [Fri, 30 Nov 2018 15:47:21 +0000 (10:47 -0500)]
build_utils: Manually clean up vagrant old domain IDs
Signed-off-by: David Galloway <dgallowa@redhat.com>
Sébastien Han [Fri, 30 Nov 2018 13:53:32 +0000 (14:53 +0100)]
Merge pull request #1211 from ceph/guits_pipeline_ca
ceph-ansible: add purge_lvm_osds_container to PRs
Guillaume Abrioux [Fri, 30 Nov 2018 13:50:42 +0000 (14:50 +0100)]
Merge branch 'master' into guits_pipeline_ca
Guillaume Abrioux [Fri, 30 Nov 2018 13:49:00 +0000 (14:49 +0100)]
ceph-ansible: add purge_lvm_osds_container to PRs
Add purge_lvm_osds_container definition to the existing list of PRs that
are automatically triggered in the pipeline.
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Sébastien Han [Fri, 30 Nov 2018 10:31:08 +0000 (11:31 +0100)]
Merge pull request #1210 from ceph/guits_pipeline_ca
ceph-ansible-pipeline: add purge_lvm_osds_container scenario
Guillaume Abrioux [Fri, 30 Nov 2018 10:22:39 +0000 (11:22 +0100)]
ceph-ansible-pipeline: add purge_lvm_osds_container scenario
Add scenario purge_lvm_osds_container for master branch.
Only run scenario for corresponding purge playbook (container vs. non
container).
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Guillaume Abrioux [Tue, 27 Nov 2018 16:48:35 +0000 (17:48 +0100)]
Merge pull request #1207 from ceph/podman-pipeline
ceph-ansible: add container_podman to the pipeline
Sébastien Han [Tue, 27 Nov 2018 16:46:23 +0000 (17:46 +0100)]
Merge branch 'master' into podman-pipeline
Zack Cerza [Tue, 20 Nov 2018 21:32:01 +0000 (14:32 -0700)]
Merge pull request #1208 from ceph/wip-sepia-optional
sepia-fog-images: Allow use of different teuth/cm branches
David Galloway [Tue, 20 Nov 2018 19:45:23 +0000 (14:45 -0500)]
sepia-fog-images: Allow use of different teuth/cm branches
Signed-off-by: David Galloway <dgallowa@redhat.com>
Sébastien Han [Fri, 16 Nov 2018 12:32:26 +0000 (13:32 +0100)]
ceph-ansible: add container_podman to the pipeline
container_podman must run on every PR as it's the same scenario as
docker but using podman.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Fri, 16 Nov 2018 08:24:01 +0000 (09:24 +0100)]
Merge pull request #1206 from ceph/guits-ca_old_scenarios
ceph-ansible-pipeline: fix luminous-bluestore_lvm_osds scenario
Guillaume Abrioux [Fri, 16 Nov 2018 08:18:19 +0000 (09:18 +0100)]
Merge branch 'master' into guits-ca_old_scenarios
Guillaume Abrioux [Fri, 16 Nov 2018 08:12:22 +0000 (09:12 +0100)]
ceph-ansible-pipeline: fix luminous-bluestore_lvm_osds scenario
this scenario should run on stable-3.2 only
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Guillaume Abrioux [Thu, 15 Nov 2018 22:12:45 +0000 (23:12 +0100)]
Merge pull request #1205 from ceph/guits-ca_old_scenarios
ceph-ansible-pipeline: remove old scenario from stable-3.2 tests
Guillaume Abrioux [Thu, 15 Nov 2018 21:22:35 +0000 (22:22 +0100)]
ceph-ansible-pipeline: remove old scenario from stable-3.2 tests
filestore|bluestore old osds scenarios shouldn't be run when branch is
stable-3.2
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Kefu Chai [Thu, 15 Nov 2018 09:20:27 +0000 (17:20 +0800)]
Merge pull request #1198 from callithea/wip-ceph-dashboard-pull-requests-tmp-remove-ubuntu
ceph-dashboard-pull-requests: Temporary remove Ubuntu node(s)
Reviewed-by: Kefu Chai <kchai@redhat.com>
Laura Paduano [Thu, 8 Nov 2018 10:51:01 +0000 (11:51 +0100)]
ceph-dashboard-pull-requests: Temporary remove Ubuntu node(s)
In order to be able to get to the step where the dashboard end-to-end tests
are being executed (which seem to pass on centos) the proposal is to temporary remove
the Ubuntu nodes.
When the current issues with Ubuntu have been fixed the plan is to
re-add Ubuntu (xenial) again to the configuration of this Jenkins job.
Since the job is being triggered automatically it would also prevent failing
jobs (which currently happens all the time and is eating up resources).
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Sébastien Han [Mon, 12 Nov 2018 13:10:42 +0000 (14:10 +0100)]
Merge pull request #1204 from ceph/guits_ca_pipeline
ceph-ansible: fix pipeline on add-osds/rgw-multisite scenarios
Guillaume Abrioux [Mon, 12 Nov 2018 12:36:06 +0000 (13:36 +0100)]
ceph-ansible: fix pipeline on add-osds/rgw-multisite scenarios
add-osds and rgw-multisite scenarios were not working as expected
in the CI regarding the destination branch.
This commits fixes the scenarios that should be run accoding to the
destination branch.
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Alfredo Deza [Mon, 12 Nov 2018 12:31:46 +0000 (07:31 -0500)]
Merge pull request #1203 from ceph/ceph-volume-dev-branches
ceph-volume-nightly check against CEPH_BRANCH, not CEPH_DEV_BRANCH
Alfredo Deza [Mon, 12 Nov 2018 12:18:10 +0000 (07:18 -0500)]
ceph-volume-nightly check against CEPH_BRANCH, not CEPH_DEV_BRANCH
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Alfredo Deza [Fri, 9 Nov 2018 18:51:46 +0000 (13:51 -0500)]
Merge pull request #1202 from ceph/ceph-ansible-branches
ceph-volume: pick stable-3.2 ceph-ansible branches
Alfredo Deza [Fri, 9 Nov 2018 18:34:22 +0000 (13:34 -0500)]
ceph-volume-nightly use stable-3.2 if luminous or mimic is targetted
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Alfredo Deza [Fri, 9 Nov 2018 18:34:00 +0000 (13:34 -0500)]
ceph-volume-ansible-prs use stable-3.2 if luminous or mimic
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Guillaume Abrioux [Thu, 8 Nov 2018 16:31:28 +0000 (17:31 +0100)]
Merge pull request #1201 from ceph/mv-ooo
Mv ooo
Sébastien Han [Thu, 8 Nov 2018 16:18:24 +0000 (17:18 +0100)]
ceph-ansible-pipeline: do not test ooo on master
Since all the scenario use lvm now, the ooo scenario will always fail
when triggered from the pipeline. The job will remain in stable branches
though.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Thu, 8 Nov 2018 16:21:27 +0000 (17:21 +0100)]
Revert "ceph-ansible-pipeline: do not test ooo"
This reverts commit
56fe3506a192c9e27a5e18801fb904febf4e58db .
We merge the wrong content, this commit was meant to run ooo on stable
branches only and not on master, we did the opposite so reverting this.
Signed-off-by: Sébastien Han <seb@redhat.com>
Guillaume Abrioux [Thu, 8 Nov 2018 16:15:01 +0000 (17:15 +0100)]
Merge pull request #1180 from ceph/rm-ooo
ceph-ansible-pipeline: do not test ooo
Sébastien Han [Thu, 8 Nov 2018 16:09:14 +0000 (17:09 +0100)]
Merge branch 'master' into rm-ooo
Erwan Velu [Thu, 8 Nov 2018 15:49:35 +0000 (16:49 +0100)]
Merge pull request #1200 from ceph/syntax-check
ceph-ansible-pr-syntax-check: always fetch origin
Sébastien Han [Thu, 8 Nov 2018 15:46:29 +0000 (16:46 +0100)]
ceph-ansible-pr-syntax-check: always fetch origin
We need to fetch origin to always ensure we have the right version.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Thu, 8 Nov 2018 15:29:48 +0000 (16:29 +0100)]
Merge pull request #1199 from ceph/guits_stable3.2
ceph-ansible-pipeline: update pipeline matrix
Guillaume Abrioux [Thu, 8 Nov 2018 14:08:25 +0000 (15:08 +0100)]
ceph-ansible-pipeline: update pipeline matrix
We must test corresponding ceph release for each stable branch of
ceph-ansible.
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Guillaume Abrioux [Thu, 8 Nov 2018 09:26:26 +0000 (10:26 +0100)]
Merge pull request #1197 from ceph/container_podman
add new container_podman scenario to ceph-ansible
Sébastien Han [Thu, 8 Nov 2018 09:23:24 +0000 (10:23 +0100)]
add new container_podman scenario to ceph-ansible
This scenario allows us testing podman instead of docker and also python
3 for ou rmodules.
Signed-off-by: Sébastien Han <seb@redhat.com>
tserlin [Wed, 7 Nov 2018 18:32:27 +0000 (13:32 -0500)]
Merge pull request #1196 from ceph/wip-nfs-ganesha-stable-build-fix
nfs-ganesha: fix debian stable build
tserlin [Wed, 7 Nov 2018 16:15:07 +0000 (11:15 -0500)]
Merge branch 'master' into wip-nfs-ganesha-stable-build-fix
Kefu Chai [Wed, 7 Nov 2018 07:53:56 +0000 (23:53 -0800)]
Merge pull request #1195 from callithea/wip-ceph-dashboard-pull-requests-switch-node
ceph-dashboard-pull-requests: switch node type from trusty to xenial
Reviewed-by: Kefu Chai <kchai@redhat.com>
Ali Maredia [Tue, 6 Nov 2018 21:07:46 +0000 (16:07 -0500)]
nfs-ganesha: fix debian stable build
various fixes to nfs-ganesha-stable deb building
update stable config defaults
add removal of old ceph packages from pbuilder
to normal nfs-ganesha debian job
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Laura Paduano [Tue, 6 Nov 2018 13:06:15 +0000 (14:06 +0100)]
ceph-dashboard-pull-requests: switch node type from trusty to xenial
Since there are currently issues with the dashboard job and its dependencies (python-jwt)
when the test node is based on trusty (and Nautilus has dropped the support for trusty anyway)
it makes sense to switch to a more recent Ubuntu version
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Guillaume Abrioux [Mon, 5 Nov 2018 14:00:15 +0000 (15:00 +0100)]
Merge pull request #1193 from ceph/diff-git
ceph-ansible syntax check
Sébastien Han [Mon, 5 Nov 2018 13:55:42 +0000 (14:55 +0100)]
ceph-ansible syntax check
Only check for Modified and Changed content, anything else is irrelevant
especially remove, rename and will make the check fail.
Signed-off-by: Sébastien Han <seb@redhat.com>
Laura Paduano [Mon, 5 Nov 2018 09:13:46 +0000 (10:13 +0100)]
Merge pull request #1190 from callithea/wip-ceph-dashboard-pull-requests-jwt-dependency
ceph-dashboard-pull-requests: add new python-jwt dependency
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Laura Paduano [Fri, 2 Nov 2018 12:41:11 +0000 (13:41 +0100)]
ceph-dashboard-pull-requests: add new python-jwt dependency
Since the JWT PR (#22833) has been merged a new dependency
has been introduced which is now missing for the dashboard Jenkins job
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Jason Dillaman [Sun, 4 Nov 2018 13:12:04 +0000 (08:12 -0500)]
Merge pull request #1192 from ceph/wip-ceph-iscsi
ceph-iscsi: new PR tox and branch trigger builders
Jason Dillaman [Sun, 4 Nov 2018 13:06:55 +0000 (08:06 -0500)]
ceph-iscsi: new PR tox and branch trigger builders
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Jason Dillaman [Fri, 2 Nov 2018 17:52:06 +0000 (13:52 -0400)]
Merge pull request #1191 from ceph/wip-ceph-iscsi
ceph-iscsi: initial version of combined iSCSI tools repo
Jason Dillaman [Fri, 2 Nov 2018 17:02:46 +0000 (13:02 -0400)]
ceph-iscsi: initial version of combined iSCSI tools repo
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Jos Collin [Fri, 2 Nov 2018 09:44:13 +0000 (15:14 +0530)]
Merge pull request #1189 from callithea/wip-ceph-dashboard-pull-requests-label
Changed config of 'white-list-label'
Reviewed-by: Jos Collin <jcollin@redhat.com>
Laura Paduano [Fri, 2 Nov 2018 09:00:41 +0000 (10:00 +0100)]
Changed config of 'white-list-label'
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Jos Collin [Fri, 2 Nov 2018 08:11:44 +0000 (13:41 +0530)]
Merge pull request #1188 from callithea/wip-ceph-dashboard-pull-requests-only-trigger-phrase
ceph-dashboard-pull-requests: re-add 'only-trigger-phrase'
Reviewed-by: Jos Collin <jcollin@redhat.com>
Laura Paduano [Thu, 1 Nov 2018 10:34:58 +0000 (11:34 +0100)]
ceph-dashboard-pull-requests: re-add 'only-trigger-phrase'
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Andrew Schoen [Wed, 31 Oct 2018 20:47:07 +0000 (15:47 -0500)]
Merge pull request #1187 from ceph/ceph-ansible-deps
ceph-volume ansible jobs need python-netaddr installed in the controlling node
Alfredo Deza [Wed, 31 Oct 2018 19:47:39 +0000 (15:47 -0400)]
ceph-volume ansible jobs need python-netaddr installed in the controlling node
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Jos Collin [Wed, 31 Oct 2018 11:02:02 +0000 (16:32 +0530)]
Merge pull request #1168 from callithea/wip-ceph-dashboard-pull-requests-trigger
ceph-dashboard-pull-requests: added 'white-list-labels'
Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Jos Collin <jcollin@redhat.com>
Laura Paduano [Thu, 18 Oct 2018 11:37:00 +0000 (13:37 +0200)]
ceph-dashboard-pull-requests: added 'white-list-labels'
and changed 'only-trigger-phrase' to be able to automatically
start this Job for all dashboard PRs
Signed-off-by: Laura Paduano <lpaduano@suse.com>
Guillaume Abrioux [Wed, 31 Oct 2018 10:41:35 +0000 (11:41 +0100)]
Merge pull request #1183 from ceph/ceph-ansible-master
ceph-ansible-pipeline: only test master
Sébastien Han [Wed, 31 Oct 2018 10:34:24 +0000 (11:34 +0100)]
Merge branch 'master' into ceph-ansible-master
Sébastien Han [Wed, 31 Oct 2018 10:32:46 +0000 (11:32 +0100)]
Merge branch 'master' into rm-ooo
Sébastien Han [Wed, 31 Oct 2018 10:32:34 +0000 (11:32 +0100)]
Merge pull request #1186 from ceph/guits_ceph-ansible3.2
ceph-ansible: update nightlies job
Sébastien Han [Wed, 31 Oct 2018 10:12:58 +0000 (11:12 +0100)]
Merge branch 'master' into ceph-ansible-master
Guillaume Abrioux [Wed, 31 Oct 2018 10:10:50 +0000 (11:10 +0100)]
ceph-ansible: update nightlies job
since recent updates and 3.2 is out, we should update the matrix of
testing for nightlies jobs.
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Ali Maredia [Tue, 30 Oct 2018 18:53:09 +0000 (14:53 -0400)]
Merge pull request #1178 from ceph/wip-tserlin-nfs-ganesha-stable
nfs-ganesha-stable: update build parameters
tserlin [Tue, 30 Oct 2018 17:51:02 +0000 (13:51 -0400)]
Merge branch 'master' into wip-tserlin-nfs-ganesha-stable
tserlin [Tue, 30 Oct 2018 17:50:44 +0000 (13:50 -0400)]
Merge pull request #1184 from ceph/wip-nfs-ganesha-build-fix
nfs-ganesha: various nfs-ganesha build fixes
tserlin [Tue, 30 Oct 2018 17:46:25 +0000 (13:46 -0400)]
Merge branch 'master' into wip-nfs-ganesha-build-fix
Andrew Schoen [Tue, 30 Oct 2018 16:03:01 +0000 (11:03 -0500)]
Merge pull request #1185 from ceph/ceph-medic-disable
ceph-medic-tests: disable all child jobs from template
Alfredo Deza [Tue, 30 Oct 2018 15:37:40 +0000 (11:37 -0400)]
ceph-medic-tests: disable all child jobs from template
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Ali Maredia [Tue, 30 Oct 2018 13:16:28 +0000 (09:16 -0400)]
nfs-ganesha: various nfs-ganesha build fixes
- fix nfs-ganesha build for debian
- add missing packages for stable rpm builds
- cleanup for rpm nfs-ganesha build scripts
Signed-off-by: Ali Maredia <amaredia@redhat.com>
Sébastien Han [Tue, 30 Oct 2018 09:22:57 +0000 (10:22 +0100)]
Merge branch 'master' into ceph-ansible-master
Guillaume Abrioux [Mon, 29 Oct 2018 17:56:17 +0000 (18:56 +0100)]
Merge pull request #1182 from ceph/rgw-multi-site
Rgw multi site
Sébastien Han [Mon, 29 Oct 2018 17:23:42 +0000 (18:23 +0100)]
ceph-ansible-pipeline: only test master
We are now ready to test on master ceph so let's switch all the pipeline
jobs to do so.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Mon, 29 Oct 2018 16:48:26 +0000 (17:48 +0100)]
ceph-ansible-pr: add rgw in devs
Add rgw_multisite_* in devs so we can test this with master upstream.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Mon, 29 Oct 2018 16:47:57 +0000 (17:47 +0100)]
ceph-ansible-pipeline: add rgw multisite
support for testing rgw multisite in the pipeline
Signed-off-by: Sébastien Han <seb@redhat.com>
Guillaume Abrioux [Mon, 29 Oct 2018 15:48:49 +0000 (16:48 +0100)]
Merge pull request #1181 from ceph/ceph-volume-trigger
Ceph volume trigger
Sébastien Han [Mon, 29 Oct 2018 15:42:02 +0000 (16:42 +0100)]
ceph-ansible-pipeline: activate batch container
Now batch container is supported so let's activate it.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Mon, 29 Oct 2018 15:41:15 +0000 (16:41 +0100)]
ceph-ansible-pipeline: run osd scenarios when module is modified
When we edit the ceph_volume.py module we should run all the OSD
scenarios.
Signed-off-by: Sébastien Han <seb@redhat.com>
Sébastien Han [Mon, 29 Oct 2018 09:53:28 +0000 (10:53 +0100)]
ceph-ansible-pipeline: do not test ooo
Since all the scenario use lvm now, the ooo scenario will always fail
when triggered from the pipeline. The job will remain in stable branches
though.
Signed-off-by: Sébastien Han <seb@redhat.com>
Guillaume Abrioux [Fri, 26 Oct 2018 17:08:40 +0000 (19:08 +0200)]
Merge pull request #1179 from ceph/guits_rgw_multisite
ceph-ansible: add rgw_multisite scenario
Sébastien Han [Fri, 26 Oct 2018 15:23:03 +0000 (17:23 +0200)]
Merge branch 'master' into guits_rgw_multisite
Sébastien Han [Fri, 26 Oct 2018 15:22:30 +0000 (17:22 +0200)]
Merge pull request #1177 from ceph/pipeline
ceph-ansible-pipeline: run all lvm test against ceph@master
Guillaume Abrioux [Fri, 26 Oct 2018 14:46:58 +0000 (16:46 +0200)]
ceph-ansible: add rgw_multisite scenario
ceph-ansible corresponding PR:
ceph/ceph-ansible#1944
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Thomas Serlin [Thu, 25 Oct 2018 19:44:28 +0000 (15:44 -0400)]
nfs-ganesha-stable: update build parameters
Andrew Schoen [Thu, 25 Oct 2018 16:23:33 +0000 (11:23 -0500)]
ceph-ansible-pipeline: run all lvm tests against ceph@master
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Andrew Schoen [Thu, 25 Oct 2018 13:11:30 +0000 (08:11 -0500)]
ceph-ansible-prs: allow all jobs to be tested with dev repos
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Erwan Velu [Wed, 24 Oct 2018 14:48:24 +0000 (16:48 +0200)]
Merge pull request #1176 from ceph/guits-remove_color
ceph-ansible-syntax: disable syntax coloration in `git_diff_to_head()`
Guillaume Abrioux [Wed, 24 Oct 2018 14:30:44 +0000 (16:30 +0200)]
ceph-ansible-syntax: disable syntax coloration in `git_diff_to_head()`
when running this code locally for debugging purposes, the sed in
`match_file()` won't match the pattern because of the ANSI codes inserted by
the git diff (colored) output.
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Sébastien Han [Wed, 24 Oct 2018 14:01:42 +0000 (16:01 +0200)]
Merge pull request #1175 from ceph/guits-revert_4a25fb2b
ceph-ansible-syntax: display matched files in group_vars_check
Guillaume Abrioux [Wed, 24 Oct 2018 12:59:14 +0000 (14:59 +0200)]
ceph-ansible-syntax: display matched files in group_vars_check
it can be useful to see which files have matched in `group_vars_check()`
function.
I'm currently facing an issue where the CI complains on that check while
it doesn't on an other environment when trying to reproduce the issue.
on the CI:
```
+ group_vars_check
++ match_file /defaults/main.yml
++ git_diff_to_head
++ wc -l
++ git diff origin/master..HEAD -- . ':(exclude)roles/*/meta/*'
++ sed -n 's|^+++.*\(/defaults/main.yml.*\)|\1|p'
+ nb=1
+ [[ 1 -eq 0 ]]
++ match_file group_vars/
++ wc -l
++ git_diff_to_head
++ git diff origin/master..HEAD -- . ':(exclude)roles/*/meta/*'
++ sed -n 's|^+++.*\(group_vars/.*\)|\1|p'
+ nb_group_vars=0
+ [[ 1 -gt 0 ]]
+ echo 'One or more files containing default variables has/have been modified.'
One or more files containing default variables has/have been modified.
+ echo 'You must run '\''generate_group_vars_sample.sh'\'' to generate the group_vars template files.'
You must run 'generate_group_vars_sample.sh' to generate the group_vars template files.
+ return 1
```
local env:
```
+ group_vars_check
++ match_file /defaults/main.yml
++ wc -l
++ git_diff_to_head
++ sed -n 's|^+++.*\(/defaults/main.yml.*\)|\1|p'
++ git diff origin/master..HEAD -- . ':(exclude)roles/*/meta/*'
+ nb=0
+ [[ 0 -eq 0 ]]
+ echo 'group_vars has not been touched.'
group_vars has not been touched.
+ return 0
```
Signed-off-by: Guillaume Abrioux <gabrioux@redhat.com>
Sébastien Han [Wed, 24 Oct 2018 13:25:48 +0000 (15:25 +0200)]
Merge pull request #1144 from ceph/guits-remove_capital_check
ceph-ansible-syntax: remove capital letter check
Sébastien Han [Wed, 24 Oct 2018 13:21:39 +0000 (15:21 +0200)]
Merge branch 'master' into guits-remove_capital_check