| opendevreview | Merged openstack/ansible-role-openstack-operations master: Fix ci https://review.opendev.org/c/openstack/ansible-role-openstack-operations/+/798117 | 00:24 |
|---|---|---|
| opendevreview | Merged openstack/ansible-role-openstack-operations master: Correct the tox option for skipping sdist generation https://review.opendev.org/c/openstack/ansible-role-openstack-operations/+/796901 | 00:25 |
| opendevreview | Merged openstack/tripleo-docs master: Update tox.ini for tripleo-docs https://review.opendev.org/c/openstack/tripleo-docs/+/789588 | 00:26 |
| opendevreview | Owen McGonagle proposed openstack/tripleo-validations master: remove using containers-prepare-parameter.yaml https://review.opendev.org/c/openstack/tripleo-validations/+/796061 | 00:51 |
| *** slagle is now known as slagle-afk | 01:02 | |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-docs master: Adding the vDPA deployment guide https://review.opendev.org/c/openstack/tripleo-docs/+/798237 | 02:05 |
| stevebaker | I want, I want my, I want my OVB | 02:48 |
| tkajinam | I have been curious about this but what ovb stands for ? | 03:00 |
| tkajinam | afk | 03:01 |
| stevebaker | tkajinam: OpenStack Virtual Baremetal, a heat stack creates nova servers plus an IPMI compatable interface which makes them look like real baremetal. This can then be used by CI jobs to deploy TripleO using Ironic to manage the 'virtual' baremetal https://opendev.org/openstack/openstack-virtual-baremetal/ | 03:09 |
| stevebaker | tkajinam: it used to be called quintupleo, like tripleo on tripleo, but that was deemed unprofessional :p | 03:10 |
| bshephar | stevebaker: Ahhhh, so quintupleo actually survived in a different form! I remember seeing the git repo for it a while ago and having a chuckle | 03:13 |
| stevebaker | yes! | 03:13 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart master: Disable tempest in undercloud jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/798786 | 03:41 |
| tkajinam | stevebaker, you solved one of the biggest mysteries I had ! | 03:46 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-ansible master: Remove hardcoded cloud_name and auth_type used https://review.opendev.org/c/openstack/tripleo-ansible/+/798522 | 03:52 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-quickstart master: Disable tempest in undercloud jobs https://review.opendev.org/c/openstack/tripleo-quickstart/+/798786 | 03:54 |
| stevebaker | tkajinam: you're welcome, bnemec did most of the early stuff https://opendev.org/openstack/openstack-virtual-baremetal/commits/branch/master?page=10 | 04:18 |
| stevebaker | 6 years | 04:18 |
| * stevebaker is so old ;( | 04:19 | |
| *** ykarel|away is now known as ykarel | 04:24 | |
| *** sshnaidm is now known as sshnaidm|afk | 04:39 | |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: sriov_config: light refactoring https://review.opendev.org/c/openstack/os-net-config/+/798787 | 04:41 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: sriov_config: light refactoring https://review.opendev.org/c/openstack/os-net-config/+/798787 | 04:54 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: sriov_config: light refactoring https://review.opendev.org/c/openstack/os-net-config/+/798787 | 04:55 |
| *** marios is now known as marios|ruck | 05:08 | |
| *** ysandeep|away is now known as ysandeep | 05:10 | |
| opendevreview | Slawek Kaplonski proposed openstack/openstack-tempest-skiplist master: Unskip test_allocate_ips_are_from_correct_subnet test https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/798111 | 05:42 |
| opendevreview | Rabi Mishra proposed openstack/tripleo-heat-templates master: Deploy standalone ironic and neutron in undercloud https://review.opendev.org/c/openstack/tripleo-heat-templates/+/794912 | 05:54 |
| *** amoralej|off is now known as amoralej | 06:13 | |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart master: Remove tripleo-operator-ansible from TQE requires https://review.opendev.org/c/openstack/tripleo-quickstart/+/790701 | 07:01 |
| opendevreview | Gregory Thiemonge proposed openstack/tripleo-heat-templates master: Enable LogOffload by default in Octavia https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798437 | 07:05 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Adding static invenory param to tripleo_validator_run https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/798793 | 07:12 |
| *** marios is now known as marios|ruck | 07:16 | |
| bandini | do we have a job that tests introspection in master? | 07:17 |
| bandini | ysandeep: ^ maybe you know? | 07:17 |
| ysandeep | bandini, yes ovb jobs in third party | 07:17 |
| bandini | ack thanks | 07:18 |
| * bandini looks | 07:18 | |
| ysandeep | bandini, let me grab an example for you | 07:18 |
| bandini | thanks! | 07:18 |
| bandini | 'cause introspection is broken for me lately | 07:18 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-ansible stable/wallaby: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-ansible/+/798794 | 07:19 |
| ysandeep | bandini: https://logserver.rdoproject.org/openstack-periodic-integration-main/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-master/8f7215c/logs/undercloud/home/zuul/overcloud_introspect.log.txt.gz | 07:21 |
| bandini | aha thanks ysandeep! | 07:21 |
| ysandeep | bandini, what error you are noticing in introspection, may be i have seen that? | 07:22 |
| bandini | ysandeep: http://paste.openstack.org/show/807060/ am still looking | 07:23 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: Add zuul.d/molecule.yaml file back in the yamllint ignore list https://review.opendev.org/c/openstack/tripleo-validations/+/797884 | 07:33 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: Sort jobs in molecule.yaml file when adding new role https://review.opendev.org/c/openstack/tripleo-validations/+/797885 | 07:33 |
| ysandeep | bandini: thanks! i have not seen similiar issue in upstream yet, but worth to note that ovb jobs are not real baremetal.. so we may have other issues if you are testing on real baremetal | 07:33 |
| *** jpena|off is now known as jpena | 07:39 | |
| bandini | ysandeep: ah thanks, yeah I think I can reproduce 100%, so I'll spend some more time debugging and then file an LP bug | 07:42 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-ansible stable/victoria: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-ansible/+/798798 | 07:42 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-ansible stable/ussuri: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-ansible/+/798799 | 07:59 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 08:00 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Coverage change job https://review.opendev.org/c/openstack/tripleo-validations/+/797285 | 08:00 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Coverage change job https://review.opendev.org/c/openstack/validations-common/+/797250 | 08:08 |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: haproxy: Enable httplog for all http endpoints https://review.opendev.org/c/openstack/puppet-tripleo/+/798323 | 08:09 |
| *** marios is now known as marios|ruck | 08:09 | |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-ansible stable/train: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-ansible/+/798801 | 08:10 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Moving setting validation from nova_event_callback https://review.opendev.org/c/openstack/tripleo-validations/+/794866 | 08:34 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: Introducing the oslo_config_validator role https://review.opendev.org/c/openstack/tripleo-validations/+/797101 | 08:38 |
| opendevreview | mbu proposed openstack/validations-common master: Remove content-provider to decrease CI time https://review.opendev.org/c/openstack/validations-common/+/798270 | 08:42 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/wallaby: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798766 | 08:47 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Deprecate --network-ports, enable it by default https://review.opendev.org/c/openstack/python-tripleoclient/+/798753 | 08:58 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/wallaby: Remove NovaVncProxyNetwork from ServiceNetMap https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798805 | 09:04 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/wallaby: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798806 | 09:05 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/victoria: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798809 | 09:13 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Callback logging unification https://review.opendev.org/c/openstack/validations-common/+/797618 | 09:14 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/victoria: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798810 | 09:14 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/victoria: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798809 | 09:15 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/victoria: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798809 | 09:19 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/ussuri: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798812 | 09:20 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/ussuri: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798813 | 09:21 |
| *** ykarel is now known as ykarel|lunch | 09:24 | |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/train: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798817 | 09:26 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/train: Enabling indentation rule for yamllint. https://review.opendev.org/c/openstack/tripleo-validations/+/797389 | 09:29 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/train: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798820 | 09:29 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Introducing the oslo_config_validator role https://review.opendev.org/c/openstack/tripleo-validations/+/797103 | 09:35 |
| opendevreview | Merged openstack/ansible-role-collect-logs master: Add errors.txt in artcl_logstash_files https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/794196 | 09:35 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations stable/victoria: Migrate back haproxy validation to tripleo-validations https://review.opendev.org/c/openstack/tripleo-validations/+/798809 | 09:49 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: [WIP] Add new metadata keys to the validation playbooks https://review.opendev.org/c/openstack/tripleo-validations/+/796674 | 09:55 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Ansible lint custom rule expansion https://review.opendev.org/c/openstack/tripleo-validations/+/798636 | 10:01 |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart master: use py3 for tq on rhel 7.8+ https://review.opendev.org/c/openstack/tripleo-quickstart/+/798823 | 10:11 |
| opendevreview | Marios Andreou proposed openstack/python-tripleoclient stable/wallaby: Default Heat images to container image prepare defaults https://review.opendev.org/c/openstack/python-tripleoclient/+/798824 | 10:16 |
| opendevreview | Harald Jensås proposed openstack/tripleo-ansible master: Add option to set template paths in env population https://review.opendev.org/c/openstack/tripleo-ansible/+/798370 | 10:31 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Add '--templates' arg to node and net provision https://review.opendev.org/c/openstack/python-tripleoclient/+/798371 | 10:31 |
| opendevreview | Harald Jensås proposed openstack/tripleo-ansible master: Add option to set template paths in env population https://review.opendev.org/c/openstack/tripleo-ansible/+/798370 | 10:33 |
| *** ykarel|lunch is now known as ykarel | 10:33 | |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates master: Add comment for libvirt/vnc network sync https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798826 | 10:34 |
| opendevreview | mbu proposed openstack/tripleo-validations master: Remove content-provider to decrease CI time https://review.opendev.org/c/openstack/tripleo-validations/+/798286 | 10:38 |
| opendevreview | mbu proposed openstack/validations-libs master: Remove content-provider to decrease CI time https://review.opendev.org/c/openstack/validations-libs/+/798267 | 10:38 |
| opendevreview | mbu proposed openstack/validations-libs master: Add validation config file mechanism https://review.opendev.org/c/openstack/validations-libs/+/792460 | 10:42 |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade master: Allow applying workaround patches in containers. https://review.opendev.org/c/openstack/tripleo-upgrade/+/703121 | 10:51 |
| *** sshnaidm|afk is now known as sshnaidm | 10:53 | |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: [CentOS-7] pin os_tempest ansible role to a known hash https://review.opendev.org/c/openstack/tripleo-ci/+/798827 | 10:57 |
| *** slagle-afk is now known as slagle | 10:58 | |
| marios|ruck | matbu: o/ check oooq i digged at train/validation component but let me know if that is what you are after | 11:04 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/victoria: Remove NovaVncProxyNetwork from ServiceNetMap https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798832 | 11:05 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/ussuri: Remove NovaVncProxyNetwork from ServiceNetMap https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798833 | 11:06 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/train: Remove NovaVncProxyNetwork from ServiceNetMap https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798834 | 11:06 |
| opendevreview | chandan kumar proposed openstack/tripleo-heat-templates stable/queens: [DNM] os_Tempest queens testing https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798835 | 11:09 |
| marios|ruck | slagle: o/ hey when you have some time please check https://bugs.launchpad.net/tripleo/+bug/1931995/comments/13 https://review.opendev.org/c/openstack/python-tripleoclient/+/798824/1#message-dfabfe6f5a86a8039661003ba1f9158c4066d8bf | 11:13 |
| opendevreview | James Slagle proposed openstack/python-tripleoclient stable/wallaby: Use ephemeral Heat by default https://review.opendev.org/c/openstack/python-tripleoclient/+/796304 | 11:13 |
| opendevreview | James Slagle proposed openstack/python-tripleoclient stable/wallaby: Default Heat images to container image prepare defaults https://review.opendev.org/c/openstack/python-tripleoclient/+/798824 | 11:13 |
| opendevreview | James Slagle proposed openstack/python-tripleoclient stable/wallaby: Handle KeyboardInterrupt with overcloud deploy https://review.opendev.org/c/openstack/python-tripleoclient/+/798836 | 11:13 |
| marios|ruck | slagle: ok i think that is my answer :) going to abandon https://review.opendev.org/c/openstack/python-tripleoclient/+/798824/ | 11:14 |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates stable/ussuri: Make memcache also listen to localhost https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798838 | 11:15 |
| marios|ruck | slagle: oh ... you used the same patch ... OK thanks | 11:15 |
| slagle | marios|ruck: no need to abandon. should be good to go now | 11:16 |
| marios|ruck | slagle: ack thanks i thought you posted a new one | 11:16 |
| bandini | ysandeep: I ended up filing https://bugs.launchpad.net/tripleo/+bug/1934116 https://bugs.launchpad.net/tripleo/+bug/1934118 and https://bugs.launchpad.net/tripleo/+bug/1934133 fwiw (all minor issues really...) | 11:17 |
| opendevreview | Mikolaj Ciecierski proposed openstack/tripleo-upgrade master: Allow applying workaround patches in containers. https://review.opendev.org/c/openstack/tripleo-upgrade/+/703121 | 11:20 |
| *** jpena is now known as jpena|lunch | 11:27 | |
| *** dviroel|out is now known as dviroel | 11:29 | |
| ysandeep | bandini, thanks! | 11:33 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: sriov_config: light refactoring https://review.opendev.org/c/openstack/os-net-config/+/798787 | 11:47 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Coverage change job https://review.opendev.org/c/openstack/validations-common/+/797250 | 11:51 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Coverage change job https://review.opendev.org/c/openstack/validations-common/+/797250 | 11:52 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 11:55 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: Coverage change job https://review.opendev.org/c/openstack/validations-common/+/797250 | 11:55 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: sriov_config: light refactoring https://review.opendev.org/c/openstack/os-net-config/+/798787 | 11:56 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Coverage change job https://review.opendev.org/c/openstack/tripleo-validations/+/797285 | 11:56 |
| *** marios|ruck is now known as marios|ruck|call | 12:01 | |
| opendevreview | Grzegorz Grasza proposed openstack/puppet-tripleo master: Set memcached server list from memcached_node_names https://review.opendev.org/c/openstack/puppet-tripleo/+/796832 | 12:02 |
| opendevreview | Grzegorz Grasza proposed openstack/tripleo-ci master: Test https://review.opendev.org/c/openstack/tripleo-ci/+/787955 | 12:04 |
| *** amoralej is now known as amoralej|lunch | 12:07 | |
| opendevreview | Lukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [WIP][FFWD][train-only] Ensure we get ovn_controller cert if needed https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798843 | 12:08 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: sriov_config: light refactoring https://review.opendev.org/c/openstack/os-net-config/+/798787 | 12:26 |
| *** jpena|lunch is now known as jpena | 12:27 | |
| cloudnull | mornings | 12:29 |
| *** gthiemon1e is now known as gthiemonge | 12:49 | |
| *** marios|ruck|call is now known as marios|ruck | 12:57 | |
| opendevreview | Ronelle Landy proposed openstack/openstack-tempest-skiplist master: Remove designate_tempest_plugin test from the skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/798390 | 13:02 |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates stable/train: Add support for keystone_authtoken/memcache_use_advanced_pool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795497 | 13:06 |
| opendevreview | Merged openstack/validations-common master: [check_cpu] Remove README.md file https://review.opendev.org/c/openstack/validations-common/+/797633 | 13:07 |
| *** amoralej|lunch is now known as amoralej | 13:16 | |
| slagle | cloudnull: do you recall the reason for the "reverse" here: https://review.opendev.org/c/openstack/tripleo-ansible/+/680485/6/tripleo_ansible/roles/tripleo-firewall/tasks/main.yml | 13:28 |
| slagle | the issue i'm having is that "999 drop all" is added first, which eventually causes my ssh connections to get dropped since it takes so long before the "003 accept ssh from all" rule is added | 13:29 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Adding static invenory param to tripleo_validator_run https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/798793 | 13:30 |
| opendevreview | Hitesh Kumar proposed openstack/diskimage-builder master: Migrate from testr to stestr https://review.opendev.org/c/openstack/diskimage-builder/+/789246 | 13:30 |
| slagle | also if the ssh connection times out, it's no longer an established connection allowed by iptables (i guess) | 13:32 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Adding static invenory param to tripleo_validator_run https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/798793 | 13:35 |
| cloudnull | slagle looking now | 13:35 |
| cloudnull | I don't specifically recall. I remember there being an issue caused by the data coming in from heat, and ordering of rules, but i don't remember all of the details. | 13:37 |
| slagle | ok, i guess i'll leave it | 13:38 |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Move atop installation after repo setup https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798412 | 13:40 |
| opendevreview | Ronelle Landy proposed openstack/openstack-tempest-skiplist master: Add rhos-17 jobs to recent skips https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/798698 | 13:43 |
| eagles | weshay|ruck: can I get you to revisit your -2 on https://review.opendev.org/c/openstack/tripleo-ci/+/797335 ? | 13:47 |
| *** ysandeep is now known as ysandeep|afk | 13:52 | |
| weshay|ruck | eagles, sure | 13:59 |
| eagles | weshay|ruck: thanks! | 14:04 |
| slagle | ramishra: hi, i'm seeing an issue with this one, https://review.opendev.org/c/openstack/tripleo-heat-templates/+/792724 | 14:07 |
| slagle | ramishra: the dropping of the "else" from the "if" seems to cause Heat to also drop the parent key | 14:08 |
| slagle | container_registry_insecure_registries is dropped from the set_fact when the "if" is false. if I added a {} as the else, then it stays | 14:08 |
| slagle | can check with: openstack stack create -t tripleo-heat-templates/deployment/podman/podman-baremetal-ansible.yaml reg | 14:09 |
| slagle | and then look at the stack outputs | 14:10 |
| ramishra | slagle: https://review.opendev.org/c/openstack/tripleo-ansible/+/793877 did not help? | 14:10 |
| mwhahaha | no because it gets set to 'undef' | 14:10 |
| mwhahaha | which ansible is like ok that's a string | 14:10 |
| mwhahaha | because ansible | 14:10 |
| ramishra | what is set to undef? AFAIK that fact would not be set at all | 14:11 |
| slagle | ramishra: no. b/c we try to use the var just later in the host_prep. in that same template | 14:11 |
| slagle | before we even execute the role | 14:11 |
| mwhahaha | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/792724/3/deployment/podman/podman-baremetal-ansible.yaml#82 | 14:12 |
| mwhahaha | the set fact is run and the var is set to 'undef' | 14:12 |
| mwhahaha | if you didn't want to define teh var, you'd need to put the task in an if condition | 14:12 |
| slagle | that is not what i'm saying | 14:13 |
| slagle | this is on the HEat side afaict, it's dropping the line entirely from the set_fact. the var does not exist at all | 14:13 |
| mwhahaha | that's what hjensas hit so he proposed https://review.opendev.org/c/openstack/tripleo-heat-templates/+/793871 | 14:13 |
| slagle | then this line blows up: https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/podman/podman-baremetal-ansible.yaml#L123 | 14:13 |
| mwhahaha | oh fun so two problems | 14:14 |
| ramishra | slagle: what's the issue if 'container_registry_insecure_registries' is not there | 14:14 |
| slagle | run that stack create command i put above | 14:14 |
| slagle | see that the var is completely absent from that set_fact task | 14:14 |
| mwhahaha | you're getting a jinja error then | 14:14 |
| slagle | ramishra: line 123 then fails with the undefined var | 14:14 |
| mwhahaha | because it's attempting to use an undefined var | 14:14 |
| ramishra | slagle: I'm in a customer call now, I'll check that later.. go ahead and patch if you see an issue | 14:15 |
| ramishra | we can mege hjensas's patch | 14:15 |
| hjensas | Should I restore it? | 14:15 |
| mwhahaha | i restored it | 14:16 |
| slagle | we should revert https://review.opendev.org/c/openstack/tripleo-heat-templates/+/792724 | 14:16 |
| slagle | that's the fix afaict. do we really want Heat dropping vars from our set_fact tasks? | 14:16 |
| ramishra | slagle: why revert? AFAIU hjensas patch should do | 14:16 |
| slagle | ramishra: it's the behavior of heat's if/else intrinsic function | 14:17 |
| slagle | with no "else", it drops the parent key | 14:17 |
| ramishra | I'll ccheck why it's dropping parent keys, it's a bug in heat then | 14:17 |
| opendevreview | Carlos Gonçalves proposed openstack/tripleo-ansible master: Add BGP EVPN support to tripleo_frr https://review.opendev.org/c/openstack/tripleo-ansible/+/798626 | 14:18 |
| opendevreview | Carlos Gonçalves proposed openstack/tripleo-ansible master: WIP DNM/DNR OVN BGP Agent support https://review.opendev.org/c/openstack/tripleo-ansible/+/785664 | 14:18 |
| slagle | ok, yea i agree that is unexpected | 14:18 |
| bogdando | oh, we need to check the other places then, there had been quite a change | 14:36 |
| bogdando | with parent keys affected over many places | 14:37 |
| *** ysandeep|afk is now known as ysandeep | 14:43 | |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates master: Do not fail if haproxy container UUID changes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798427 | 14:49 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: remove using containers-prepare-parameter.yaml https://review.opendev.org/c/openstack/tripleo-validations/+/796061 | 14:51 |
| *** ysandeep is now known as ysandeep|away | 15:02 | |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/wallaby: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798806 | 15:04 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/ussuri: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798812 | 15:06 |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/victoria: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798810 | 15:07 |
| *** ykarel is now known as ykarel|away | 15:08 | |
| opendevreview | yogananth Subramanian proposed openstack/tripleo-heat-templates stable/train: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798817 | 15:08 |
| *** amoralej is now known as amoralej|off | 15:34 | |
| ramishra | slagle: I just checked and heat behaviour is as expected, I initially thought that it was removing the 'set_fact' key when you said parent key, it's not doing that... The issue is we're using the var in L123 without a default | 15:44 |
| ramishra | http://pastebin.test.redhat.com/976031 | 15:44 |
| ramishra | So we either use a default there or restore hjensas patch | 15:45 |
| ramishra | the enclosing item, in this case 'container_registry_insecure_registries' would be exluded and that's expected behaviour | 15:46 |
| slagle | i think we should add the "else" back then. we don't need to set a default at 123, bc we had already set it earlier | 15:47 |
| slagle | i don't think we want heat munging our ansible tasks | 15:47 |
| slagle | i think that changes a lot of assumptions | 15:47 |
| ramishra | probably not, if we want to use the vars elsewhere in the tasks you can either use 'else' or default | 15:48 |
| ramishra | when I did that changes I missed that the var was used in L123 | 15:48 |
| marios|ruck | weshay|ruck: fyi http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023409.html (stein/queens) | 16:01 |
| weshay|ruck | wooot /me reads | 16:01 |
| weshay|ruck | this feels GOOOOOD | 16:02 |
| marios|ruck | weshay|ruck: just posted the reviews its been 2 weeks since we sent it | 16:02 |
| opendevreview | David Vallee Delisle proposed openstack/os-net-config master: vdpa: Introducing the vdpa PF flag https://review.opendev.org/c/openstack/os-net-config/+/798926 | 16:05 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-docs master: Adding the vDPA deployment guide https://review.opendev.org/c/openstack/tripleo-docs/+/798237 | 16:11 |
| *** marios|ruck is now known as marios|out | 16:12 | |
| opendevreview | James Slagle proposed openstack/python-tripleoclient stable/wallaby: Default Heat images to container image prepare defaults https://review.opendev.org/c/openstack/python-tripleoclient/+/798824 | 16:14 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Implement protected resource types https://review.opendev.org/c/openstack/python-tripleoclient/+/794593 | 16:27 |
| bogdando | ramishra: but http://pastebin.test.redhat.com/976031 shows that container_registry_insecure_registries has been removed from the output set_fact... | 16:30 |
| bogdando | and there only container_registry_login* remained | 16:31 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Introducing the ComputeVdpa role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797529 | 16:31 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Add rhos-17 jobs to recent skips https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/798698 | 16:32 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: remove designate plugin from skip https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/798388 | 16:32 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Dynamic defaults for roles, networks, vips and bm https://review.opendev.org/c/openstack/python-tripleoclient/+/797979 | 16:40 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Deprecate network data v1 definition https://review.opendev.org/c/openstack/python-tripleoclient/+/798642 | 16:41 |
| hjensas | another CR on https://review.opendev.org/c/openstack/tripleo-heat-templates/+/791447 please. 2x +2 and 1x +1 from upgrades already. | 16:46 |
| *** jpena is now known as jpena|off | 16:49 | |
| *** anbanerj|rover is now known as frenzy_friday | 17:07 | |
| ade_lee | weshay|ruck, rlandy hey - so I'm going to try creating a ci job that has fips enabled -- I wads thinking to start by taking fs39 and maybe the standalone ipa job and modify that | 17:31 |
| ade_lee | weshay|ruck, rlandy can you guys remind me where those jobs are defined? | 17:31 |
| weshay|ruck | upstream jobs are in tripleo-ci/zuul.d | 17:32 |
| ade_lee | weshay|ruck, ah cool - looking | 17:32 |
| weshay|ruck | ade_lee, I may have to kick a puppy if fs039 continues to live on | 17:32 |
| ade_lee | weshay|ruck, glad I'm inspiring your good side :) | 17:33 |
| rlandy | ade_lee: getting | 17:33 |
| weshay|ruck | ade_lee, w/ regards to FIPS.. is it something we can only enable w/ IDM? | 17:33 |
| ade_lee | weshay|ruck, no - not at all | 17:34 |
| rlandy | https://github.com/openstack/tripleo-ci/blob/master/zuul.d/standalone-jobs.yaml#L774 | 17:34 |
| weshay|ruck | ade_lee, THEN DEAR LORD WHY | 17:34 |
| * weshay|ruck perpares my -2 | 17:34 | |
| rlandy | https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/ovb-jobs.yaml#L45 | 17:34 |
| ade_lee | weshay|ruck, I just figured it would be a good idea to test that all the ssl connections and ciphers are ok | 17:35 |
| weshay|ruck | rlandy, don't show him | 17:35 |
| weshay|ruck | rlandy, RUN! | 17:35 |
| weshay|ruck | for you life | 17:35 |
| weshay|ruck | ade_lee, quick chat? | 17:35 |
| rlandy | weshay|ruck: he'll find them at some point :) | 17:35 |
| ade_lee | weshay|ruck, sure | 17:35 |
| weshay|ruck | ade_lee, meet.google.com/hng-dfxy-hpj | 17:35 |
| opendevreview | Yariv proposed openstack/tripleo-upgrade master: [FFWD upgrade] Adding support in compute name pattern https://review.opendev.org/c/openstack/tripleo-upgrade/+/798211 | 17:50 |
| opendevreview | Yariv proposed openstack/tripleo-upgrade master: [FFWD upgrade] Adding support in compute name pattern https://review.opendev.org/c/openstack/tripleo-upgrade/+/798211 | 17:51 |
| opendevreview | Merged openstack/python-tripleoclient master: Fix copying of deployment archive https://review.opendev.org/c/openstack/python-tripleoclient/+/798251 | 17:54 |
| weshay|ruck | ade_lee, https://opendev.org/openstack/tripleo-ci/src/branch/master/zuul.d/base.yaml#L380 | 17:59 |
| weshay|ruck | ade_lee, https://github.com/rdo-infra/rdo-jobs/blob/master/zuul.d/component-jobs.yaml | 17:59 |
| weshay|ruck | ade_lee, 0/ ianw | 18:00 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Enable DPDK OVS PMD Auto Load Balance https://review.opendev.org/c/openstack/tripleo-ansible/+/798794 | 18:04 |
| *** gfidente is now known as gfidente|afk | 18:11 | |
| opendevreview | James Slagle proposed openstack/tripleo-heat-templates master: Preface tasks path with {{playbook_dir}} https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798944 | 18:31 |
| cloudnull | weshay|ruck have you seen http://paste.openstack.org/show/807085/ ? -- looking at https://review.opendev.org/c/openstack/tripleo-common/+/796003 | 18:36 |
| cloudnull | content provider job failing, last run was 7 week ago, something known? | 18:37 |
| opendevreview | Kevin Carter proposed openstack/tripleo-common master: directly use files instread of tcib_gather_files https://review.opendev.org/c/openstack/tripleo-common/+/796003 | 18:37 |
| weshay|ruck | cloudnull, sec.. | 18:41 |
| weshay|ruck | looking | 18:50 |
| weshay|ruck | cloudnull, hrm.. bad answer but we should see what happens w/ the current run https://zuul.openstack.org/status#796003 | 18:51 |
| weshay|ruck | and then spend time | 18:51 |
| cloudnull | ++ i came to the same conclusion :D | 18:51 |
| weshay|ruck | it just started | 19:08 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Dynamic defaults for roles, networks, vips and bm https://review.opendev.org/c/openstack/python-tripleoclient/+/797979 | 19:09 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Deprecate network data v1 definition https://review.opendev.org/c/openstack/python-tripleoclient/+/798642 | 19:09 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Add build for ansible collections in build-test https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798641 | 19:14 |
| opendevreview | Ade Lee proposed openstack/tripleo-ci master: WIP/DNM - Add fips job https://review.opendev.org/c/openstack/tripleo-ci/+/798948 | 19:23 |
| opendevreview | Harald Jensås proposed openstack/tripleo-ansible master: Add option to set template paths in env population https://review.opendev.org/c/openstack/tripleo-ansible/+/798370 | 19:25 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Add '--templates' arg to node and net provision https://review.opendev.org/c/openstack/python-tripleoclient/+/798371 | 19:29 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: WIP version 2 - Add build for ansible collections in build-test https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798950 | 19:32 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Implement protected resource types https://review.opendev.org/c/openstack/python-tripleoclient/+/794593 | 20:03 |
| weshay|ruck | cloudnull, same thing /me looks | 20:13 |
| cloudnull | weird | 20:14 |
| cloudnull | idk what would cause the tests to fail, the builds seems to be working? | 20:15 |
| weshay|ruck | cloudnull, ya.. the job has a solid history.. the build report is not helping much here | 20:23 |
| * weshay|ruck poking | 20:24 | |
| weshay|ruck | https://5586db96f74bd1350af6-5e6b29b878d4494b037f975f2db877ec.ssl.cf1.rackcdn.com/796003/8/check/tripleo-ci-centos-8-content-provider/a388c39/logs/report.html | 20:24 |
| weshay|ruck | https://5586db96f74bd1350af6-5e6b29b878d4494b037f975f2db877ec.ssl.cf1.rackcdn.com/796003/8/check/tripleo-ci-centos-8-content-provider/a388c39/logs/container-builds/a2ef6483-2c64-417e-9149-567ce29e2d7f/base/os/iscsid/iscsid-build.log | 20:25 |
| weshay|ruck | is marked failed and has a log | 20:25 |
| weshay|ruck | cp: cannot stat '/usr/share/tripleo-common/container-images/kolla/iscsid/extend_start.sh': No such file or directory | 20:25 |
| weshay|ruck | ? | 20:25 |
| weshay|ruck | https://review.opendev.org/c/openstack/tripleo-common/+/796003/8/container-images/tcib/base/os/iscsid/iscsid.yaml | 20:26 |
| weshay|ruck | cloudnull, you broke it :) | 20:26 |
| weshay|ruck | flips tables | 20:26 |
| cloudnull | wat!? | 20:26 |
| weshay|ruck | that's right.. | 20:26 |
| weshay|ruck | I'm giving you 3 seconds.. to get your lily... etc..etc.. https://www.youtube.com/watch?v=CF7FRcjR6Rw | 20:28 |
| cloudnull | haha | 20:28 |
| cloudnull | cp: cannot stat '/usr/share/tripleo-common/container-images/kolla/iscsid/extend_start.sh': No such file or directory | 20:28 |
| weshay|ruck | yes sir | 20:29 |
| cloudnull | OK , so something missing in the base packge... | 20:33 |
| opendevreview | Kevin Carter proposed openstack/tripleo-common master: directly use files instread of tcib_gather_files https://review.opendev.org/c/openstack/tripleo-common/+/796003 | 20:36 |
| cloudnull | or the path was just wrong. | 20:36 |
| * cloudnull not my patch, just following up on it | 20:36 | |
| weshay|ruck | likely story | 20:38 |
| weshay|ruck | this is how we speak to people if we're from colorado | 20:38 |
| weshay|ruck | we all do it | 20:38 |
| weshay|ruck | we are a mean mean bunch | 20:38 |
| * cloudnull is from california, my feelings have been hugely crushed. | 20:39 | |
| * cloudnull devastated | 20:40 | |
| cloudnull | what's a cloud to do | 20:40 |
| cloudnull | 😛 | 20:40 |
| *** dviroel is now known as dviroel|out | 20:41 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Add build for ansible collections in build-test https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798641 | 20:47 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci-health-queries master: WIP: Add query "No valid host was found." https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/798958 | 20:47 |
| opendevreview | Harald Jensås proposed openstack/tripleo-heat-templates master: Fix podman on the host configuration https://review.opendev.org/c/openstack/tripleo-heat-templates/+/793871 | 20:51 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: WIP version 2 - Add build for ansible collections in build-test https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798950 | 21:03 |
| spotz | weshay|ruck: Time to rip out Stein and Queens from the docs? Didn't get a chance to ask marios earlier | 21:09 |
| weshay|ruck | lolz.. ya | 21:12 |
| weshay|ruck | good call | 21:13 |
| weshay|ruck | oh man sshnaidm is still working | 21:13 |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Handle KeyboardInterrupt with overcloud deploy https://review.opendev.org/c/openstack/python-tripleoclient/+/798836 | 21:21 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Delete network VIPs on stack delete https://review.opendev.org/c/openstack/tripleo-ansible/+/798758 | 21:21 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci-health-queries master: WIP: Add queries from Sova https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/798958 | 21:21 |
| spotz | Ok I'll start working on that patch | 21:27 |
| weshay|ruck | spotz, thanks.. I'll start removing ci jobs, pipelines, criteria... all the good stuff... fun | 21:29 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: WIP version 2 - Add build for ansible collections in build-test https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/798950 | 21:31 |
| spotz | weshay|ruck: But removing stuff is FUN!!:) | 21:32 |
| weshay|ruck | indeed.. | 21:32 |
| opendevreview | Merged openstack/tripleo-ci master: Move content providers to content-provider template and fix branches https://review.opendev.org/c/openstack/tripleo-ci/+/793144 | 21:54 |
| opendevreview | Merged openstack/tripleo-ci master: Add milestone and close message to bugs in close_bugs.py https://review.opendev.org/c/openstack/tripleo-ci/+/776246 | 21:54 |
| opendevreview | Merged openstack/tripleo-ci master: Adds tripleo-repos in the tripleo-ci-base-required-projects https://review.opendev.org/c/openstack/tripleo-ci/+/791908 | 21:56 |
| spotz | weshay|ruck: You still about? | 22:05 |
| spotz | If anyone still about knows if Wallaby is released yet for Tripleo let me know so I can add it in:) | 22:09 |
| weshay|ruck | spotz, http://lists.openstack.org/pipermail/openstack-discuss/2021-May/022276.html | 22:25 |
| opendevreview | Owen McGonagle proposed openstack/tripleo-validations master: remove using containers-prepare-parameter.yaml https://review.opendev.org/c/openstack/tripleo-validations/+/796061 | 22:25 |
| spotz | Sweet, I'll add the repos in. I'm leaving some historical references in with some slight rewording. And I suspect new features for wallaby might need to be added stiill | 22:27 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Fix ordering of Swift pipeline values https://review.opendev.org/c/openstack/tripleo-heat-templates/+/798963 | 22:45 |
| opendevreview | Steve Baker proposed openstack/diskimage-builder master: Add a growvols utility for growing LVM volumes https://review.opendev.org/c/openstack/diskimage-builder/+/791083 | 23:17 |
| opendevreview | Amy Marrich (spotz) proposed openstack/tripleo-docs master: EOL Queens and Stein. Added repos for Wallaby https://review.opendev.org/c/openstack/tripleo-docs/+/798965 | 23:29 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!