*** zzzeek has joined #openstack-kolla | 00:01 | |
*** k_mouza has joined #openstack-kolla | 00:24 | |
*** k_mouza has quit IRC | 00:28 | |
*** zzzeek has quit IRC | 00:38 | |
*** zzzeek has joined #openstack-kolla | 00:42 | |
*** k_mouza has joined #openstack-kolla | 00:44 | |
*** k_mouza has quit IRC | 00:49 | |
*** k_mouza has joined #openstack-kolla | 00:54 | |
openstackgerrit | zhoulinhui proposed openstack/kolla-ansible master: Stop install unnecessary packages when run tox -e releasenotes https://review.opendev.org/750692 | 00:58 |
---|---|---|
*** k_mouza has quit IRC | 00:58 | |
openstackgerrit | zhoulinhui proposed openstack/kolla master: Reuse the docs deps for releasenotes to benefit from constraints https://review.opendev.org/750850 | 01:02 |
openstackgerrit | zhoulinhui proposed openstack/kolla master: Reuse the docs deps for releasenotes to benefit from constraints https://review.opendev.org/750850 | 01:03 |
*** k_mouza has joined #openstack-kolla | 01:06 | |
*** LinPeiWen has joined #openstack-kolla | 01:08 | |
*** k_mouza has quit IRC | 01:11 | |
*** k_mouza has joined #openstack-kolla | 01:20 | |
openstackgerrit | zhoulinhui proposed openstack/kolla-ansible master: Remove monasca-log-api stuff since it has been deprecated https://review.opendev.org/750543 | 01:22 |
*** k_mouza has quit IRC | 01:24 | |
*** k_mouza has joined #openstack-kolla | 01:33 | |
*** k_mouza has quit IRC | 01:38 | |
*** vishalmanchanda has joined #openstack-kolla | 01:58 | |
*** zzzeek has quit IRC | 02:14 | |
*** zzzeek has joined #openstack-kolla | 02:15 | |
*** zzzeek has quit IRC | 02:45 | |
*** zzzeek has joined #openstack-kolla | 02:48 | |
*** k_mouza has joined #openstack-kolla | 03:16 | |
openstackgerrit | wu.chunyang proposed openstack/kolla-ansible master: implement automatic deploy of octavia https://review.opendev.org/740180 | 03:17 |
*** k_mouza has quit IRC | 03:20 | |
*** skramaja has joined #openstack-kolla | 03:24 | |
*** k_mouza has joined #openstack-kolla | 03:25 | |
*** k_mouza has quit IRC | 03:30 | |
*** k_mouza has joined #openstack-kolla | 03:35 | |
*** k_mouza has quit IRC | 03:39 | |
*** brinzhang_ has joined #openstack-kolla | 03:51 | |
*** brinzhang0 has quit IRC | 03:54 | |
*** dave-mccowan has quit IRC | 04:12 | |
*** bhagyashris|rove is now known as bhagyashri|rover | 04:32 | |
*** k_mouza has joined #openstack-kolla | 04:33 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #openstack-kolla | 04:33 | |
*** k_mouza has quit IRC | 04:37 | |
*** bersace has quit IRC | 05:00 | |
*** k_mouza has joined #openstack-kolla | 05:07 | |
*** k_mouza has quit IRC | 05:12 | |
openstackgerrit | wu.chunyang proposed openstack/kolla-ansible master: add prepare task for octavia upgrade https://review.opendev.org/746005 | 05:13 |
*** k_mouza has joined #openstack-kolla | 05:14 | |
*** rphillips has quit IRC | 05:15 | |
*** rphillips has joined #openstack-kolla | 05:15 | |
*** k_mouza has quit IRC | 05:19 | |
*** k_mouza has joined #openstack-kolla | 05:43 | |
*** k_mouza has quit IRC | 05:48 | |
*** zzzeek has quit IRC | 05:57 | |
*** k_mouza has joined #openstack-kolla | 05:57 | |
*** zijlboot has joined #openstack-kolla | 05:59 | |
*** zzzeek has joined #openstack-kolla | 06:01 | |
*** k_mouza has quit IRC | 06:01 | |
*** k_mouza has joined #openstack-kolla | 06:08 | |
*** ricolin has quit IRC | 06:10 | |
*** k_mouza has quit IRC | 06:12 | |
*** cah_link has joined #openstack-kolla | 06:13 | |
openstackgerrit | wu.chunyang proposed openstack/kolla-ansible master: implement automatic deploy of octavia https://review.opendev.org/740180 | 06:15 |
openstackgerrit | wu.chunyang proposed openstack/kolla-ansible master: implement automatic deploy of octavia https://review.opendev.org/740180 | 06:16 |
*** brinzhang0 has joined #openstack-kolla | 06:17 | |
*** brinzhang_ has quit IRC | 06:19 | |
*** k_mouza has joined #openstack-kolla | 06:21 | |
*** k_mouza has quit IRC | 06:25 | |
*** suryasingh has joined #openstack-kolla | 06:33 | |
*** k_mouza has joined #openstack-kolla | 06:35 | |
*** k_mouza has quit IRC | 06:39 | |
yoctozepto | morning kolla | 06:47 |
*** jbadiapa has joined #openstack-kolla | 06:47 | |
yoctozepto | kolla hit by http://lists.openstack.org/pipermail/openstack-discuss/2020-September/017155.html - do not recheck on master for now | 06:48 |
*** k_mouza has joined #openstack-kolla | 06:49 | |
*** k_mouza has quit IRC | 06:54 | |
*** k_mouza has joined #openstack-kolla | 07:06 | |
*** k_mouza has quit IRC | 07:10 | |
*** brinzhang_ has joined #openstack-kolla | 07:14 | |
*** bengates has joined #openstack-kolla | 07:15 | |
*** priteau has joined #openstack-kolla | 07:16 | |
*** brinzhang0 has quit IRC | 07:18 | |
mnasiadka | well, I think not only kolla | 07:25 |
yoctozepto | mnasiadka: yeah, i'm patching the world another day | 07:27 |
mnasiadka | another day of firefighting | 07:27 |
mnasiadka | and I guess ,,normal'' projects have their freeze today | 07:28 |
yoctozepto | mnasiadka: yeah, it's a great lol | 07:28 |
yoctozepto | mnasiadka: freeze tomorrow | 07:28 |
mnasiadka | well, today, tomorrow, still people running around and screaming | 07:29 |
mnasiadka | no need for obstacles like this one :) | 07:29 |
yoctozepto | mnasiadka: I have masakari under my control but it's ultimately only me doing stuff in there atm ;/ | 07:29 |
yoctozepto | mnasiadka: true that | 07:29 |
mnasiadka | yoctozepto: another semi-dead project? | 07:29 |
yoctozepto | mnasiadka: I'm thinking about declaring a benevolent dictatorship | 07:29 |
yoctozepto | mnasiadka: indeed :/ | 07:29 |
priteau | yoctozepto: feature freeze is today (deadlines of the release schedule are on Thursday) | 07:31 |
priteau | I am working on a fix for kayobe | 07:31 |
yoctozepto | priteau: ah! thanks for explaining | 07:31 |
yoctozepto | priteau: kayobe looks fine regarding focal though as it mostly doesn't care about focal at all | 07:32 |
priteau | no, it's broken | 07:32 |
yoctozepto | priteau: but it has no lower-constraints to test | 07:32 |
yoctozepto | priteau: what does it fail on now? | 07:32 |
priteau | focal is not compatible with the old hacking version that we use | 07:32 |
yoctozepto | except for ironic thing | 07:32 |
yoctozepto | oh my..... | 07:32 |
priteau | https://zuul.opendev.org/t/openstack/build/df4b79a4f3454949aebbd8db5f39da4b | 07:32 |
yoctozepto | good I patched it early for kolla and k-a | 07:33 |
priteau | gmann proposed patches on various projects for this, but kayobe wasn't included | 07:33 |
yoctozepto | oh well | 07:33 |
mnasiadka | usually QA does not care about deployment projects :) | 07:36 |
*** e0ne has joined #openstack-kolla | 07:43 | |
*** dougsz has joined #openstack-kolla | 07:46 | |
*** priteau has quit IRC | 07:47 | |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750896 | 07:47 |
*** zzzeek has quit IRC | 07:48 | |
*** k_mouza has joined #openstack-kolla | 07:48 | |
*** zzzeek has joined #openstack-kolla | 07:50 | |
openstackgerrit | Pierre Riteau proposed openstack/kayobe master: Update hacking for Python3 https://review.opendev.org/750897 | 07:53 |
*** k_mouza has quit IRC | 07:53 | |
*** priteau has joined #openstack-kolla | 07:53 | |
yoctozepto | priteau: how does one test inspector api? | 07:54 |
priteau | pip install python-ironic-inspector-client | 07:55 |
priteau | openstack baremetal introspection rule list | 07:55 |
priteau | Something like that | 07:55 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750896 | 07:56 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750898 | 07:56 |
*** k_mouza has joined #openstack-kolla | 07:57 | |
*** k_mouza has quit IRC | 08:01 | |
*** shyamb has joined #openstack-kolla | 08:03 | |
*** nikparasyr has joined #openstack-kolla | 08:06 | |
*** brinzhang0 has joined #openstack-kolla | 08:09 | |
yoctozepto | mnasiadka: merge me that l-c fixes | 08:11 |
*** brinzhang_ has quit IRC | 08:12 | |
mnasiadka | yoctozepto: done | 08:12 |
*** shyam89 has joined #openstack-kolla | 08:13 | |
yoctozepto | priteau: ok, then we are not testing it | 08:13 |
yoctozepto | priteau: could it be that nothing hits it otherwise? | 08:13 |
*** shyamb has quit IRC | 08:15 | |
*** k_mouza has joined #openstack-kolla | 08:19 | |
*** k_mouza has quit IRC | 08:23 | |
*** jaicaa has quit IRC | 08:29 | |
*** k_mouza has joined #openstack-kolla | 08:30 | |
*** jaicaa has joined #openstack-kolla | 08:31 | |
priteau | yoctozepto: I don't know what your scenarios do, but it's quite likely that it isn't used at all | 08:32 |
priteau | The failing jobs in Kayobe CI are fixed by https://review.opendev.org/#/c/750897/ | 08:33 |
patchbot | patch 750897 - kayobe - Update hacking for Python3 - 1 patch set | 08:33 |
openstackgerrit | Merged openstack/kolla master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750898 | 08:34 |
*** kevko has joined #openstack-kolla | 08:36 | |
yoctozepto | priteau: k, commented | 08:36 |
openstackgerrit | Merged openstack/kolla-ansible master: Fix l-c testing for Ubuntu Focal (py38) https://review.opendev.org/750896 | 08:37 |
*** k_mouza has quit IRC | 08:47 | |
*** k_mouza has joined #openstack-kolla | 08:48 | |
openstackgerrit | Pierre Riteau proposed openstack/kayobe master: Update hacking for Python3 https://review.opendev.org/750897 | 08:48 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible master: Revert "Add support for encrypting Ironic API" https://review.opendev.org/750804 | 08:49 |
*** gfidente has joined #openstack-kolla | 08:49 | |
priteau | Thanks for the quick CI fix yoctozepto | 08:49 |
yoctozepto | priteau: yw | 08:55 |
*** abdysn has joined #openstack-kolla | 08:59 | |
yoctozepto | priteau: ironic-inspector api logs are empty so yeah, it's not tested at all | 09:03 |
*** shyam89 has quit IRC | 09:07 | |
*** ricolin has joined #openstack-kolla | 09:09 | |
*** Fl1nt has joined #openstack-kolla | 09:10 | |
Fl1nt | Hi everyone! | 09:12 |
yoctozepto | hi Fl1nt | 09:13 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Test Ironic Inspector API https://review.opendev.org/750909 | 09:14 |
*** stingrayza has joined #openstack-kolla | 09:27 | |
*** shyamb has joined #openstack-kolla | 09:29 | |
*** also_stingrayza has quit IRC | 09:30 | |
openstackgerrit | Radosław Piliszek proposed openstack/kolla master: Stop install unnecessary packages when run tox -edocs https://review.opendev.org/750691 | 09:35 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla master: Reuse the docs deps for releasenotes to benefit from constraints https://review.opendev.org/750850 | 09:36 |
*** skramaja has quit IRC | 09:39 | |
*** skramaja has joined #openstack-kolla | 09:39 | |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Support building source images with in-review changes https://review.opendev.org/750314 | 09:39 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: Stop install unnecessary packages when run tox -e releasenotes https://review.opendev.org/750692 | 09:43 |
yoctozepto | priteau: qq, in kayobe are you testing the binary or source images? | 09:47 |
yoctozepto | (in zuul ci I mean) | 09:47 |
priteau | I think it is binary | 09:48 |
*** k_mouza_ has joined #openstack-kolla | 09:49 | |
*** k_mouza has quit IRC | 09:51 | |
*** brinzhang0 has quit IRC | 09:54 | |
*** brinzhang has joined #openstack-kolla | 09:58 | |
*** cah_link has quit IRC | 10:17 | |
yoctozepto | priteau: oddly, my proposal passes | 10:25 |
yoctozepto | priteau: but it logs https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_74e/750909/1/check/kolla-ansible-centos8-binary-ironic/74ebc20/primary/logs/kolla/ironic/ironic-inspector-error.txt | 10:26 |
yoctozepto | 2020-09-10 09:43:28.642057 Target WSGI script not found or unable to stat: /usr/bin/ironic-inspector-wsgi | 10:26 |
yoctozepto | weeird | 10:26 |
Fl1nt | On the same topic (CI passing but not really) when you build a kolla image that use curl to retrieve a required package, typically, the prometheus exporters, it doesn't fail even if the curl of the package failed, so it build an "empty" container. | 10:28 |
yoctozepto | Fl1nt: report, curl needs a flag most likely | 10:28 |
Fl1nt | interesting, a flag the redirect the error on stdout explicitly in order for Kolla build to be able to identify the error as failed ? | 10:29 |
Fl1nt | -the + that | 10:30 |
yoctozepto | Fl1nt: yeah, I believe curl might be having this forgiving behaviour due to some other flag and it needs yet another flag to make it right | 10:31 |
yoctozepto | Fl1nt: I believe it was some -sS thingy, not sure atm | 10:31 |
Fl1nt | ok, I'll test that out ^^ | 10:33 |
Fl1nt | and add this little patch on my todo. | 10:33 |
Fl1nt | oh, btw, my git review refuse to work correctly. I've my account, my SSH Key declared within my profile and my .ssh/config correctly set to use that key with my username, but git review continue to complain that it can't connect. | 10:34 |
Fl1nt | I'm wondering if it came from the merging of my ubuntu one accounts. | 10:34 |
Fl1nt | @yoctozepto, is ssh://<username>@review.opendev.org:29418/openstack/kolla.git the correct URL for git-review ? | 10:37 |
yoctozepto | Fl1nt: yes | 10:40 |
yoctozepto | Fl1nt: you sure you've got the key added in gerrit? | 10:40 |
yoctozepto | (as opposed to launchpad) | 10:40 |
Fl1nt | it should, I'm using the same computer than my previous patch. | 10:41 |
Fl1nt | it is in here: https://review.opendev.org/#/settings/ssh-keys | 10:42 |
*** shyamb has quit IRC | 10:44 | |
yoctozepto | Fl1nt: then no idea | 10:49 |
Fl1nt | ok, it seems like I'm a little bit screwed at the moment because my launchpad account isn't yet completely merged with my former company account. | 10:50 |
openstackgerrit | Merged openstack/kolla stable/ussuri: Add collectd-libpod-stats to image https://review.opendev.org/750796 | 10:51 |
openstackgerrit | Merged openstack/kolla master: Stop install unnecessary packages when run tox -edocs https://review.opendev.org/750691 | 10:51 |
*** Fl1nt has quit IRC | 10:53 | |
*** Fl1nt has joined #openstack-kolla | 10:56 | |
Fl1nt | arff, windows updates... | 10:56 |
openstackgerrit | Merged openstack/kolla-ansible master: Stop install unnecessary packages when run tox -e releasenotes https://review.opendev.org/750692 | 10:57 |
*** k_mouza_ has quit IRC | 10:57 | |
priteau | yoctozepto: the WSGI not found error is exactly what I expected: it ships in a different package called openstack-ironic-inspector-api, which I understand needs to be used separately from the ironic-inspector-conductor, by disabling the standalone option | 10:57 |
priteau | So if we're to use WSGI, we need two separate containers | 10:58 |
*** shyamb has joined #openstack-kolla | 10:59 | |
openstackgerrit | Merged openstack/kolla master: Reuse the docs deps for releasenotes to benefit from constraints https://review.opendev.org/750850 | 11:03 |
priteau | A 404 on openstack baremetal introspection rule list should return an exit code of 1 though | 11:04 |
yoctozepto | priteau: weird, no? | 11:11 |
priteau | That's what it does in my local env | 11:11 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible master: [CI] Test Ironic Inspector API https://review.opendev.org/750909 | 11:12 |
priteau | I am trying something ^^ | 11:12 |
yoctozepto | priteau: good, cool | 11:14 |
yoctozepto | priteau: well, using --debug could change the logic | 11:14 |
priteau | not in my env | 11:14 |
yoctozepto | priteau: k | 11:15 |
yoctozepto | but it runs under errexit and normally it exits (like many times in the past weeks) | 11:15 |
yoctozepto | don't tell me they broke bash | 11:15 |
*** k_mouza has joined #openstack-kolla | 11:24 | |
*** dougsz has quit IRC | 11:36 | |
*** shyamb has quit IRC | 11:43 | |
*** shyamb has joined #openstack-kolla | 11:43 | |
*** zijlboot_ has joined #openstack-kolla | 11:45 | |
*** dave-mccowan has joined #openstack-kolla | 11:46 | |
*** zijlboot has quit IRC | 11:47 | |
*** shyamb has quit IRC | 11:52 | |
priteau | yoctozepto: with my change test-ironic.sh exited with an error | 12:00 |
priteau | https://dfd72ec7cbe3fd16a016-16c430a2363acd21a3fbf84e4ca76ae5.ssl.cf2.rackcdn.com/750909/2/check/kolla-ansible-ubuntu-source-ironic/7f81d1e/primary/logs/ansible/test-ironic | 12:02 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible master: [DNM] Test whether errexit works https://review.opendev.org/750969 | 12:06 |
mnasiadka | yoctozepto: don't move kolla to storyboard :) | 12:24 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible stable/ussuri: [DNM] Test whether errexit works https://review.opendev.org/750983 | 13:03 |
priteau | yoctozepto: ironic tests are passing with https://review.opendev.org/#/c/750969/ :D | 13:04 |
patchbot | patch 750969 - kolla-ansible - [DNM] Test whether errexit works - 1 patch set | 13:04 |
oklhost | any idea how designate could fail because a duplicate zone master? :D | 13:04 |
priteau | maybe bash is broken... | 13:04 |
devfaz | hi, just wondering a bit how kolla-ansible external-ceph ever worked :) | 13:09 |
devfaz | rbd_user in nova.conf is by default "nova" | 13:09 |
devfaz | ceph_nova_keyring is by default "client.cinder" | 13:10 |
devfaz | so nova-compute is trying to do a "ceph df --id nova", but config.json only copies the "client.cinder"-file => ceph df fails. | 13:10 |
devfaz | currently dont get how this ever worked :) | 13:10 |
*** priteau has quit IRC | 13:11 | |
devfaz | globals.yml: default for ceph_nova_user: "nova"; ceph_nova_keyring: "{{ ceph_cinder_keyring }}" - confused :) | 13:12 |
*** priteau has joined #openstack-kolla | 13:16 | |
priteau | devfaz: See the big red warning here: https://docs.openstack.org/kolla-ansible/latest/reference/storage/external-ceph-guide.html#nova | 13:18 |
devfaz | priteau: i have seperate keys for cinder and kolla. | 13:18 |
devfaz | kolla-ansible is using telling nova-compute.. "use client.nova", but copies "client.cinder" into the container?! | 13:19 |
devfaz | think i got it | 13:20 |
devfaz | https://github.com/openstack/kolla-ansible/commit/1f929336e363f1c3d168bad569460401e122de8f | 13:20 |
devfaz | => if "enable_ceph" is true => "client.nova" ; else "client.cinder". | 13:20 |
devfaz | external ceph doesnt set enable_ceph isnt it? | 13:21 |
devfaz | enable_ceph got removed from all.yml since above commit. | 13:23 |
yoctozepto | mnasiadka: geez, I don't want to; I want to move kayobe to launchpad rather ;p | 13:24 |
yoctozepto | priteau: I see, good catch but wth is bash thinking ;p | 13:24 |
yoctozepto | we rely on errexit in many places | 13:24 |
yoctozepto | could be there is some quirk we don't see just yet | 13:25 |
priteau | I don't know if that's new. I cherry-picked to ussuri to check | 13:25 |
priteau | You said you saw those scripts fail recently? | 13:25 |
*** brinzhang_ has joined #openstack-kolla | 13:25 | |
devfaz | https://github.com/openstack/kolla-ansible/commit/4e6fe7a6da4d5471cd0425c781f25f3e9567d3a0#diff-607e6e5925d0031dafa79eb80d198640 | 13:26 |
devfaz | removed enable_ceph-var | 13:26 |
devfaz | in stable/train the var is still there. | 13:26 |
devfaz | https://github.com/openstack/kolla-ansible/commit/4e6fe7a6da4d5471cd0425c781f25f3e9567d3a0#diff-607e6e5925d0031dafa79eb80d198640R982 | 13:28 |
devfaz | so the default was changed to "client.cind"Âer" | 13:28 |
devfaz | dont think this is correct, but maybe someone is able to convince me :) | 13:28 |
*** brinzhang has quit IRC | 13:29 | |
devfaz | I think ceph_nova_user should match ceph_nova_keyring isnt it? | 13:30 |
*** TrevorV has joined #openstack-kolla | 13:30 | |
yoctozepto | priteau: not test-ironic specifically | 13:43 |
yoctozepto | priteau: but other test scripts yeah | 13:43 |
yoctozepto | priteau: I guess there might be a quirk in how test-ironic is structured or what | 13:44 |
yoctozepto | priteau: the "false" is passing happily | 13:44 |
*** Luzi has joined #openstack-kolla | 13:54 | |
openstackgerrit | Rafael Weingärtner proposed openstack/kolla-ansible master: Introduce `schema_version` in the federated attribute mapping API https://review.opendev.org/750996 | 13:55 |
*** zijlboot_ has quit IRC | 13:57 | |
*** zijlboot has joined #openstack-kolla | 13:57 | |
yoctozepto | devfaz: this changed per ceph upstream docs | 15:03 |
*** nikparasyr has left #openstack-kolla | 15:09 | |
priteau | yoctozepto: I found the issue with the ironic test script | 15:09 |
yoctozepto | priteau: http://paste.openstack.org/show/797730/ | 15:12 |
priteau | https://stackoverflow.com/a/19789651 | 15:13 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible master: Fix test-ironic.sh not catching errors https://review.opendev.org/750969 | 15:13 |
yoctozepto | priteau: commented | 15:14 |
priteau | I followed the exac tsame pattern used by other scripts | 15:15 |
yoctozepto | priteau: hmm, well then they don't make sense either :D | 15:15 |
yoctozepto | priteau: but that's ok | 15:16 |
priteau | Let's fix it in a follow up | 15:18 |
priteau | I notice your masakari test script checks the return value of each openstack command, while ironic does not | 15:18 |
priteau | So, we could remove errexit from the script and follow your approach in masakari | 15:20 |
priteau | That works: | 15:20 |
priteau | + test_masakari_logged | 15:20 |
priteau | + result=1 | 15:20 |
priteau | + [[ 1 != 0 ]] | 15:20 |
priteau | + echo 'Testing Masakari failed. See ansible/test-masakari for details' | 15:20 |
priteau | Testing Masakari failed. See ansible/test-masakari for details | 15:20 |
priteau | + return 1 | 15:20 |
priteau | Or, just set errexit and remove all the error handling code | 15:20 |
yoctozepto | priteau: that is saner, yes | 15:21 |
yoctozepto | priteau: nice to learn that one quirk | 15:21 |
yoctozepto | I'm sad it's so contagious though | 15:21 |
yoctozepto | like if you have an 'if' anywhere in your stack and your errexit is doomed | 15:21 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Test Ironic Inspector API https://review.opendev.org/750909 | 15:23 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Test Ironic Inspector API https://review.opendev.org/750909 | 15:23 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible master: [CI] Test Ironic Inspector API https://review.opendev.org/750909 | 15:23 |
priteau | concurrent rebase :/ | 15:24 |
openstackgerrit | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Test Ironic Inspector API https://review.opendev.org/750909 | 15:24 |
yoctozepto | priteau: yeah, fixed | 15:24 |
yoctozepto | no problem | 15:24 |
*** hongbin has joined #openstack-kolla | 15:25 | |
priteau | Thanks | 15:25 |
openstackgerrit | Pierre Riteau proposed openstack/kolla-ansible master: Revert "Add support for encrypting Ironic API" https://review.opendev.org/750804 | 15:26 |
*** skramaja has quit IRC | 15:35 | |
*** zijlboot has quit IRC | 15:36 | |
*** skramaja has joined #openstack-kolla | 15:38 | |
*** k_mouza has quit IRC | 15:41 | |
*** Luzi has quit IRC | 15:44 | |
*** k_mouza has joined #openstack-kolla | 15:54 | |
*** bengates has quit IRC | 15:58 | |
*** skramaja has quit IRC | 15:59 | |
*** brinzhang0 has joined #openstack-kolla | 16:03 | |
*** k_mouza has quit IRC | 16:04 | |
*** brinzhang_ has quit IRC | 16:06 | |
*** diurnalist has joined #openstack-kolla | 16:09 | |
*** k_mouza has joined #openstack-kolla | 16:15 | |
devfaz | yoctozepto: could you point me to the upstream docs you mentioned? ceph-ansible? | 16:24 |
*** diurnalist has quit IRC | 16:24 | |
devfaz | nope - https://docs.ceph.com/docs/master/rbd/rbd-openstack/ :) | 16:27 |
devfaz | but the docs also say "rbd_user = cinder | 16:29 |
devfaz | so above is still correct => "I think ceph_nova_user should match ceph_nova_keyring isnt it?" ;) | 16:29 |
*** diurnalist has joined #openstack-kolla | 16:33 | |
devfaz | well i think as upstream (ceph) also moved away from client.nova, changing the default for ceph_nova_user to "cinder" would allow kolla-ansible-users to use external ceph without any custom settings. But on the other side - changing the default will kill existing setups. | 16:38 |
*** abdysn has quit IRC | 16:38 | |
*** e0ne has quit IRC | 16:39 | |
*** k_mouza has quit IRC | 16:49 | |
*** diurnalist has quit IRC | 16:51 | |
*** dking has joined #openstack-kolla | 16:58 | |
*** k_mouza has joined #openstack-kolla | 17:04 | |
*** gfidente is now known as gfidente|afk | 17:07 | |
*** tonyb has quit IRC | 17:15 | |
*** jbadiapa has quit IRC | 17:18 | |
mloza | why is kolla using /32 subnet mask for the VIP address? | 17:25 |
mloza | is there way to override the VIP subnet mask? | 17:26 |
kplant | that's a pretty standard thing to do | 17:33 |
kplant | say you have a /29 or something, your nodes can have static addresses within that /29 assigned as .N/29 or whatever and then a .N/32 within that /29 that can move between them | 17:33 |
*** suryasingh has quit IRC | 17:35 | |
*** k_mouza has quit IRC | 17:37 | |
*** suff has joined #openstack-kolla | 17:41 | |
*** tonyb has joined #openstack-kolla | 17:57 | |
*** gmann is now known as gmann_afk | 18:11 | |
openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/kolla-ansible master: Add support to OpenID Connect Authentication flow https://review.opendev.org/695432 | 18:45 |
*** e0ne has joined #openstack-kolla | 18:57 | |
*** priteau has quit IRC | 19:16 | |
*** e0ne has quit IRC | 19:33 | |
*** Trevor_V has joined #openstack-kolla | 19:53 | |
*** TrevorV has quit IRC | 19:57 | |
*** openstackgerrit has quit IRC | 20:36 | |
*** openstackgerrit has joined #openstack-kolla | 20:56 | |
openstackgerrit | Rafael Weingärtner proposed openstack/kolla-ansible master: Introduce `schema_version` in the federated attribute mapping API https://review.opendev.org/750996 | 20:56 |
*** openstackgerrit has quit IRC | 21:01 | |
*** Trevor_V has quit IRC | 21:09 | |
*** gfidente|afk has quit IRC | 21:21 | |
*** openstackgerrit has joined #openstack-kolla | 21:30 | |
openstackgerrit | James Kirsch proposed openstack/kolla-ansible master: Add support for encrypting Neutron Server API https://review.opendev.org/749616 | 21:30 |
*** Fl1nt has quit IRC | 22:07 | |
*** vishalmanchanda has quit IRC | 22:21 | |
*** zzzeek has quit IRC | 22:26 | |
*** zzzeek has joined #openstack-kolla | 22:27 | |
*** gmann_afk is now known as gmann | 22:49 | |
*** suff has quit IRC | 23:20 | |
*** zzzeek has quit IRC | 23:24 | |
*** zzzeek has joined #openstack-kolla | 23:29 | |
*** zzzeek has quit IRC | 23:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!