Thursday, 2021-09-30

opendevreviewBoxiang Zhu proposed openstack/kolla-ansible master: Add skyline role  https://review.opendev.org/c/openstack/kolla-ansible/+/81056602:14
opendevreviewBoxiang Zhu proposed openstack/kolla master: Add skyline Dockerfile  https://review.opendev.org/c/openstack/kolla/+/81079603:03
opendevreviewBoxiang Zhu proposed openstack/kolla-ansible master: Add skyline role  https://review.opendev.org/c/openstack/kolla-ansible/+/81056603:04
opendevreviewBoxiang Zhu proposed openstack/kolla master: Add skyline Dockerfile  https://review.opendev.org/c/openstack/kolla/+/81079603:59
*** amoralej|off is now known as amoralej06:03
mnasiadkayuriys: we are publishing images to both, just the default configuration in Xena has changed to use quay instead of docker hub.07:39
opendevreviewMark Goddard proposed openstack/kayobe stable/wallaby: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/81178207:41
opendevreviewMark Goddard proposed openstack/kayobe stable/victoria: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/81178307:41
opendevreviewMark Goddard proposed openstack/kayobe stable/victoria: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/81178307:45
opendevreviewMark Goddard proposed openstack/kayobe stable/ussuri: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/81178407:46
opendevreviewVerification of a change to openstack/kayobe stable/wallaby failed: CI: Disable libvirt debug logs  https://review.opendev.org/c/openstack/kayobe/+/81150007:57
mgoddardmorning08:04
kevkomorning \o/08:06
opendevreviewMark Goddard proposed openstack/kolla master: Build source type images by default  https://review.opendev.org/c/openstack/kolla/+/81174308:07
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Deploy source type images by default  https://review.opendev.org/c/openstack/kolla-ansible/+/81174408:07
mnasiadkamorning08:08
kevkomnasiadka: 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
mnasiadkakevko: it's feature freeze08:09
kevkoyeah, I know ... I was just asking if you tried skyline .. (not depends if via kolla or manually )08:10
mnasiadkaNo, I don't use Horizon, so who cares about replacement :)08:10
kevko:D 08:11
kevkoit's actually cool at first glance ..quite fast08:12
opendevreviewMark Goddard proposed openstack/kayobe master: Build and deploy source type images by default  https://review.opendev.org/c/openstack/kayobe/+/81188808:13
mnasiadkamgoddard: I think we should merge this as well - https://review.opendev.org/c/openstack/kolla/+/81101408:28
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: docs: ceph: add copy-on-write for Glance  https://review.opendev.org/c/openstack/kolla-ansible/+/80999208:32
opendevreviewBoxiang Zhu proposed openstack/kolla master: Add skyline Dockerfile  https://review.opendev.org/c/openstack/kolla/+/81079608:35
opendevreviewMark Goddard proposed openstack/kayobe stable/ussuri: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/81178408:50
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Add support for Ceph RadosGW integration  https://review.opendev.org/c/openstack/kolla-ansible/+/77666209:04
mnasiadkadial tcp: lookup quay.io on 127.0.0.1:53: no such host09:08
mnasiadkahow nice09:08
kevko:D 09:08
opendevreviewPierre Riteau proposed openstack/kayobe master: Use merge_configs and merge_yaml to generate Kolla configs  https://review.opendev.org/c/openstack/kayobe/+/80406809:20
opendevreviewPierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile  https://review.opendev.org/c/openstack/kayobe/+/81142409:24
opendevreviewPierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile  https://review.opendev.org/c/openstack/kayobe/+/81142409:26
opendevreviewPierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile  https://review.opendev.org/c/openstack/kayobe/+/81142409:29
jingvarUse merge_configs and merge_yaml to generate Kolla configs  - I hope it will be merged into victoria09:31
opendevreviewPierre Riteau proposed openstack/kayobe master: Add support for configuring tuned profile  https://review.opendev.org/c/openstack/kayobe/+/81142409:31
opendevreviewMerged openstack/kolla-ansible master: Transition Keystone admin user to system scope  https://review.opendev.org/c/openstack/kolla-ansible/+/69217909:33
opendevreviewMerged openstack/kolla-ansible stable/wallaby: [CI] Test instance health after upgrade  https://review.opendev.org/c/openstack/kolla-ansible/+/80638309:40
opendevreviewMerged openstack/kolla-ansible stable/victoria: [CI] Cinder upgrade testing  https://review.opendev.org/c/openstack/kolla-ansible/+/80800309:40
opendevreviewMerged openstack/kolla-ansible stable/victoria: [CI] Test instance health after upgrade  https://review.opendev.org/c/openstack/kolla-ansible/+/80638409:41
opendevreviewMerged openstack/kayobe master: Bump up Ansible supported versions  https://review.opendev.org/c/openstack/kayobe/+/81141010:04
opendevreviewMerged openstack/kayobe master: seed: Disable SNAT configuration by default  https://review.opendev.org/c/openstack/kayobe/+/80268410:35
opendevreviewVerification of a change to openstack/kayobe master failed: Add support for apt proxy setting  https://review.opendev.org/c/openstack/kayobe/+/79962910:35
opendevreviewMerged openstack/kolla master: Build source type images by default  https://review.opendev.org/c/openstack/kolla/+/81174311:02
yoctozepto++11:17
opendevreviewVerification of a change to openstack/kayobe stable/wallaby failed: CI: Disable libvirt debug logs  https://review.opendev.org/c/openstack/kayobe/+/81150011:40
*** amoralej is now known as amoralej|lunch12:15
opendevreviewVerification of a change to openstack/kolla-ansible master failed: Add support for Ceph RadosGW integration  https://review.opendev.org/c/openstack/kolla-ansible/+/77666212:17
yoctozeptowhat the12:40
yoctozeptoInternal Server Error (\"Get \"https://primary:4000/v2/\": http: server gave HTTP response to HTTPS client12:41
mnasiadkashouldn't that be an insecure registry?12:41
yoctozeptoyeah12:41
yoctozeptoI have no idea what's happening12:41
mnasiadkaInsecure Registries:12:42
mnasiadka  127.0.0.0/812:42
mnasiadkaseems it's not getting configured proprly12:42
yoctozeptoyeah, but it is not getting configured to use it unless it needs it12:43
yoctozeptoand then it's marked as insecure12:43
yoctozeptoit did not build the images either12:47
mnasiadkait did build them and push them in the centos job12:49
mnasiadkayoctozepto: but it didn't set docker_registry_insecure to True12:52
yoctozeptomnasiadka: yeah, I mixed dirs12:55
yoctozeptohmm12:55
mnasiadkabut tell me why it templated globals.yaml without that12:55
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration  https://review.opendev.org/c/openstack/kolla-ansible/+/77666212:55
mnasiadkarebased? :)12:55
yoctozeptoyeah, it seems in the gate pipeline it used some weird revision12:56
yoctozeptoa fresh rebase on top of the master branch should help12:56
mnasiadkaI guess12:56
yoctozeptobut this really triggers questions about the sanity of gate12:56
yoctozeptoit should have merged with the current state of master12:57
yoctozeptook, so the image build was triggered by 811743 in the gate queue which makes sense13:00
yoctozeptobut it seems it used some old kolla-ansible definitions for CI files which did not include the gate fix13:01
mnasiadkacorner case I guess13:01
yoctozeptoyeah, but looks like a bug in zuul13:02
yoctozeptonothing that we can fix13:02
yoctozeptooh, bugger13:06
yoctozeptomnasiadka: look at the diff https://review.opendev.org/c/openstack/kolla-ansible/+/77666213:06
yoctozeptoit removes the line13:06
yoctozeptoso it merged it "properly"13:06
yoctozeptoit's just that git had a different idea how it really should be13:06
yoctozeptooh my13:06
yoctozeptoI wonder if anything else is b0rken as well13:07
mnasiadkalol13:07
mnasiadkaglobals-default is the only file where something was removed13:08
mnasiadkalet's just add it back13:08
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible master: Add support for Ceph RadosGW integration  https://review.opendev.org/c/openstack/kolla-ansible/+/77666213:08
yoctozeptoyeah, I added it back and removed the other re-addition13:08
yoctozeptook, reviewed it again and it's the only file that got ducked up13:10
yoctozeptoand I really can't explain what the openstack has happened13:10
mnasiadkaI guess let's hope it's the only weirdness we hit before X release :D13:12
yoctozeptooh my, that true13:26
yoctozeptobut this one really freaks me out tbh13:27
yoctozeptoI can't believe the diffs anymore13:27
yoctozeptoah, no13:28
yoctozeptohttps://review.opendev.org/c/openstack/kolla-ansible/+/776662/19..2013:28
yoctozeptoby the trace it shows mgoddard modified those lines on purpose13:28
yoctozeptothough I believe he did not, just was misguided by the editor13:28
yoctozeptobeware the gerrit editor13:28
*** amoralej|lunch is now known as amoralej13:34
yoctozeptomnasiadka: approve it so that it merges today and be gone13:41
mnasiadkadone13:41
yoctozepto:D13:42
opendevreviewMark Goddard proposed openstack/kayobe stable/ussuri: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/81178414:08
mnasiadkayoctozepto: https://review.opendev.org/c/openstack/kolla-ansible/+/791743 - removing? :)14:10
opendevreviewMark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility  https://review.opendev.org/c/openstack/kolla/+/81197714:12
opendevreviewMark Goddard proposed openstack/kolla-ansible master: CI: monasca: ignore exited monasca_thresh container  https://review.opendev.org/c/openstack/kolla-ansible/+/80768914:17
yoctozeptomnasiadka: sure, later14:21
opendevreviewMerged openstack/kolla-ansible master: Deploy source type images by default  https://review.opendev.org/c/openstack/kolla-ansible/+/81174414:30
opendevreviewMerged openstack/kayobe stable/victoria: CI: Disable libvirt debug logs  https://review.opendev.org/c/openstack/kayobe/+/81150114:42
opendevreviewMerged openstack/kayobe master: Add support for apt proxy setting  https://review.opendev.org/c/openstack/kayobe/+/79962914:44
*** amoralej is now known as amoralej|off14:58
opendevreviewMerged openstack/kayobe stable/wallaby: CI: Disable libvirt debug logs  https://review.opendev.org/c/openstack/kayobe/+/81150015:31
mnasiadkamgoddard: what if we would just use remove_on_exit for monasca_thresh?15:59
mgoddardmnasiadka: it needs to stay exited16:00
mnasiadkait's so not normal, that it hurts, but not going to waste any more time on Monasca16:04
opendevreviewMerged openstack/kolla-ansible master: Add support for Ceph RadosGW integration  https://review.opendev.org/c/openstack/kolla-ansible/+/77666216:06
mnasiadkayay16:07
yoctozeptoduck yeah16:08
opendevreviewMark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility  https://review.opendev.org/c/openstack/kolla/+/81197716:13
opendevreviewMark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility  https://review.opendev.org/c/openstack/kolla/+/81197716:16
yoctozeptomnasiadka: quick merge ~> https://review.opendev.org/c/openstack/kolla/+/80361716:23
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible master: [CI] Stop adding the DROP workaround  https://review.opendev.org/c/openstack/kolla-ansible/+/79585616:47
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible master: [CI] Test Ironic on Debian  https://review.opendev.org/c/openstack/kolla-ansible/+/79719016:48
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Remove chrony role from kolla  https://review.opendev.org/c/openstack/kolla-ansible/+/79174316:56
*** mchlumsky4 is now known as mchlumsky18:23
opendevreviewMerged openstack/kolla-ansible stable/wallaby: Use Docker healthchecks for memcached services  https://review.opendev.org/c/openstack/kolla-ansible/+/80651218:34
opendevreviewMerged openstack/kolla-ansible master: docs: ceph: add copy-on-write for Glance  https://review.opendev.org/c/openstack/kolla-ansible/+/80999218:34
*** mchlumsky0 is now known as mchlumsky18:38
opendevreviewMark Goddard proposed openstack/kolla master: logstash: pin to 7.9.* for Elasticsearch OSS compatibility  https://review.opendev.org/c/openstack/kolla/+/81197719:22
opendevreviewMerged openstack/kolla master: Avoid the use of --force in groupadd  https://review.opendev.org/c/openstack/kolla/+/80361720:22
opendevreviewMerged openstack/kolla-ansible master: Remove chrony role from kolla  https://review.opendev.org/c/openstack/kolla-ansible/+/79174321:07

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!