SUSE / DeepSea

A collection of Salt files for deploying, managing and automating Ceph.
GNU General Public License v3.0
161 stars 75 forks source link

Adapt logic for global.conf changes #1737

Closed jschmid1 closed 4 years ago

jschmid1 commented 5 years ago

Signed-off-by: Joshua Schmid jschmid@suse.de

~Ignore whitespace and indentation fixes. [auto]~

suse_internal: bsc#1149400


Checklist: ~- [ ] Added unittests and or functional tests~ ~- [ ] Adapted documentation~

jschmid1 commented 5 years ago

note to self: split up commits in

jschmid1 commented 5 years ago

note to self: split up commits in

~> * whitespace changes~

  • actual code changes

For now, let's stick to the actual code changes until we reached consensus on the style-guide

jschmid1 commented 5 years ago

@susebot run teuthology

susebot commented 5 years ago

Commit edbb5ad2381332027aac7b9268bbd8c768fbd773 is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/282/

jschmid1 commented 5 years ago

Commit edbb5ad is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/282/

the CI caught a bug in this patch, fixing

jschmid1 commented 5 years ago

ready for a second round of reviews

jschmid1 commented 5 years ago

@susebot run teuthology

susebot commented 5 years ago

Commit 320c1c954db48af8977cabbc1bc8380e9698718a is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/283/

jschmid1 commented 5 years ago

unrelated to this pr, but it seems we have a failure in stage.4.. re-running to verify

jschmid1 commented 5 years ago

@susebot run teuthology

susebot commented 5 years ago

Commit 320c1c954db48af8977cabbc1bc8380e9698718a is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/285/

jschmid1 commented 5 years ago

@smithfarm It seems that the CI is currently not able to deploy nfs-ganesha. Did we update the build artifacts already?

jschmid1 commented 5 years ago

@susebot run teuthology

smithfarm commented 5 years ago

@jschmid1 I don't see where the CI is unable to deploy nfs-ganesha. What I see is this:

2019-09-16T14:00:38.503 INFO:teuthology.orchestra.run.target192168000079.stderr:scripts/ganesha_smoke_test.sh: line 225: /root/mnt/rgw/bucket1/hello.txt: No such file or directory
susebot commented 5 years ago

Commit 320c1c954db48af8977cabbc1bc8380e9698718a is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/287/

jschmid1 commented 5 years ago

The failure is weird though. All tests before and after were successful.. This branch is clearly not changing anything in the NFS-ganesha area.. re-triggering to be extra-extra-sure

jschmid1 commented 5 years ago

@susebot run teuthology

jschmid1 commented 5 years ago

apparently we have two nfs-ganesha tests:

1) with mds backend 2) with rgw backed

only 1) fails. Maybe the mds state fails while checking for changed files somehow? Why wouldn't the standard(non-ganesha) mds test fail?..

susebot commented 5 years ago

Commit 320c1c954db48af8977cabbc1bc8380e9698718a is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/290/

jschmid1 commented 5 years ago

I'm puzzled..

susebot commented 5 years ago

Commit 320c1c954db48af8977cabbc1bc8380e9698718a is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: https://ceph-ci.suse.de/job/pr-deepsea/5/

susebot commented 4 years ago

Commit 320c1c954db48af8977cabbc1bc8380e9698718a is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: https://ceph-ci.suse.de/job/pr-deepsea/1/

jschmid1 commented 4 years ago

Closing.. Reopen if needed