opendevreview | Boxiang Zhu proposed openstack/kolla-ansible master: Add skyline role https://review.opendev.org/c/openstack/kolla-ansible/+/810566 | 02:14 |
---|---|---|
opendevreview | Boxiang Zhu proposed openstack/kolla master: Add skyline Dockerfile https://review.opendev.org/c/openstack/kolla/+/810796 | 03:03 |
opendevreview | Boxiang Zhu proposed openstack/kolla-ansible master: Add skyline role https://review.opendev.org/c/openstack/kolla-ansible/+/810566 | 03:04 |
opendevreview | Boxiang Zhu proposed openstack/kolla master: Add skyline Dockerfile https://review.opendev.org/c/openstack/kolla/+/810796 | 03:59 |
*** amoralej|off is now known as amoralej | 06:03 | |
mnasiadka | yuriys: we are publishing images to both, just the default configuration in Xena has changed to use quay instead of docker hub. | 07:39 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/wallaby: Make setup module arguments configurable https://review.opendev.org/c/openstack/kayobe/+/811782 | 07:41 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/victoria: Make setup module arguments configurable https://review.opendev.org/c/openstack/kayobe/+/811783 | 07:41 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/victoria: Make setup module arguments configurable https://review.opendev.org/c/openstack/kayobe/+/811783 | 07:45 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/ussuri: Make setup module arguments configurable https://review.opendev.org/c/openstack/kayobe/+/811784 | 07:46 |
opendevreview | Verification of a change to openstack/kayobe stable/wallaby failed: CI: Disable libvirt debug logs https://review.opendev.org/c/openstack/kayobe/+/811500 | 07:57 |
mgoddard | morning | 08:04 |
kevko | morning \o/ | 08:06 |
opendevreview | Mark Goddard proposed openstack/kolla master: Build source type images by default https://review.opendev.org/c/openstack/kolla/+/811743 | 08:07 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible master: Deploy source type images by default https://review.opendev.org/c/openstack/kolla-ansible/+/811744 | 08:07 |
mnasiadka | morning | 08:08 |
kevko | mnasiadka: i saw you've added review-priority: -1 to skyline patches - I agree, it's in very bad shape ...but I am curious if you tried to deploy skyline | 08:09 |
mnasiadka | kevko: it's feature freeze | 08:09 |
kevko | yeah, I know ... I was just asking if you tried skyline .. (not depends if via kolla or manually ) | 08:10 |
mnasiadka | No, I don't use Horizon, so who cares about replacement :) | 08:10 |
kevko | :D | 08:11 |
kevko | it's actually cool at first glance ..quite fast | 08:12 |
opendevreview | Mark Goddard proposed openstack/kayobe master: Build and deploy source type images by default https://review.opendev.org/c/openstack/kayobe/+/811888 | 08:13 |
mnasiadka | mgoddard: I think we should merge this as well - https://review.opendev.org/c/openstack/kolla/+/811014 | 08:28 |
opendevreview | Michal Nasiadka proposed openstack/kolla-ansible master: docs: ceph: add copy-on-write for Glance https://review.opendev.org/c/openstack/kolla-ansible/+/809992 | 08:32 |
opendevreview | Boxiang Zhu proposed openstack/kolla master: Add skyline Dockerfile https://review.opendev.org/c/openstack/kolla/+/810796 | 08:35 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/ussuri: Make setup module arguments configurable https://review.opendev.org/c/openstack/kayobe/+/811784 | 08:50 |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 09:04 |
mnasiadka | dial tcp: lookup quay.io on 127.0.0.1:53: no such host | 09:08 |
mnasiadka | how nice | 09:08 |
kevko | :D | 09:08 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate Kolla configs https://review.opendev.org/c/openstack/kayobe/+/804068 | 09:20 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile https://review.opendev.org/c/openstack/kayobe/+/811424 | 09:24 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile https://review.opendev.org/c/openstack/kayobe/+/811424 | 09:26 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile https://review.opendev.org/c/openstack/kayobe/+/811424 | 09:29 |
jingvar | Use merge_configs and merge_yaml to generate Kolla configs - I hope it will be merged into victoria | 09:31 |
opendevreview | Pierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile https://review.opendev.org/c/openstack/kayobe/+/811424 | 09:31 |
opendevreview | Merged openstack/kolla-ansible master: Transition Keystone admin user to system scope https://review.opendev.org/c/openstack/kolla-ansible/+/692179 | 09:33 |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: [CI] Test instance health after upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/806383 | 09:40 |
opendevreview | Merged openstack/kolla-ansible stable/victoria: [CI] Cinder upgrade testing https://review.opendev.org/c/openstack/kolla-ansible/+/808003 | 09:40 |
opendevreview | Merged openstack/kolla-ansible stable/victoria: [CI] Test instance health after upgrade https://review.opendev.org/c/openstack/kolla-ansible/+/806384 | 09:41 |
opendevreview | Merged openstack/kayobe master: Bump up Ansible supported versions https://review.opendev.org/c/openstack/kayobe/+/811410 | 10:04 |
opendevreview | Merged openstack/kayobe master: seed: Disable SNAT configuration by default https://review.opendev.org/c/openstack/kayobe/+/802684 | 10:35 |
opendevreview | Verification of a change to openstack/kayobe master failed: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 10:35 |
opendevreview | Merged openstack/kolla master: Build source type images by default https://review.opendev.org/c/openstack/kolla/+/811743 | 11:02 |
yoctozepto | ++ | 11:17 |
opendevreview | Verification of a change to openstack/kayobe stable/wallaby failed: CI: Disable libvirt debug logs https://review.opendev.org/c/openstack/kayobe/+/811500 | 11:40 |
*** amoralej is now known as amoralej|lunch | 12:15 | |
opendevreview | Verification of a change to openstack/kolla-ansible master failed: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 12:17 |
yoctozepto | what the | 12:40 |
yoctozepto | Internal Server Error (\"Get \"https://primary:4000/v2/\": http: server gave HTTP response to HTTPS client | 12:41 |
mnasiadka | shouldn't that be an insecure registry? | 12:41 |
yoctozepto | yeah | 12:41 |
yoctozepto | I have no idea what's happening | 12:41 |
mnasiadka | Insecure Registries: | 12:42 |
mnasiadka | 127.0.0.0/8 | 12:42 |
mnasiadka | seems it's not getting configured proprly | 12:42 |
yoctozepto | yeah, but it is not getting configured to use it unless it needs it | 12:43 |
yoctozepto | and then it's marked as insecure | 12:43 |
yoctozepto | it did not build the images either | 12:47 |
mnasiadka | it did build them and push them in the centos job | 12:49 |
mnasiadka | yoctozepto: but it didn't set docker_registry_insecure to True | 12:52 |
yoctozepto | mnasiadka: yeah, I mixed dirs | 12:55 |
yoctozepto | hmm | 12:55 |
mnasiadka | but tell me why it templated globals.yaml without that | 12:55 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 12:55 |
mnasiadka | rebased? :) | 12:55 |
yoctozepto | yeah, it seems in the gate pipeline it used some weird revision | 12:56 |
yoctozepto | a fresh rebase on top of the master branch should help | 12:56 |
mnasiadka | I guess | 12:56 |
yoctozepto | but this really triggers questions about the sanity of gate | 12:56 |
yoctozepto | it should have merged with the current state of master | 12:57 |
yoctozepto | ok, so the image build was triggered by 811743 in the gate queue which makes sense | 13:00 |
yoctozepto | but it seems it used some old kolla-ansible definitions for CI files which did not include the gate fix | 13:01 |
mnasiadka | corner case I guess | 13:01 |
yoctozepto | yeah, but looks like a bug in zuul | 13:02 |
yoctozepto | nothing that we can fix | 13:02 |
yoctozepto | oh, bugger | 13:06 |
yoctozepto | mnasiadka: look at the diff https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 13:06 |
yoctozepto | it removes the line | 13:06 |
yoctozepto | so it merged it "properly" | 13:06 |
yoctozepto | it's just that git had a different idea how it really should be | 13:06 |
yoctozepto | oh my | 13:06 |
yoctozepto | I wonder if anything else is b0rken as well | 13:07 |
mnasiadka | lol | 13:07 |
mnasiadka | globals-default is the only file where something was removed | 13:08 |
mnasiadka | let's just add it back | 13:08 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 13:08 |
yoctozepto | yeah, I added it back and removed the other re-addition | 13:08 |
yoctozepto | ok, reviewed it again and it's the only file that got ducked up | 13:10 |
yoctozepto | and I really can't explain what the openstack has happened | 13:10 |
mnasiadka | I guess let's hope it's the only weirdness we hit before X release :D | 13:12 |
yoctozepto | oh my, that true | 13:26 |
yoctozepto | but this one really freaks me out tbh | 13:27 |
yoctozepto | I can't believe the diffs anymore | 13:27 |
yoctozepto | ah, no | 13:28 |
yoctozepto | https://review.opendev.org/c/openstack/kolla-ansible/+/776662/19..20 | 13:28 |
yoctozepto | by the trace it shows mgoddard modified those lines on purpose | 13:28 |
yoctozepto | though I believe he did not, just was misguided by the editor | 13:28 |
yoctozepto | beware the gerrit editor | 13:28 |
*** amoralej|lunch is now known as amoralej | 13:34 | |
yoctozepto | mnasiadka: approve it so that it merges today and be gone | 13:41 |
mnasiadka | done | 13:41 |
yoctozepto | :D | 13:42 |
opendevreview | Mark Goddard proposed openstack/kayobe stable/ussuri: Make setup module arguments configurable https://review.opendev.org/c/openstack/kayobe/+/811784 | 14:08 |
mnasiadka | yoctozepto: https://review.opendev.org/c/openstack/kolla-ansible/+/791743 - removing? :) | 14:10 |
opendevreview | Mark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility https://review.opendev.org/c/openstack/kolla/+/811977 | 14:12 |
opendevreview | Mark Goddard proposed openstack/kolla-ansible master: CI: monasca: ignore exited monasca_thresh container https://review.opendev.org/c/openstack/kolla-ansible/+/807689 | 14:17 |
yoctozepto | mnasiadka: sure, later | 14:21 |
opendevreview | Merged openstack/kolla-ansible master: Deploy source type images by default https://review.opendev.org/c/openstack/kolla-ansible/+/811744 | 14:30 |
opendevreview | Merged openstack/kayobe stable/victoria: CI: Disable libvirt debug logs https://review.opendev.org/c/openstack/kayobe/+/811501 | 14:42 |
opendevreview | Merged openstack/kayobe master: Add support for apt proxy setting https://review.opendev.org/c/openstack/kayobe/+/799629 | 14:44 |
*** amoralej is now known as amoralej|off | 14:58 | |
opendevreview | Merged openstack/kayobe stable/wallaby: CI: Disable libvirt debug logs https://review.opendev.org/c/openstack/kayobe/+/811500 | 15:31 |
mnasiadka | mgoddard: what if we would just use remove_on_exit for monasca_thresh? | 15:59 |
mgoddard | mnasiadka: it needs to stay exited | 16:00 |
mnasiadka | it's so not normal, that it hurts, but not going to waste any more time on Monasca | 16:04 |
opendevreview | Merged openstack/kolla-ansible master: Add support for Ceph RadosGW integration https://review.opendev.org/c/openstack/kolla-ansible/+/776662 | 16:06 |
mnasiadka | yay | 16:07 |
yoctozepto | duck yeah | 16:08 |
opendevreview | Mark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility https://review.opendev.org/c/openstack/kolla/+/811977 | 16:13 |
opendevreview | Mark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility https://review.opendev.org/c/openstack/kolla/+/811977 | 16:16 |
yoctozepto | mnasiadka: quick merge ~> https://review.opendev.org/c/openstack/kolla/+/803617 | 16:23 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Stop adding the DROP workaround https://review.opendev.org/c/openstack/kolla-ansible/+/795856 | 16:47 |
opendevreview | Radosław Piliszek proposed openstack/kolla-ansible master: [CI] Test Ironic on Debian https://review.opendev.org/c/openstack/kolla-ansible/+/797190 | 16:48 |
opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Remove chrony role from kolla https://review.opendev.org/c/openstack/kolla-ansible/+/791743 | 16:56 |
*** mchlumsky4 is now known as mchlumsky | 18:23 | |
opendevreview | Merged openstack/kolla-ansible stable/wallaby: Use Docker healthchecks for memcached services https://review.opendev.org/c/openstack/kolla-ansible/+/806512 | 18:34 |
opendevreview | Merged openstack/kolla-ansible master: docs: ceph: add copy-on-write for Glance https://review.opendev.org/c/openstack/kolla-ansible/+/809992 | 18:34 |
*** mchlumsky0 is now known as mchlumsky | 18:38 | |
opendevreview | Mark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility https://review.opendev.org/c/openstack/kolla/+/811977 | 19:22 |
opendevreview | Merged openstack/kolla master: Avoid the use of --force in groupadd https://review.opendev.org/c/openstack/kolla/+/803617 | 20:22 |
opendevreview | Merged openstack/kolla-ansible master: Remove chrony role from kolla https://review.opendev.org/c/openstack/kolla-ansible/+/791743 | 21:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!