| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: DNM Working around addr_assign_type change https://review.opendev.org/c/openstack/os-net-config/+/810532 | 00:38 |
|---|---|---|
| *** pmannidi is now known as pmannidi|AFK | 02:20 | |
| *** bhagyashris is now known as bhagyashris|rover | 04:08 | |
| *** pmannidi|AFK is now known as pmannidi | 05:15 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: [WIP] Remove EOL releases code https://review.opendev.org/c/openstack/tripleo-quickstart/+/810543 | 05:24 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: [WIP] Remove EOL releases code https://review.opendev.org/c/openstack/tripleo-quickstart/+/810543 | 05:30 |
| *** ysandeep|away is now known as ysandeep | 05:51 | |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: Wrap stopping podman -t with systemd timeouts https://review.opendev.org/c/openstack/tripleo-ansible/+/806753 | 05:59 |
| *** amoralej|off is now known as amoralej | 06:07 | |
| opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart master: Fix get-overcloud-nodes.py https://review.opendev.org/c/openstack/tripleo-quickstart/+/810546 | 06:26 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Allow to disable GSSAPI authentication https://review.opendev.org/c/openstack/tripleo-ansible/+/759057 | 06:27 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 06:29 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart master: Fix get-overcloud-nodes.py https://review.opendev.org/c/openstack/tripleo-quickstart/+/810546 | 06:31 |
| ramishra_ | ysandeep, chandankumar ^^ would possibly fix the ovb log collection issue | 06:31 |
| ysandeep | ramishra_: ack, thank you! | 06:35 |
| marios | chandankumar: anbanerj|ruck: am i reading that correctly needs something further or am i mistaken there https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175/4#message-5e8330f87b876c27a2ad56793483b3be6cc5b88d | 06:37 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart-extras master: [WIP] Remove EOL releases code https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/810551 | 06:51 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-ci master: Remove EOL releases code https://review.opendev.org/c/openstack/tripleo-ci/+/810552 | 06:56 |
| opendevreview | Merged openstack/tripleo-upgrade stable/ussuri: Allow applying workaround patches in containers. https://review.opendev.org/c/openstack/tripleo-upgrade/+/807385 | 07:02 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix the failure condition for HAproxy chgrp script https://review.opendev.org/c/openstack/tripleo-heat-templates/+/806951 | 07:02 |
| anbanerj|ruck | marios, hey. Yes something seems to be missing in this patch ^ . I am checking | 07:07 |
| *** anbanerj|ruck is now known as frenzy_friday | 07:08 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: Remove EOL releases code https://review.opendev.org/c/openstack/tripleo-repos/+/810555 | 07:08 |
| marios | ack thanks an | 07:20 |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates master: Only delete heat data in the DB if it exists https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810556 | 07:21 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 07:23 |
| *** jpena|off is now known as jpena | 07:30 | |
| *** rpittau|afk is now known as rpittau | 07:30 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 07:50 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 07:52 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: WIP Generate the inventory all the time in cli-config-download.yaml https://review.opendev.org/c/openstack/tripleo-ansible/+/808013 | 08:02 |
| manoj_katari | Hi experts, during overcloud node introspection, node tries for PXE boot but it couldn't recieve any DHCP offers, i want to see why dhcpserver is not offering it. ...the container neutron_dhcp should run dhcpserver right? | 08:04 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 08:25 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 08:35 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible stable/wallaby: fix galaxy meta requirements https://review.opendev.org/c/openstack/tripleo-ansible/+/810574 | 08:44 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible stable/wallaby: fix galaxy meta requirements https://review.opendev.org/c/openstack/tripleo-ansible/+/810574 | 08:52 |
| opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: Explicitly set python3 for ansible with get-hash to fix c7 jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/810410 | 09:04 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 09:14 |
| *** holser is now known as holser_ | 09:18 | |
| Tengu | marios: I don't see centos-7 job for your patch.. ? | 09:19 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 09:23 |
| Tengu | pfff... that damn dependency resolution.. | 09:23 |
| marios | Tengu: ack yeah it was tested in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/809687/3#message-1af73fc66b50695ae80c32ab29f0838724126958 | 09:23 |
| marios | Tengu: thanks for having a look | 09:24 |
| *** ysandeep is now known as ysandeep|lunch | 09:24 | |
| marios | Tengu: also waiting for more 3rd party sanity check at https://review.rdoproject.org/r/c/testproject/+/35745 | 09:25 |
| Tengu | :) | 09:25 |
| Tengu | marios: just wanted to ensure we actually have python3 on the centos-7 env... do we? | 09:25 |
| marios | Tengu: yeah we do and that is the source of the bug in fact. it is all py3 and all the deps etc are py3. but for ansible/python it uses py2 and so it cant find deps | 09:25 |
| marios | Tengu: https://bugs.launchpad.net/tripleo/+bug/1943968 comments 4 and 6 if interested for more | 09:26 |
| Tengu | oh, ok. perfect then | 09:26 |
| Tengu | marios: btw - if you toggle the "WIP" flag on the DNM reviews, it will really prevent any unwanted merge. We (TripleO) should really use this new feature in gerrit | 09:27 |
| Tengu | marios: this can even be done with `git review -w' (and -W to remove the WIP flag) | 09:27 |
| Tengu | btw.... I think git-review manpage is missing a lot of options. | 09:27 |
| marios | Tengu: ack yeah i've seen folks using it the WIP i mean | 09:27 |
| Tengu | :) | 09:28 |
| * marios foooood | 09:29 | |
| Tengu | soon :) | 09:29 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 09:30 |
| Tengu | hmmm... and if we extend the use of that WIP flag, we might even ignore them from the opendevreview bot... that would be... yay | 09:31 |
| jpodivin | Tengu: that would be swell. I feel bad about flooding this chan with these | 09:34 |
| Tengu | same here with my attempts at getting that role-addition back voting | 09:35 |
| *** holser_ is now known as holser | 09:36 | |
| Tengu | marios: ah, thanks for your review on my gssapi thing | 09:37 |
| Tengu | weird, I don't see the molecule report.html though.... it's empty | 09:37 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Allow to disable GSSAPI authentication https://review.opendev.org/c/openstack/tripleo-ansible/+/759057 | 09:40 |
| Tengu | dang.... stupid typo spotted. | 09:40 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible stable/wallaby: fix galaxy meta requirements https://review.opendev.org/c/openstack/tripleo-ansible/+/810574 | 09:40 |
| *** holser is now known as holser_ | 09:45 | |
| opendevreview | Harald Jensås proposed openstack/tripleo-quickstart-extras master: Use network-v2 workflow for Controlplane VIP https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/806562 | 09:46 |
| hjensas | Can I get another review on https://review.opendev.org/c/openstack/tripleo-upgrade/+/807879 please? | 09:48 |
| chandankumar | ramishra_: thanks for fixing it | 09:53 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Fix the failure condition for HAproxy chgrp script https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810577 | 09:59 |
| opendevreview | Ananya proposed openstack/tripleo-quickstart master: Fail collect-logs playbook when "fetch overcloud node names and IPs" returns 1 https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175 | 10:01 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-heat-templates master: Add 'enabled' parameter for systemd module https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810602 | 10:02 |
| opendevreview | Ananya proposed openstack/tripleo-quickstart master: Fail collect-logs playbook when "fetch overcloud node names and IPs" returns 1 https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175 | 10:03 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible stable/wallaby: fix galaxy meta requirements https://review.opendev.org/c/openstack/tripleo-ansible/+/810574 | 10:09 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Revert "Ensure startup files exist before containers might be started" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810474 | 10:10 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-upgrade master: Fix update tasks for Ephemeral Heat https://review.opendev.org/c/openstack/tripleo-upgrade/+/806565 | 10:12 |
| *** holser_ is now known as holser | 10:13 | |
| opendevreview | Ananya proposed openstack/tripleo-quickstart master: Fail collect-logs playbook when "fetch overcloud node names and IPs" returns 1 https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175 | 10:20 |
| *** ysandeep|lunch is now known as ysandeep | 10:24 | |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-quickstart master: WIP: Test the abscense of tripleo.operator https://review.opendev.org/c/openstack/tripleo-quickstart/+/805805 | 10:26 |
| *** beagles is now known as eagles | 10:28 | |
| *** jcapitao is now known as jcapitao_lunch | 10:42 | |
| *** rlandy is now known as rlandy|rover | 10:49 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 10:55 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible stable/wallaby: fix galaxy meta requirements https://review.opendev.org/c/openstack/tripleo-ansible/+/810574 | 10:58 |
| arxcruz|ruck | dtantsur: hey man, around? need your help with something | 10:59 |
| arxcruz|ruck | dtantsur: https://bugs.launchpad.net/tripleo/+bug/1944617 | 10:59 |
| arxcruz|ruck | dtantsur: can you check the two comments there? | 10:59 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.3.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 11:05 |
| hjensas | arxcruz|ruck: ^^ shoud be fixed by https://review.opendev.org/c/openstack/tripleo-quickstart/+/810546 | 11:08 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 11:11 |
| arxcruz|ruck | hjensas: cool, thanks!!! | 11:11 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 11:12 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Fail collect-logs playbook when "fetch overcloud node names and IPs" returns 1 https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175 | 11:13 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 11:14 |
| dtantsur | arxcruz|ruck: hey, I guess hjensas addressed your question? | 11:16 |
| arxcruz|ruck | dtantsur: yup :) | 11:17 |
| dtantsur | I'm around but I haven't touched TripleO for a good while (working on openshift nowadays) | 11:17 |
| arxcruz|ruck | dtantsur: alles gut, danke mein Freund :) | 11:17 |
| dtantsur | :) | 11:17 |
| dtantsur | arxcruz|ruck: I see your German is progressing? | 11:18 |
| arxcruz|ruck | dtantsur: ich kann ein bischen auf Deutsch sprechen, aber nicht in langer Konversation | 11:19 |
| *** ysandeep is now known as ysandeep|afk | 11:19 | |
| dtantsur | toll :) | 11:20 |
| opendevreview | amolkahat proposed openstack/tripleo-quickstart-extras master: Added list type support for var workers_args https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/808748 | 11:25 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.4.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 11:25 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/victoria: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810409 | 11:28 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/ussuri: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810470 | 11:29 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/train: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810471 | 11:29 |
| *** ysandeep|afk is now known as ysandeep | 11:29 | |
| *** jpena is now known as jpena|lunch | 11:31 | |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/victoria: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810579 | 11:34 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/ussuri: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810580 | 11:34 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/victoria: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810612 | 11:38 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.5.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 11:39 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/victoria: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810579 | 11:40 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/ussuri: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810580 | 11:40 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 11:41 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/train: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802469 | 11:43 |
| opendevreview | Ananya proposed openstack/tripleo-quickstart master: Fail collect-logs playbook when "fetch overcloud node names and IPs" returns 1 https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175 | 11:44 |
| *** jcapitao_lunch is now known as jcapitao | 11:50 | |
| opendevreview | Douglas Viroel proposed openstack/tripleo-repos master: WIP - Improvements to yum-config module https://review.opendev.org/c/openstack/tripleo-repos/+/808216 | 11:51 |
| *** amoralej is now known as amoralej|lunch | 12:07 | |
| bhagyashris|rover | ramishra_, hi i have reported bug related to network provisioning https://bugs.launchpad.net/tripleo/+bug/1944715 i see you have fixed this issue in master same stuff we will need in stable wallaby as well https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810082 | 12:08 |
| bhagyashris|rover | can I cherry pick the same change for stable wallaby ? | 12:09 |
| ramishra_ | bhagyashris|rover: yes | 12:09 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/ussuri: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810470 | 12:10 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-heat-templates stable/wallaby: Add ipv6 network_data/vip_data for CI jobs https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810582 | 12:11 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-heat-templates stable/wallaby: Add ipv6 network_data/vip_data for CI jobs https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810582 | 12:11 |
| ramishra_ | ok, we did that at the same time:) | 12:11 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/train: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810471 | 12:12 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.5.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 12:13 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/ussuri: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810470 | 12:14 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations stable/train: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/810471 | 12:14 |
| opendevreview | amolkahat proposed openstack/tripleo-operator-ansible master: Modified shell_args plugin for nested lists. https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/807778 | 12:21 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-validations master: oslo-config-validator: container plugin is replaced with collections https://review.opendev.org/c/openstack/tripleo-validations/+/804193 | 12:30 |
| *** jpena|lunch is now known as jpena | 12:33 | |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-ansible stable/wallaby: fix galaxy meta requirements https://review.opendev.org/c/openstack/tripleo-ansible/+/810574 | 12:37 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.5.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 12:39 |
| bshephar | hjensas: Yeah, it should work with the later version of molecule thanks to Tengu's patch in molecule | 12:45 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-heat-templates master: Fx swift log redirection https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810602 | 12:45 |
| hjensas | bshephar: 3.5.x got me past the dependency conflict, but then I hit galaxy meta requirement issues. Let's see how CI likes the latest patchset. | 12:46 |
| bshephar | Yeah, that should do it. Similar stuff here: https://review.opendev.org/c/openstack/tripleo-ansible/+/809643 | 12:48 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Allow custom os sesrvice workers https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/810256 | 12:52 |
| Tengu | hjensas, bshephar note that apparently wallaby is also affected by this metadata thing | 12:53 |
| hjensas | dang, I missed one. And tripleo-operator-molecule-tripleo_overcloud_node_discover is broken for other reasons. | 12:53 |
| Tengu | woohoo | 12:54 |
| timeu | d | 12:55 |
| *** amoralej|lunch is now known as amoralej | 12:59 | |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.5.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 13:03 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Set cgroupns=host for neutron_dhcp and ovn-metadata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807604 | 13:17 |
| opendevreview | Merged openstack/tripleo-validations master: Tests for the 'icmp_ping' module. https://review.opendev.org/c/openstack/tripleo-validations/+/795269 | 13:18 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates stable/wallaby: Set cgroupns=host for neutron_dhcp and ovn-metadata https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810585 | 13:20 |
| *** holser is now known as holser_ | 13:21 | |
| opendevreview | wes hayutin proposed openstack/openstack-tempest-skiplist master: octavia load balancer tests are NOT consistently passing https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/810667 | 13:21 |
| opendevreview | Merged openstack/tripleo-ansible master: bnr: Fixed the ERROR 1141 when users has fqdn https://review.opendev.org/c/openstack/tripleo-ansible/+/807727 | 13:25 |
| opendevreview | Merged openstack/tripleo-common master: Detect image format for qemu-nbd --format https://review.opendev.org/c/openstack/tripleo-common/+/809486 | 13:25 |
| opendevreview | Artom Lifshitz proposed openstack/tripleo-heat-templates master: docs: AllowResizeToSameHost needs nova-api restarted https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810671 | 13:27 |
| opendevreview | Merged openstack/tripleo-ansible master: Revert "Use project_id with neutron resources" https://review.opendev.org/c/openstack/tripleo-ansible/+/799761 | 13:27 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Don't use *-dist.conf files in commands https://review.opendev.org/c/openstack/tripleo-heat-templates/+/809540 | 13:27 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-quickstart master: Raise tempest workers and set os workers numbers https://review.opendev.org/c/openstack/tripleo-quickstart/+/810264 | 13:28 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ci master: Raise os workers counts to meet tempest workers https://review.opendev.org/c/openstack/tripleo-ci/+/810672 | 13:28 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: DNM Working around addr_assign_type change https://review.opendev.org/c/openstack/os-net-config/+/810532 | 13:29 |
| opendevreview | Merged openstack/openstack-virtual-baremetal master: Use OS::Heat::None for extra role BMC https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/793804 | 13:31 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Bump molecule[test] to 3.5.x https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 13:33 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/810557 | 13:37 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-ci master: Raise os workers counts to meet tempest workers https://review.opendev.org/c/openstack/tripleo-ci/+/810672 | 13:43 |
| opendevreview | Martin Schuppert proposed openstack/puppet-tripleo stable/victoria: Extend fencing to hosts using fence_kubevirt agent. https://review.opendev.org/c/openstack/puppet-tripleo/+/810586 | 13:46 |
| opendevreview | Martin Schuppert proposed openstack/puppet-tripleo stable/ussuri: Extend fencing to hosts using fence_kubevirt agent. https://review.opendev.org/c/openstack/puppet-tripleo/+/810587 | 13:46 |
| opendevreview | Harald Jensås proposed openstack/tripleo-operator-ansible master: Add '--vip-file' to overcloud deploy https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/805011 | 13:50 |
| opendevreview | Martin Schuppert proposed openstack/puppet-tripleo stable/train: Extend fencing to hosts using fence_kubevirt agent. https://review.opendev.org/c/openstack/puppet-tripleo/+/810682 | 13:51 |
| opendevreview | Arx Cruz proposed openstack/openstack-tempest-skiplist master: octavia load balancer tests are NOT consistently passing https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/810667 | 13:51 |
| opendevreview | Merged openstack/python-tripleoclient master: Update deployment status after upgrade converge https://review.opendev.org/c/openstack/python-tripleoclient/+/810140 | 13:52 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Don't create ExtraConfig resources per role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795204 | 13:55 |
| opendevreview | Merged openstack/tripleo-upgrade master: Add extra_args to update prepare and converge https://review.opendev.org/c/openstack/tripleo-upgrade/+/807879 | 13:55 |
| opendevreview | Arx Cruz proposed openstack/openstack-tempest-skiplist master: Add 'all' wildcard for releases https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/810010 | 13:55 |
| Tengu | cloudnull, mwhahaha heya! I'm trying to re-enable voting on tripleo-ansible "role-addition" job, but there's, really, an issue at some point. And I just don't get it at all :(. https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 if you have any idea.. | 14:14 |
| mwhahaha | Tengu: there's a bug around the log uploading failing somewhere | 14:15 |
| Tengu | mwhahaha: it's not really due to failed upload log :/ | 14:15 |
| * mwhahaha invokes arxcruz|ruck rlandy|rover weshay|ruck | 14:15 | |
| Tengu | there are logs. sort of. but mostly, the job seems to be stuck on the deps install for the tox job | 14:16 |
| rlandy|rover | looking | 14:16 |
| mwhahaha | that's also a bug | 14:16 |
| mwhahaha | which some would like to solve via pip-compile, and I would like to avoid pip-compile like the plague (not the covid one) | 14:16 |
| Tengu | mwhahaha: yeah - that's why I'm trying to fix "just" a couple of dependencies, but it's not changing anything in the end. and «locally it works», and is really fast. | 14:17 |
| mwhahaha | i guess i could spend today in ci land | 14:20 |
| Tengu | mwhahaha: that would be really nice - though I'd like to understand what's going on, so if we could pair a bit... :) | 14:20 |
| mwhahaha | pip is stupid | 14:20 |
| Tengu | not new | 14:21 |
| Tengu | :) | 14:21 |
| mwhahaha | right, it's gotten stupider | 14:21 |
| mwhahaha | i might even call it american at this point | 14:21 |
| Tengu | is it possible? | 14:21 |
| mwhahaha | yes | 14:21 |
| Tengu | what, trumpian stupid? | 14:21 |
| * Tengu runs away | 14:21 | |
| mwhahaha | quite possibly at this rate | 14:22 |
| rlandy|rover | Tengu: are you looking for anything still from our side - we have bugs open against log upload - but not in this case | 14:23 |
| mwhahaha | rlandy|rover: where's the bug for stuck pip installs | 14:24 |
| Tengu | rlandy|rover: I don't think it's hitting any bug upload, it's just pip not doing anything | 14:24 |
| rlandy|rover | https://bugs.launchpad.net/tripleo/+bug/1938266? | 14:26 |
| Tengu | would that be something related? https://github.com/pypa/pip/issues/9969 ? | 14:26 |
| mwhahaha | https://bugs.launchpad.net/tripleo/+bug/1942704 | 14:27 |
| mwhahaha | but yes, it looks like it's that issue | 14:27 |
| Tengu | oh, that the one leading to the pip-compile. oook. | 14:27 |
| mwhahaha | i really really don't want pip-compile so let me see if there's an alternative | 14:28 |
| Tengu | mwhahaha: maybe ensuring we're running latest pip ? | 14:29 |
| mwhahaha | we're at the mercy of upstream for pip versions | 14:30 |
| mwhahaha | and that issue wasn't resolved | 14:30 |
| mwhahaha | just closed | 14:30 |
| mwhahaha | oh maybe it was https://github.com/pypa/pip/pull/9993 | 14:30 |
| mwhahaha | let me look | 14:30 |
| Tengu | hmmm not 100% sure - the OC seems to say it's still not solved. | 14:31 |
| mwhahaha | ya i'll poke at it | 14:31 |
| Tengu | but ensuring we're running the latest pip can help. maybe. or introduce new issues :D | 14:31 |
| Tengu | is pip running podman under the hood? | 14:31 |
| * Tengu runs | 14:31 | |
| Tengu | (or the reverse - podman runs pip...) | 14:32 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: DNM Removing validations CLI from tripleoclient https://review.opendev.org/c/openstack/python-tripleoclient/+/810700 | 14:32 |
| mwhahaha | there's likely a specific dependency we're including thats causing it because tripleo-operator-ansible is working | 14:32 |
| Tengu | mwhahaha: still true nowadays? | 14:33 |
| Tengu | when was the last CI run for that bad boy? | 14:33 |
| mwhahaha | harold just fixed ci this morning | 14:35 |
| mwhahaha | https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/810609 | 14:36 |
| Tengu | hmmm there isn't that role-addition job | 14:36 |
| mwhahaha | it's the molecule part of role-addition where it's timing out | 14:38 |
| mwhahaha | 2021-09-23 11:07:08.119514 | centos-8 | role-addition installdeps: pre-commit, virtualenv, -r/home/zuul/src/opendev.org/openstack/tripleo-ansible/ansible-requirements.txt, -r/home/zuul/src/opendev.org/openstack/tripleo-ansible/molecule-requirements.txt | 14:38 |
| Tengu | mwhahaha: that molecule job seems to actually call testenv:role-addition - isn't it? | 14:39 |
| mwhahaha | not certain, i'll poke at it. but if we're just including ansible/molecule it seems related to those deps | 14:39 |
| Tengu | oh. wait. so... it's failing *before* actually hitting that tox env? | 14:39 |
| mwhahaha | yea | 14:40 |
| mwhahaha | it's on the install | 14:40 |
| Tengu | that tox testenv has a huge bunch of deps, btw | 14:40 |
| mwhahaha | it might be pre-commit tbh | 14:40 |
| Tengu | wondering if we really need pre-commit though? | 14:40 |
| mwhahaha | we don't need it | 14:40 |
| mwhahaha | but undoing it might be a bit rough | 14:41 |
| Tengu | so we probably can remove/replace {[testenv:linters]deps} | 14:41 |
| Tengu | and put -r {toxinidir}/ansible-requirements.txt instead. | 14:41 |
| Tengu | especially since "linters" testenv seems to be launched everytime we run tox. | 14:42 |
| Tengu | better: there's a dedicated "tox-linters". | 14:42 |
| Tengu | (job) | 14:42 |
| mwhahaha | no comment | 14:42 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 14:43 |
| Tengu | let's test..... | 14:43 |
| Tengu | mwhahaha: might be due to the molecule version bump leading to some weird dependency things for that specific testenv. hopefully reducing the amount of installed stuff will help. | 14:49 |
| Tengu | job started.... | 14:50 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Fix libvirt/qemu logs location https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810170 | 14:52 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade master: Fix update tasks for Ephemeral Heat https://review.opendev.org/c/openstack/tripleo-upgrade/+/806565 | 14:53 |
| *** amoralej is now known as amoralej|off | 15:04 | |
| Tengu | mwhahaha: so yep. not related to pre-commit. now it just has role-addition installdeps: -r/home/zuul/src/opendev.org/openstack/tripleo-ansible/ansible-requirements.txt, -r/home/zuul/src/opendev.org/openstack/tripleo-ansible/molecule-requirements.txt | 15:09 |
| Tengu | mwhahaha: aaand I'm wondering if, at that point, we really need to get the molecule-requirements.txt | 15:09 |
| opendevreview | Merged openstack/tripleo-repos master: Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 15:10 |
| Tengu | hmmm. apparently that role-addition is triggered as soon as we modify molecule related things (among other files) | 15:10 |
| Tengu | yeah. it will time out. | 15:16 |
| Tengu | so does my brain actually. | 15:16 |
| Tengu | guess I'll check it back tomorrow. | 15:16 |
| Tengu | mwhahaha: if you want to take that patch over, please, feel free. | 15:16 |
| mwhahaha | k will poke at it in a bit | 15:16 |
| Tengu | thanks | 15:16 |
| opendevreview | Giulio Fidente proposed openstack/tripleo-ansible master: Allow LVM filter to work when either allowlist or denylist are empty https://review.opendev.org/c/openstack/tripleo-ansible/+/810714 | 15:32 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-quickstart master: Update featureset 10 to deploy with secure RBAC's project personas https://review.opendev.org/c/openstack/tripleo-quickstart/+/809749 | 15:49 |
| opendevreview | Lance Bragstad proposed openstack/tripleo-quickstart master: Update featureset 20 to use secure RBAC project personas https://review.opendev.org/c/openstack/tripleo-quickstart/+/809811 | 15:49 |
| *** rpittau is now known as rpittau|afk | 16:02 | |
| opendevreview | Merged openstack/openstack-virtual-baremetal master: Set explicit boot mode even for bios boot https://review.opendev.org/c/openstack/openstack-virtual-baremetal/+/810332 | 16:05 |
| *** marios is now known as marios|out | 16:06 | |
| *** ysandeep is now known as ysandeep|dinner | 16:06 | |
| *** rlandy|rover is now known as rlandy|ruck | 16:24 | |
| mwhahaha | Tengu, cloudnull: FTR it's an issue with dependency resolution of molecule | 16:28 |
| *** jpena is now known as jpena|off | 16:33 | |
| mwhahaha | openstack + molecule on py36 doesn't appear to be a winning combination anymore | 16:42 |
| cloudnull | yay! | 16:50 |
| mwhahaha | ansible-core + molecule + os-net-config | 16:54 |
| mwhahaha | that's all it takes for pip to fail | 16:54 |
| mwhahaha | but only if in a requirements file, you can totally pip install those three | 16:57 |
| *** ysandeep|dinner is now known as ysandeep | 17:09 | |
| mwhahaha | https://github.com/ansible-community/molecule/pull/3221 likely did it | 17:10 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-ansible master: DNM os_net_config logging to a file https://review.opendev.org/c/openstack/tripleo-ansible/+/810752 | 17:28 |
| opendevreview | Merged openstack/tripleo-quickstart master: Fix get-overcloud-nodes.py https://review.opendev.org/c/openstack/tripleo-quickstart/+/810546 | 17:39 |
| opendevreview | Merged openstack/tripleo-puppet-elements master: Add octavia client to the installed packages for the overcloud-image https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/807791 | 17:39 |
| opendevreview | Merged openstack/python-tripleoclient master: Remove zaqar https://review.opendev.org/c/openstack/python-tripleoclient/+/809474 | 17:39 |
| opendevreview | Merged openstack/tripleo-common master: Removal mistral https://review.opendev.org/c/openstack/tripleo-common/+/809464 | 17:39 |
| opendevreview | Alex Schultz proposed openstack/tripleo-ansible master: Don't use upper-constraints with molecule https://review.opendev.org/c/openstack/tripleo-ansible/+/810760 | 17:58 |
| cloudnull | ^ fun | 17:58 |
| mwhahaha | pretty much | 17:58 |
| mwhahaha | meh seems still broke | 18:00 |
| * mwhahaha flips tables | 18:00 | |
| mwhahaha | the new pip resolver is terrible | 18:28 |
| mwhahaha | that is all | 18:28 |
| mwhahaha | it worked in ci ?! | 18:44 |
| opendevreview | Alex Schultz proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 18:45 |
| opendevreview | Alex Schultz proposed openstack/tripleo-quickstart-extras master: Rework requirements and drop pip-compile https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/810767 | 18:56 |
| Tengu | wow, well done mwhahaha | 19:09 |
| opendevreview | Alex Schultz proposed openstack/tripleo-quickstart-extras master: Rework requirements and drop pip-compile https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/810767 | 19:14 |
| opendevreview | Alex Schultz proposed openstack/tripleo-quickstart master: Rework requirements and drop pip-compile https://review.opendev.org/c/openstack/tripleo-quickstart/+/810769 | 19:19 |
| *** ysandeep is now known as ysandeep|out | 19:33 | |
| *** dviroel is now known as dviroel|out | 20:56 | |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Fail collect-logs playbook when "fetch overcloud node names and IPs" returns 1 https://review.opendev.org/c/openstack/tripleo-quickstart/+/810175 | 21:29 |
| opendevreview | David Hill proposed openstack/tripleo-heat-templates master: Add NovaAllowResizeSameHost to set nova::api::allow_resize_to_same_host https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810780 | 21:54 |
| rlandy|ruck | stevebaker: hi ... tracking this error on ci.centos jobs: https://jenkins-cloudsig-ci.apps.ocp.ci.centos.org/job/tripleo-quickstart-promote-victoria-current-tripleo-delorean-minimal/22/consoleFull | 22:02 |
| rlandy|ruck | MSG: | 22:03 |
| rlandy|ruck | The conditional check 'not whole_disk_images|bool' failed. The error was: error while evaluating conditional (not whole_disk_images|bool): 'whole_disk_images' is undefined | 22:03 |
| rlandy|ruck | https://github.com/openstack/tripleo-quickstart/blob/bd3f9e2a370e510864d0e946ef6be98c9887fe66/roles/libvirt/setup/undercloud/tasks/main.yml#L73 | 22:04 |
| stevebaker | rlandy|ruck: yeah I see | 22:04 |
| rlandy|ruck | default whole_disk_images? | 22:05 |
| rlandy|ruck | to true? | 22:05 |
| rlandy|ruck | or add a value to all the release files in centosci? | 22:06 |
| rlandy|ruck | stevebaker: ^^ which is the right way to go here? | 22:06 |
| opendevreview | Ade Lee proposed openstack/tripleo-ansible master: Fix octavia deployment tasks for FIPS environment https://review.opendev.org/c/openstack/tripleo-ansible/+/809174 | 22:07 |
| opendevreview | Ade Lee proposed openstack/tripleo-ansible master: Fix tripleo_nova_image_cache to not use md5 for validation https://review.opendev.org/c/openstack/tripleo-ansible/+/810783 | 22:07 |
| stevebaker | rlandy|ruck: it should be false by default. tripleo-quickstart-extras sets that in extras-common. But the libvirt role is the only one which uses it in tripleo-quickstart, so I guess the default should be set in roles/libvirt/defaults | 22:09 |
| stevebaker | rlandy|ruck: actually, roles/common/defaults would be good | 22:09 |
| * rlandy|ruck logging bug - will add patch in a sec | 22:10 | |
| stevebaker | rlandy|ruck: btw I would really like a sanitized copy of instackenv.json to be published in the log, and I found this https://review.opendev.org/c/openstack/tripleo-ci/+/689273 | 22:10 |
| rlandy|ruck | hmmm ... we publish that in ovb | 22:11 |
| opendevreview | Ade Lee proposed openstack/tripleo-ci master: WIP/DNM - Add fips job https://review.opendev.org/c/openstack/tripleo-ci/+/808215 | 22:11 |
| stevebaker | rlandy|ruck: really? I've not found it | 22:11 |
| rlandy|ruck | hmm - oh yeah and then removed :( | 22:12 |
| rlandy|ruck | ok - will add a task for that | 22:12 |
| opendevreview | Michele Baldessari proposed openstack/puppet-tripleo stable/ussuri: Fix rabbitmq certificate reload after it is resubmitted https://review.opendev.org/c/openstack/puppet-tripleo/+/806330 | 22:14 |
| stevebaker | rlandy|ruck: I think only pm_user, pm_password would need to be sanitized. I assume ironic logs are full of those mac and IP addresses | 22:16 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Default whole_disk_images in tripleo-quickstart https://review.opendev.org/c/openstack/tripleo-quickstart/+/810784 | 22:23 |
| rlandy|ruck | stevebaker: ^^ pls review | 22:25 |
| opendevreview | Ade Lee proposed openstack/tripleo-ci master: WIP/DNM - Add fips job https://review.opendev.org/c/openstack/tripleo-ci/+/808215 | 22:27 |
| opendevreview | Alex Schultz proposed openstack/tripleo-heat-templates master: Nova: Add support for initial_*_allocation_ratio https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803293 | 22:30 |
| opendevreview | Ade Lee proposed openstack/tripleo-ci master: DNM - Testing FIPS in multinode jobs too https://review.opendev.org/c/openstack/tripleo-ci/+/810786 | 22:33 |
| opendevreview | Alex Schultz proposed openstack/tripleo-ansible master: Toggle role-addition back to voting https://review.opendev.org/c/openstack/tripleo-ansible/+/810547 | 22:49 |
| opendevreview | Carlos Eduardo proposed openstack/puppet-tripleo master: Add support for share backend availability zones https://review.opendev.org/c/openstack/puppet-tripleo/+/810791 | 23:40 |
| opendevreview | Carlos Eduardo proposed openstack/tripleo-heat-templates master: Add parameters for manila storage availability zones https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810792 | 23:42 |
| opendevreview | Lewis Denny proposed openstack/tripleo-heat-templates master: Fix delegate_to to use group rather then undercloud https://review.opendev.org/c/openstack/tripleo-heat-templates/+/810389 | 23:56 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!