Friday, 2021-06-18

opendevreviewPierre Riteau proposed openstack/kolla master: Revert "libvirt: pin edk2-ovmf"  https://review.opendev.org/c/openstack/kolla/+/79549005:35
opendevreviewPierre Riteau proposed openstack/kolla stable/wallaby: Revert "libvirt: pin edk2-ovmf"  https://review.opendev.org/c/openstack/kolla/+/79549905:36
priteauc8s AV repo finally updated, crossing fingers that it works!05:37
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Bump up Ansible to 2.11  https://review.opendev.org/c/openstack/kolla-ansible/+/79675806:05
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Bump up Ansible to 3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/79675806:07
*** hrww is now known as hrw06:32
*** hrw is now known as Guest260806:33
*** Guest2608 is now known as hrw06:33
opendevreviewMichal Nasiadka proposed openstack/kolla stable/victoria: Add support for CentOS 8 Stream  https://review.opendev.org/c/openstack/kolla/+/79276307:11
mnasiadkapriteau: adv-virt rpm in extras released07:18
priteauI know, I posted a couple of hours ago ;-)07:19
priteauUnfortunately it didn't get included in the build07:19
mnasiadkapriteau: my irc client stalled it seems, now it refreshed ;)07:19
priteauIs opendev using actual mirrors (not caching proxies with short TTL) for RPMs?07:19
priteauhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_bf1/795490/2/check/kolla-build-centos8s-source/bf14d42/kolla/build/base.log07:20
priteaucentos-release-advanced-virtualization  noarch  1.0-2.el807:20
mnasiadkaactual mirrors07:20
mnasiadkapriteau: and now it seems it's hard to check what is in the mirror from the internet...07:22
priteau:( so we need to wait until they sync07:22
priteauyeah I am trying to browse http://mirror.iad.rax.opendev.org:8080/ but only getting 404s07:22
mnasiadkaI think :8080 is the proxy url we use for container images07:23
mnasiadkaRUN sed -i         -e "s|^\(mirrorlist.*\)|#\1|"         -e "s|^#baseurl=http://mirror.centos.org|baseurl=http://mirror-int.iad.rax.opendev.org|"         /etc/yum.repos.d/CentOS-*.repo07:23
mnasiadkano port in the centos repos override07:23
mnasiadkapriteau: anyway, it will get updated over the weekend and we'll check on Monday07:24
priteauI thought we could release today. Oh well07:25
priteauI updated the whiteboard07:28
priteauyoctozepto, hrw: Could either of you please review https://review.opendev.org/c/openstack/kolla-ansible/+/780848?07:32
priteaumnasiadka: at https://mirror.iad.rax.opendev.org/centos/8-stream/extras/x86_64/os/Packages/ I see the 1.0-3 package07:34
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Support editable installation in all cases  https://review.opendev.org/c/openstack/kolla-ansible/+/79686707:37
opendevreviewRadosław Piliszek proposed openstack/kolla stable/wallaby: Fix _get_images_dir when pip-installed with --user  https://review.opendev.org/c/openstack/kolla/+/79686807:38
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/victoria: Support editable installation in all cases  https://review.opendev.org/c/openstack/kolla-ansible/+/79686907:39
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Support editable installation in all cases  https://review.opendev.org/c/openstack/kolla-ansible/+/79687007:43
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/ussuri: Support editable installation in all cases  https://review.opendev.org/c/openstack/kolla-ansible/+/79687007:43
opendevreviewRadosław Piliszek proposed openstack/kolla stable/victoria: Fix _get_images_dir when pip-installed with --user  https://review.opendev.org/c/openstack/kolla/+/79687107:43
mnasiadkapriteau: you don't know when it showed up, and I doubt releasing on Friday is a good habit :)07:43
mnasiadkaAnd I would be more confident if we let the CI jobs to run through the weekend and check for failures :)07:44
priteauInstalling:07:44
priteau centos-release-advanced-virtualization  noarch  1.0-3.el807:44
yoctozeptopriteau: done07:44
yoctozeptomorning07:45
priteaurecheck is pulling the updated package now :)07:45
priteaumnasiadka: if we can get a +1 from zuul we can at least get reviews today07:46
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/train: Support editable installation in all cases  https://review.opendev.org/c/openstack/kolla-ansible/+/79701407:47
opendevreviewRadosław Piliszek proposed openstack/kolla stable/ussuri: Fix _get_images_dir when pip-installed with --user  https://review.opendev.org/c/openstack/kolla/+/79701507:47
kevko\o/07:47
priteauand if merged today, we get docker builds on sunday and can run aufn early next week before releasing07:47
opendevreviewRadosław Piliszek proposed openstack/kolla stable/train: Fix _get_images_dir when pip-installed with --user  https://review.opendev.org/c/openstack/kolla/+/79701607:47
yoctozeptoaufn?07:48
priteauyoctozepto: https://github.com/stackhpc/a-universe-from-nothing/07:49
yoctozeptoah, I understand it only in context :-)07:52
yoctozeptokayobe aufn would ring a bell07:52
yoctozeptoanyway07:52
yoctozeptotrain is ready to cut out em07:52
yoctozeptowhich we should do this week for sure07:52
yoctozeptothere are two optional we might want to squeeze for slightly better experienc07:53
yoctozeptooptional patches*07:53
yoctozeptohttps://review.opendev.org/c/openstack/kolla-ansible/+/79701407:53
yoctozeptohttps://review.opendev.org/c/openstack/kolla/+/79701607:53
opendevreviewMichal Arbet proposed openstack/kolla-ansible master: Add missing region_name in keystoneauth sections  https://review.opendev.org/c/openstack/kolla-ansible/+/79198007:53
mnasiadkayoctozepto: squeezed ;)07:55
priteaulibvirt-daemon x86_64 7.4.0-1.el8s08:06
priteauThis time it's working08:06
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701708:10
yoctozeptomnasiadka: thx08:16
yoctozeptoI will update the release patch once it merges08:16
mnasiadkapriteau: good, less backports to make stable/victoria working with Stream08:17
*** rpittau|afk is now known as rpittau08:17
opendevreviewScott Solkhon proposed openstack/kolla-ansible master: Add support for storing passwords in Hashicorp Vault  https://review.opendev.org/c/openstack/kolla-ansible/+/79299708:24
opendevreviewMark Goddard proposed openstack/kayobe master: docs: Improve all-in-one scenario  https://review.opendev.org/c/openstack/kayobe/+/79700308:26
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701708:31
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701708:43
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701708:44
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701708:45
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: CentOS 8 Stream: Use variable image tag suffix  https://review.opendev.org/c/openstack/kolla-ansible/+/79700508:47
opendevreviewMerged openstack/kolla-ansible master: Update blazar.conf template  https://review.opendev.org/c/openstack/kolla-ansible/+/78084808:47
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/wallaby: Update blazar.conf template  https://review.opendev.org/c/openstack/kolla-ansible/+/79701808:50
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701708:53
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/victoria: Update blazar.conf template  https://review.opendev.org/c/openstack/kolla-ansible/+/79701908:54
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/ussuri: Update blazar.conf template  https://review.opendev.org/c/openstack/kolla-ansible/+/79702008:55
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/train: Update blazar.conf template  https://review.opendev.org/c/openstack/kolla-ansible/+/79702108:56
*** hrw is now known as Guest262009:06
*** hrww is now known as hrw09:06
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible stable/victoria: WIP: CI: Move to centos-8-stream  https://review.opendev.org/c/openstack/kolla-ansible/+/79701709:26
priteauJobs are passing with libvirt 7.4.0 \o/09:38
priteaue.g. https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_2fe/795490/2/check/kolla-ansible-centos8s-binary/2fe46ba/primary/logs/kolla/libvirt/libvirtd.txt09:38
yoctozeptoyeah, i saw it09:40
yoctozeptoI suggest we merge09:40
yoctozeptoand release on Monday09:40
yoctozeptowdyt09:40
priteauhttps://review.opendev.org/q/I1f58262109dd27a4c4ee9b60d87010170b5ea7bb awaiting reviews09:40
mnasiadkadone09:41
yoctozepto++09:41
priteauThanks Michal09:42
priteauyoctozepto: Monday sounds good09:42
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/wallaby: Redis configuration syntax update  https://review.opendev.org/c/openstack/kolla-ansible/+/79702309:45
opendevreviewRadosław Piliszek proposed openstack/kolla-ansible stable/victoria: Redis configuration syntax update  https://review.opendev.org/c/openstack/kolla-ansible/+/77774709:57
yoctozeptomnasiadka: could you drop cr+2 off https://review.opendev.org/c/openstack/kolla-ansible/+/777746 ?10:02
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/wallaby: Merge glance sections for nova.conf.j2  https://review.opendev.org/c/openstack/kolla-ansible/+/79702510:05
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/victoria: Merge glance sections for nova.conf.j2  https://review.opendev.org/c/openstack/kolla-ansible/+/79702610:07
mnasiadkayoctozepto: done10:09
opendevreviewPierre Riteau proposed openstack/kolla-ansible stable/ussuri: Merge glance sections for nova.conf.j2  https://review.opendev.org/c/openstack/kolla-ansible/+/79704210:09
*** hrww is now known as hrw10:29
*** hrw is now known as Guest263110:29
scottsolo/ 10:30
scottsoldoes anyone know of any helpful resources to add new tests into the CI? I am planning on pushing a new feature which could possibly do with some new tests but this is not something I have needed to do before in my previous patches 10:30
*** Guest2631 is now known as hrw10:35
opendevreviewMerged openstack/kolla-ansible stable/train: Support editable installation in all cases  https://review.opendev.org/c/openstack/kolla-ansible/+/79701410:38
mnasiadkascottsol: I don't think we have any docs, but feel free to look into tests/run.yml and ask questions :)10:41
scottsolmnasiadka: Thanks, thats a good place to start :). Is there anything special I need to do to run them locally before pushing up to Zuul?10:47
mnasiadkascottsol: well, it might be hard to run that playbook locally because it relies on zuul vars, but you can see what is the logic, and we run test-*.sh scripts based on scenario (job) name - those probably can be run locally with some minimal effort.10:48
scottsolaA yes, I see. I'll give that a go. Thanks!10:50
opendevreviewMerged openstack/kolla master: Revert "libvirt: pin edk2-ovmf"  https://review.opendev.org/c/openstack/kolla/+/79549010:59
opendevreviewScott Solkhon proposed openstack/kolla-ansible master: Add support for storing passwords in Hashicorp Vault  https://review.opendev.org/c/openstack/kolla-ansible/+/79299711:18
opendevreviewMark Goddard proposed openstack/kayobe master: Support configuration of firewalld  https://review.opendev.org/c/openstack/kayobe/+/79706811:22
opendevreviewVerification of a change to openstack/kolla failed: Fix _get_images_dir when pip-installed with --user  https://review.opendev.org/c/openstack/kolla/+/79701611:43
yoctozeptomnasiadka: thx11:52
opendevreviewMichal Nasiadka proposed openstack/kayobe master: kolla: Set neutron_plugin_agent to OVN when it's enabled  https://review.opendev.org/c/openstack/kayobe/+/79707212:11
opendevreviewMichal Nasiadka proposed openstack/kayobe master: kolla: Set neutron_plugin_agent to OVN when it's enabled  https://review.opendev.org/c/openstack/kayobe/+/79707212:13
opendevreviewVerification of a change to openstack/kolla failed: Revert "libvirt: pin edk2-ovmf"  https://review.opendev.org/c/openstack/kolla/+/79549912:43
mnasiadkauhh, we lost a job? nice12:58
mnasiadkait's first time a see a job with state LOST :)12:58
yoctozeptoah, so the ci is like a lottery: sometimes you win, sometimes you lose :-)13:01
opendevreviewMerged openstack/kolla-ansible stable/wallaby: Merge glance sections for nova.conf.j2  https://review.opendev.org/c/openstack/kolla-ansible/+/79702513:19
opendevreviewMerged openstack/kolla-ansible stable/victoria: Merge glance sections for nova.conf.j2  https://review.opendev.org/c/openstack/kolla-ansible/+/79702613:19
opendevreviewMerged openstack/kolla-ansible stable/ussuri: Merge glance sections for nova.conf.j2  https://review.opendev.org/c/openstack/kolla-ansible/+/79704213:19
opendevreviewMichal Nasiadka proposed openstack/kayobe master: kolla: Set neutron_plugin_agent to OVN when it's enabled  https://review.opendev.org/c/openstack/kayobe/+/79707213:38
opendevreviewMichal Nasiadka proposed openstack/kolla-ansible master: WIP: Bump up Ansible to 3.0  https://review.opendev.org/c/openstack/kolla-ansible/+/79675813:42
*** samcat11_ is now known as samcat11613:57
samcat116running into this error when trying to run seed hypervisor host configure in kayobe13:58
samcat116http://paste.openstack.org/show/806767/13:58
priteaumnasiadka: I have also never seen LOST before13:58
yoctozeptoyeah, I have not either, but it has good reviews in imdb: https://www.imdb.com/title/tt0411008/14:00
scottsolmnasiadka: I have a script to run which will test my patch. This apples to storing the passwords generated in the passwords.yml externally in Hashi Vault. This test needs to be run as part of a configuration test so it doesn't require an openstack control plane to be deployed. Is there a way to set this so that no services are deployed and instead it just sets up the configuration then runs my script?14:01
priteausamcat116: did you configure seed_hypervisor_libvirt_networks in etc/kayobe/seed-hypervisor.yml?14:02
samcat116Yes its in the bottom of the paste14:02
yoctozeptoscottsol: you don't want to run any other deployment/tests?14:02
priteausamcat116: sorry, didn't see that14:03
scottsolthis is pre deployment so does not need to deploy openstack as part of the test 14:03
priteausamcat116: so have you set <admin-net-name>_libvirt_network_name for it?14:04
samcat116I have set it to a list with one entry which is what I have in networks.yml as the name for the admin_oc_net_name14:05
priteauyou set admin_oc_net_libvirt_network_name to a list? it should be a string14:06
priteautry "admin_oc_net_libvirt_network_name: admin"14:07
priteauthis will be the name used inside libvirt14:07
samcat116Hmm ok14:07
samcat116The comments in the kayobe-config repo says "List of libvert networks for the seed hypervisor."14:08
priteauso, in seed-hypervisor.yml, you set seed_hypervisor_libvirt_networks: to a list14:09
priteauit's a list of names of kayobe networks14:09
*** rpittau is now known as rpittau|afk14:09
priteaubut each network needs to be defined in networks.yml14:09
samcat116Yep14:10
priteauand have a *_libvirt_network_name variable associated14:10
priteauwhich itself is a string14:10
priteauhope that makes sense14:10
samcat116Ah, thats the part I missed. didn't realize you needed libvirt networks there as well14:10
priteauonly for those networks that are used by libvirt14:10
priteauto be honest maybe we could default to the network name instead14:10
priteauand only use libvirt name if set14:10
samcat116Or maybe add a section about it in here -> https://opendev.org/openstack/kayobe-config/src/branch/master/etc/kayobe/networks.yml14:11
yoctozeptoscottsol: but you don't want to test kolla ansible? or do you want to add a pre-deployment test? I think I need a link to the proposal to understand it; I guess you could propose without the tests first so that we could help you14:12
scottsolyoctozepto: https://review.opendev.org/c/openstack/kolla-ansible/+/792997 I have done just that :)14:13
scottsolI have written a script to bring up a Hashi Vault dev server and then it tests writing the passwords.yml into Vault and then reading that to regenerate the same password.yml 14:14
yoctozeptoscottsol: can kolla-ansible use these passwords directly from the vault now?14:14
yoctozeptoah, this way14:14
yoctozeptoI think it would need a separate playbook as it doesn't require like 95% magic that we have there14:16
yoctozeptomight need refactoring to do it right14:16
scottsolokay might need some help if thats the case as this would be my first time pushing any new CI jobs 14:25
scottsolis it worth me pushing up the script too, and then next try to work out how to get the CI to run it in the best way possible 14:26
opendevreviewScott Solkhon proposed openstack/kolla-ansible master: Add support for storing passwords in Hashicorp Vault  https://review.opendev.org/c/openstack/kolla-ansible/+/79299714:45
opendevreviewMerged openstack/kolla stable/train: Fix _get_images_dir when pip-installed with --user  https://review.opendev.org/c/openstack/kolla/+/79701615:01
yoctozeptopriteau: is kayobe train em ready? (just noticed the patch is about all the kolla deliverables)15:55
yoctozeptohttps://review.opendev.org/q/project:openstack/kayobe+branch:stable/train+status:open15:56
yoctozeptoanything important? or important backports?15:56
priteauThe centos8.4 one is just a docs patch, if mgoddard or mnasiadka could approve it15:58
priteauI guess we want to abandon https://review.opendev.org/c/openstack/kayobe/+/73820815:59
opendevreviewMark Goddard proposed openstack/kayobe master: Support configuration of firewalld  https://review.opendev.org/c/openstack/kayobe/+/79706815:59
mgoddardappoved the docs patch15:59
mgoddardwe don't need to abandon patches on EM branches16:00
priteauthanks mgoddard16:00
priteauI confused em and eol16:00
mgoddardEM = we don't promise to support16:00
mgoddardEOL = we promise not to support16:00
priteaumore importantly, stable/train still open while in em, while it's gone for eol, right?16:01
mgoddardyes16:01
opendevreviewMerged openstack/kayobe stable/train: Update CentOS 8 docs to use 8.4 image  https://review.opendev.org/c/openstack/kayobe/+/79683316:07
opendevreviewMerged openstack/kolla stable/wallaby: Revert "libvirt: pin edk2-ovmf"  https://review.opendev.org/c/openstack/kolla/+/79549916:17
yoctozeptoyes, open and keep the patches16:21
yoctozeptoonly abandon those you would abandon anyway16:21
yoctozeptonothing special16:21
yoctozeptook, so I can update the release patch16:21
yoctozeptohad to ask release team how we should bump the sha now16:25
yoctozeptoshould we release numbered first?16:25
yoctozeptoperhaps mgoddard remembers?16:25
yoctozeptobtw, wallaby is also ready, if someone wanted to propose rc2...16:28
priteaumgoddard: want to approve https://review.opendev.org/c/openstack/kayobe-config-dev/+/795692 as well?16:28
yoctozeptooh, so I should wait anyway, good16:28
priteau(not needed for train-em, it's CI only)16:28
priteauJust cleaning up open patches16:29
mgoddardyoctozepto: there is a new-release script, or you can just manually edit the file16:29
yoctozeptomgoddard: yeah, I know the technical; but need to know the process; got reply from hberaud - separate patches is the way to go16:29
yoctozeptopriteau: ah, thanks16:30
yoctozeptoI think - kolla:feature; kolla-ansible:bugfix; kayobe:feature16:33
yoctozeptoversion bumps16:33
yoctozeptoconsidering renos16:33
yoctozeptomgoddard: https://review.opendev.org/c/openstack/releases/+/79714016:39
mgoddardLGTM, thanks16:41
opendevreviewMark Goddard proposed openstack/kayobe master: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kayobe/+/79197616:59
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Use ansible_facts to reference facts  https://review.opendev.org/c/openstack/kolla-ansible/+/79127617:02
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kolla-ansible/+/78339217:06
opendevreviewBuddhika Sanjeewa proposed openstack/kolla stable/victoria: Fix build of masakari-monitors image  https://review.opendev.org/c/openstack/kolla/+/79710217:36
opendevreviewBuddhika Sanjeewa proposed openstack/kolla stable/victoria: Fix build of masakari-monitors image  https://review.opendev.org/c/openstack/kolla/+/79710217:39
opendevreviewBuddhika Sanjeewa proposed openstack/kolla stable/victoria: Fix build of masakari-monitors image  https://review.opendev.org/c/openstack/kolla/+/79710217:53
opendevreviewMerged openstack/kayobe-config-dev stable/train: CI: Sync with Kayobe jobs  https://review.opendev.org/c/openstack/kayobe-config-dev/+/79569218:21
mgoddardcephadm jobs seem a bit broken18:51
mgoddardhttps://80137ce53930819135d8-42d904af0faa486c8226703976d821a0.ssl.cf2.rackcdn.com/783392/12/check/kolla-ansible-centos8s-source-cephadm/175dba7/primary/logs/kolla/nova/nova-compute.txt18:51
mgoddardah18:52
mgoddardhttps://80137ce53930819135d8-42d904af0faa486c8226703976d821a0.ssl.cf2.rackcdn.com/783392/12/check/kolla-ansible-centos8s-source-cephadm/175dba7/primary/logs/docker_logs/nova_libvirt.txt18:52
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Make setup module arguments configurable  https://review.opendev.org/c/openstack/kolla-ansible/+/78339218:53
mgoddardBroken by https://review.opendev.org/c/openstack/kolla-ansible/+/65085318:53
yoctozeptowho approves such oldies? :P18:54
mgoddardto be fair it failed in the last run, just NV jobs18:55
yoctozeptolet's perhaps switch one of them to be voting18:56
yoctozeptoand gating18:56
mgoddardwell, you know how often they fail18:56
yoctozeptowell, a tradeoff18:57
yoctozeptonon-upgrades should not be as painful18:57
mgoddardyeah18:59
mgoddardwe could try it18:59
opendevreviewMark Goddard proposed openstack/kolla-ansible master: Fix up 'Persist nova libvirt secrets in a Docker volume'  https://review.opendev.org/c/openstack/kolla-ansible/+/79715118:59
mgoddard^ I think that'll do it18:59
mgoddard\o19:00
*** samcat11_ is now known as samcat11620:20

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