Tuesday, 2022-08-16

opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates stable/wallaby: Disable [oslo_messaging_rabbit] heartbeat_in_pthread  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85271300:00
opendevreviewBrendan Shephard proposed openstack/tripleo-common master: Add multi-rhel support for Nova services  https://review.opendev.org/c/openstack/tripleo-common/+/85321100:34
opendevreviewBrendan Shephard proposed openstack/tripleo-common master: Add multi-rhel support for Nova services  https://review.opendev.org/c/openstack/tripleo-common/+/85321101:13
opendevreviewBrendan Shephard proposed openstack/tripleo-common master: Add multi-rhel support for Storage services  https://review.opendev.org/c/openstack/tripleo-common/+/85320701:26
opendevreviewBrendan Shephard proposed openstack/tripleo-quickstart master: Default master to el9  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85307401:27
opendevreviewBrendan Shephard proposed openstack/tripleo-common master: Add multi-rhel support for Storage services  https://review.opendev.org/c/openstack/tripleo-common/+/85320701:40
opendevreviewBrendan Shephard proposed openstack/tripleo-quickstart master: Default master to el9  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85307404:19
*** ysandeep|holiday is now known as ysandeep05:28
*** ysandeep is now known as ysandeep|ruck05:29
opendevreviewPranali Deore proposed openstack/puppet-tripleo master: Glance: Include the required glance::limit class for unified quotas  https://review.opendev.org/c/openstack/puppet-tripleo/+/84945805:55
opendevreviewCedric Jeanneret proposed openstack/tripleo-puppet-elements master: Add nftables to the OC images  https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/85322406:04
Tengutkajinam: -^^06:07
Tengutkajinam: would be good to get either solutions (tripleo-puppet-elements or https://review.opendev.org/c/openstack/tripleo-common/+/852838) merged asap, so that we can move on with the OC testing :)06:07
tkajinamTengu, thanks. that looks good to me. I will vote +2 once we get CI result.06:10
Tenguthanks :)06:10
Tengulemme abandon the other one then.06:10
*** amoralej|off is now known as amoralej06:18
Tengufolks, care to have a look at https://review.opendev.org/c/openstack/ansible-role-redhat-subscription/+/852846 ? tkajinam, bshephar maybe? :)06:34
opendevreviewravi singh proposed openstack/tripleo-validations master: Change hosts for nova-svirt.yaml playbook to nova_libvirt  https://review.opendev.org/c/openstack/tripleo-validations/+/85322706:37
tkajinamTengu, checking06:37
Tengutkajinam: thanks :)06:37
Tenguah, bshephar was faster :)06:38
Tenguthanks :)06:38
bshepharTengu: Bit annoying we need to set it to an empty string though.06:38
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: Switch default firewall engine  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85280806:38
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: Switch default firewall engine  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85280806:38
Tenguysandeep|ruck: fyi: updated Depends-On + rebased against master -^^06:38
* tkajinam was downloading the change to do grep06:38
tkajinamX-(06:39
ysandeep|ruckack06:39
bshephartkajinam: I can remove my +w if you like06:39
bshepharLooks fine to me though. Looking at the candlepin patch. 06:39
tkajinambshephar, voted my late +2 :-)06:40
* Tengu tried to put all the needed info in the commit message06:40
opendevreviewMerged openstack/ansible-role-redhat-subscription master: Update rhsm_disable_container_check default value  https://review.opendev.org/c/openstack/ansible-role-redhat-subscription/+/85284606:46
Tenguperfect.06:49
Tenguhmmm iirc I'll have to take it down to wallaby.06:50
Tenguah. no. no branch. good.06:50
opendevreviewCyril Lopez proposed openstack/tripleo-heat-templates master: Fix wrong parameter assignation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85277307:15
opendevreviewravi singh proposed openstack/tripleo-validations master: Run nova-svirt validation in nodes with nova_libvirt service  https://review.opendev.org/c/openstack/tripleo-validations/+/85322707:17
opendevreviewravi singh proposed openstack/tripleo-validations master: Run nova-svirt validation in nodes with nova_libvirt service  https://review.opendev.org/c/openstack/tripleo-validations/+/85322707:24
opendevreviewravi singh proposed openstack/tripleo-validations master: Run nova-svirt validation in nodes with nova_libvirt service  https://review.opendev.org/c/openstack/tripleo-validations/+/85322707:25
opendevreviewravi singh proposed openstack/tripleo-validations master: Run nova-svirt validation in nodes with nova_libvirt service  https://review.opendev.org/c/openstack/tripleo-validations/+/85322707:33
*** jpena|off is now known as jpena07:34
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: Use the new tripleo_auditd ansible role instead of puppet  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85195807:36
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Ensure nftables is installed on the UC node  https://review.opendev.org/c/openstack/tripleo-ansible/+/85325207:56
*** ysandeep|ruck is now known as ysandeep|ruck|lunch08:05
opendevreviewPranali Deore proposed openstack/puppet-tripleo master: Glance: Use glance::limit class to enable unified quotas  https://review.opendev.org/c/openstack/puppet-tripleo/+/84945808:16
opendevreviewravi singh proposed openstack/tripleo-validations master: Run nova-svirt validation in nodes with nova_libvirt service  https://review.opendev.org/c/openstack/tripleo-validations/+/85322708:22
*** ysandeep|ruck|lunch is now known as ysandeep|ruck08:28
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove stdout logging support (9): Rsyslog sidecar  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84586109:08
opendevreviewMilana Levy proposed openstack/tripleo-validations master: Implement validation for manila config  https://review.opendev.org/c/openstack/tripleo-validations/+/85238909:22
opendevreviewDamien Ciabrini proposed openstack/tripleo-heat-templates stable/train: Galera: initial support for mariabackup SST  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85100009:34
opendevreviewMarios Andreou proposed openstack/tripleo-quickstart-extras master: Adds dlrn_hash resolution tripleo-get-hash for periodic mixed rhel  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85316710:28
*** ysandeep|ruck is now known as ysandeep|ruck|afk10:46
opendevreviewMerged openstack/tripleo-ansible master: Preserve action: accept in tripleo_iptables  https://review.opendev.org/c/openstack/tripleo-ansible/+/85148211:15
opendevreviewMerged openstack/tripleo-quickstart master: [FIPS] Exclude gnutls 3.7.6 which is failing on fips  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85291411:15
opendevreviewMerged openstack/tripleo-puppet-elements master: Add nftables to the OC images  https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/85322411:15
*** ysandeep|ruck|afk is now known as ysandeep|ruck11:17
Tenguw00t!11:19
Tenguysandeep|ruck, marios -^^  merged!11:19
Tenguwe're getting closer!11:19
mariosthanks Tengu o/ 11:19
Tenguysandeep|ruck: I think we need to get the custom VXLAN rules for the OC nodes as well: https://logserver.rdoproject.org/54/31954/66/check/periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001-master/f79e0ef/logs/overcloud-controller-0/var/log/extra/dropped-packets.txt.gz  iirc that UDP/68 is related to vxlan, isn't it?11:20
opendevreviewPranali Deore proposed openstack/puppet-tripleo master: Glance: Use glance::limit class to enable unified quotas  https://review.opendev.org/c/openstack/puppet-tripleo/+/84945811:21
Tenguhmmm nope. bootp client11:21
Tenguwondering if this is really something we need.11:22
opendevreviewMikolaj Ciecierski proposed openstack/tripleo-common master: Update UNDERCLOUD_SERVICES list to reflect current undercloud services  https://review.opendev.org/c/openstack/tripleo-common/+/85327211:28
opendevreviewMikolaj Ciecierski proposed openstack/tripleo-common master: Update UNDERCLOUD_SERVICES list to reflect current undercloud services  https://review.opendev.org/c/openstack/tripleo-common/+/85327211:36
*** dviroel_ is now known as dviroel11:38
opendevreviewManojkatari proposed openstack/tripleo-ansible master: Add tripleo_etcd ansible role  https://review.opendev.org/c/openstack/tripleo-ansible/+/84902411:49
opendevreviewFrancesco Pantano proposed openstack/tripleo-heat-templates master: DNM - QUINCY - Test Pending Ceph daemons in CI  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/77891511:51
opendevreviewTakashi Kajinami proposed openstack/tripleo-common master: Remove ceilomter-common from swift images  https://review.opendev.org/c/openstack/tripleo-common/+/85327811:54
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: Ensure nftables is installed on the nodes  https://review.opendev.org/c/openstack/tripleo-ansible/+/85325211:55
opendevreviewTakashi Kajinami proposed openstack/tripleo-quickstart master: master/wallaby: Test against the latest puppet-octavia  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85248111:57
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates master: Fix reversed is_set conditions for nova-compute  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/84733412:09
opendevreviewTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove redundant Tmpwatch service  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85328012:13
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: Fix scenario path in config detector  https://review.opendev.org/c/openstack/tripleo-ansible/+/85277612:16
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: We don't need the final drop rule anymore  https://review.opendev.org/c/openstack/tripleo-ansible/+/85328212:17
Tenguysandeep|ruck: -^^12:17
* Tengu loves the depends-on12:17
Tenguysandeep|ruck: do you have a testproject running against https://review.opendev.org/c/openstack/tripleo-heat-templates/+/852808 with sc001?12:18
* Tengu wants to add a depends-on, but that would crash the run12:18
ysandeep|ruckTengu, not yet12:19
Tenguysandeep|ruck: good!12:19
TenguI'll add the depends-on tripleo-ansible/tripleo_bootstrap12:19
opendevreviewCedric Jeanneret proposed openstack/tripleo-heat-templates master: Switch default firewall engine  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85280812:19
Tengugood.12:19
chandankumarTengu: good to merge https://review.opendev.org/c/openstack/tripleo-ansible/+/852884 this, thanks :-)12:20
Tenguchandankumar: indeed :)12:21
opendevreviewMerged openstack/tripleo-heat-templates master: Fix wrong parameter assignation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85277312:25
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: Fix scenario path in config detector  https://review.opendev.org/c/openstack/tripleo-ansible/+/85277612:25
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: [DNM] mol job testing  https://review.opendev.org/c/openstack/tripleo-ansible/+/85294112:30
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: Fix scenario path in config detector  https://review.opendev.org/c/openstack/tripleo-ansible/+/85277612:45
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: Fix scenario path in config detector  https://review.opendev.org/c/openstack/tripleo-ansible/+/85277612:54
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: [DNM] mol job testing  https://review.opendev.org/c/openstack/tripleo-ansible/+/85294112:55
*** amoralej is now known as amoralej|lunch13:00
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: tripleo_auditd: Clear unused default of tripleo_auditd_rules  https://review.opendev.org/c/openstack/tripleo-ansible/+/85329713:27
*** jgilaber_ is now known as jgilaber13:30
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: New role, replacing puppet-auditd  https://review.opendev.org/c/openstack/tripleo-ansible/+/85258013:33
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible stable/wallaby: tripleo_auditd: Clear unused default of tripleo_auditd_rules  https://review.opendev.org/c/openstack/tripleo-ansible/+/85329713:33
Tenguslagle: -^^  now it should be better :)13:34
Tenguand thanks for the +W on the master version of tkajinam's patch!13:34
opendevreviewCyril Lopez proposed openstack/tripleo-heat-templates stable/train: Fix wrong parameter assignation  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85329913:35
Tengutkajinam: dang, you were faster than me on that one!! -^13:37
tkajinamTengu, ;-)13:37
Tenguwas probably manually cherry-picked with `git review -X -i'13:38
tkajinamyeah13:42
opendevreviewMerged openstack/tripleo-ansible master: Fix missing ansible paths in shared molecule configs  https://review.opendev.org/c/openstack/tripleo-ansible/+/85288413:50
*** amoralej|lunch is now known as amoralej13:54
*** dasm|off is now known as dasm13:55
tkajinamMay I ask some cores to review https://review.opendev.org/c/openstack/tripleo-heat-templates/+/852579 ?14:32
tkajinamwe need to backport this to stable/wallaby (backport was already proposed for early testing) because the described bug of oslo.messaging affects stable/wallaby as well14:32
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: Fix scenario path in config detector  https://review.opendev.org/c/openstack/tripleo-ansible/+/85277614:43
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215214:43
chandankumarTengu: https://review.opendev.org/c/openstack/tripleo-ansible/+/852776 is also good to go :-)14:48
Tenguchandankumar: ok - was waiting for the resolution of your comment :)14:49
Tengu+2, waiting for CI.14:49
chandankumarok thank you :-)14:50
Tenguchandankumar, ysandeep|ruck, slagle, bogdando care to have a look at https://review.opendev.org/c/openstack/tripleo-ansible/+/853252 ? It's needed in order to ensure nftables is present, even if we're using pre-provisioned nodes and the like.14:50
bogdandoLGTM14:50
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215214:51
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84365914:51
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node  https://review.opendev.org/c/openstack/tripleo-ansible/+/84050914:53
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node  https://review.opendev.org/c/openstack/tripleo-ansible/+/84050914:56
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: mol  https://review.opendev.org/c/openstack/tripleo-ansible/+/85331214:57
opendevreviewBogdan Dobrelya proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215214:57
*** amoralej is now known as amoralej|off15:01
*** ysandeep|ruck is now known as ysandeep|dinner15:11
opendevreviewJaganathan Palanisamy proposed openstack/python-tripleoclient master: Derive parameters clean up  https://review.opendev.org/c/openstack/python-tripleoclient/+/85123615:19
*** marios is now known as marios|out15:29
opendevreviewMartin Schuppert proposed openstack/tripleo-heat-templates stable/wallaby: Add ipv6 support in environments/deployed-network-environment.j2.yaml  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85324715:33
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: Ship inventory as a part of tripleo_ansible package  https://review.opendev.org/c/openstack/tripleo-ansible/+/84792915:41
chandankumarslagle: hello15:47
slaglechandankumar: hi15:47
chandankumarslagle: https://review.opendev.org/c/openstack/tripleo-ansible/+/840509 is hitting invalid configuration error, it might be issue with depends on15:47
chandankumarwhen you get time, please have a look15:47
chandankumarI am testing it here: https://review.opendev.org/c/openstack/tripleo-ansible/+/849604/1 with multiple depends on15:48
chandankumarthank you :-)15:48
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84365915:52
slaglechandankumar: that might fix it ^. i did a recheck on 84050915:53
*** ysandeep|dinner is now known as ysandeep15:54
*** ysandeep is now known as ysandeep|out16:00
chandankumarslagle: thank you, I think we need to rebase https://review.opendev.org/c/openstack/tripleo-ansible/+/840509/ it, still complaining Error merging gerrit/openstack/tripleo-ansible for 843659,2716:15
slaglechandankumar: a rebase won't fix it. it's that 2 of the depends-on cause a conflict16:30
slaglesince they both modify zuul.d/molecule.yaml at the same line #16:30
slaglei can try and put those in the same tree16:31
chandankumarslagle: thank you :-)16:32
*** jpena is now known as jpena|off16:33
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215216:39
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84365916:39
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node  https://review.opendev.org/c/openstack/tripleo-ansible/+/84050916:39
opendevreviewMilana Levy proposed openstack/tripleo-validations master: Implement validation for manila config  https://review.opendev.org/c/openstack/tripleo-validations/+/85238916:40
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84365916:46
slaglechandankumar: i think that fixed it. at least the zuul configuration error17:20
opendevreviewamolkahat proposed openstack/tripleo-quickstart master: Set dlrn_hash_tag to current-tripleo if mixed os job  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85273317:35
*** mburns is now known as mburns_biab17:46
*** mburns_biab is now known as mburns18:41
*** dviroel is now known as dviroel|brb20:02
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Update sensubility's container health check  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85092920:14
opendevreviewMerged openstack/tripleo-heat-templates master: Use the new tripleo_auditd ansible role instead of puppet  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85195820:19
opendevreviewMerged openstack/tripleo-heat-templates master: Disable [oslo_messaging_rabbit] heartbeat_in_pthread  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85257920:37
opendevreviewMerged openstack/tripleo-heat-templates master: nova-migration-target: Re-enable sftp subsystem  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85027320:37
opendevreviewMerged openstack/tripleo-ansible master: Fix scenario path in config detector  https://review.opendev.org/c/openstack/tripleo-ansible/+/85277620:37
*** dviroel|brb is now known as dviroel21:15
*** dviroel is now known as dviroel|afk21:40
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Make libvirt-guests service work with deprecated monolithic libvirt  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85295721:58
fultonjShould a compute node deployed by standalone roles [1] be able to boot a VM using OVN?22:13
fultonjMine is Refusing to bind port fd961446-73e2-4cb2-af9e-4c227030d564 due to no OVN chassis for host [2]22:13
fultonj[1] https://etherpad.opendev.org/p/tripleo-standalone-roles22:14
fultonj[2] https://paste.opendev.org/show/b3q6EVwcQC4QLGh5qlr1/22:14
opendevreviewMerged openstack/tripleo-heat-templates stable/wallaby: Set host_routes in environments/deployed-network-environment.j2.yaml  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85309322:23
opendevreviewMerged openstack/tripleo-common master: Remove sysfsutils from nova container  https://review.opendev.org/c/openstack/tripleo-common/+/79007022:23
*** dasm is now known as dasm|off22:26
opendevreviewBrendan Shephard proposed openstack/tripleo-common master: Add multi-rhel support for Storage services  https://review.opendev.org/c/openstack/tripleo-common/+/85320722:41
bshepharfultonj: Do you have this one? https://review.opendev.org/c/openstack/tripleo-ansible/+/84565323:09
opendevreviewTakashi Kajinami proposed openstack/puppet-tripleo master: Remove manifest for auditd service  https://review.opendev.org/c/openstack/puppet-tripleo/+/85259823:17

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