Friday, 2022-09-23

opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_sshd: update for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215000:24
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: Remove unused molecule Dockerfile.j2  https://review.opendev.org/c/openstack/tripleo-ansible/+/85900300:25
*** rlandy|rover|biab is now known as rlandy|rover00:31
*** rlandy|rover is now known as rlandy|out00:45
*** ysandeep|out is now known as ysandeep01:47
*** ysandeep is now known as ysandeep|afk03:15
*** akahat|ruck is now known as akahat03:46
*** ysandeep|afk is now known as ysandeep03:55
*** ysandeep is now known as ysandeep|afk04:32
*** chandankumar is now known as chkumar|rover04:32
*** yadnesh|away is now known as yadnesh04:58
*** ysandeep|afk is now known as ysandeep05:07
opendevreviewchandan kumar proposed openstack/tripleo-ansible stable/wallaby: We need to downgrade in the pre.yml as well  https://review.opendev.org/c/openstack/tripleo-ansible/+/85890506:05
chkumar|roverfultonj: https://review.opendev.org/c/openstack/tripleo-ansible/+/858905 will fix the issue gpg key beta issue06:05
chkumar|roverfmount_: ^^06:09
opendevreviewCedric Jeanneret proposed openstack/tripleo-quickstart-extras master: Allow console stream for standalone  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85886806:12
opendevreviewFrancesco Pantano proposed openstack/tripleo-ansible stable/wallaby: Fix typo when nova pool is built and wrong flag during cli creation  https://review.opendev.org/c/openstack/tripleo-ansible/+/85863706:14
opendevreviewFrancesco Pantano proposed openstack/tripleo-ansible stable/wallaby: Fix typo when nova pool is built and wrong flag during cli creation  https://review.opendev.org/c/openstack/tripleo-ansible/+/85863706:15
opendevreviewFrancesco Pantano proposed openstack/tripleo-ansible stable/wallaby: Fix typo when nova pool is built and wrong flag during cli creation  https://review.opendev.org/c/openstack/tripleo-ansible/+/85863706:16
opendevreviewRafal Lewandowski proposed openstack/diskimage-builder master: Added cloud-init growpart element  https://review.opendev.org/c/openstack/diskimage-builder/+/85585606:24
*** amoralej|off is now known as amoralej06:34
fmount_chkumar|rover: ack thanks, just rebased my tripleo-ansible patch on top of yours06:35
chkumar|rovermolecule jobs passed06:35
chkumar|roverfmount_: fpantano thank you for reporting it :-)06:35
chkumar|rovermarios: Tengu https://review.opendev.org/c/openstack/tripleo-ansible/+/858905 and https://review.opendev.org/c/openstack/tripleo-ansible/+/859003 please add in your review queue, thank you :-)06:37
Tenguchkumar|rover: uho, needed a backport?06:38
Tenguchkumar|rover: for the dockerfile removal, would it be possible to get a test run? I'd rather be cautious.06:41
shyambHi06:51
shyambAnybody expert on this active passive cinder volume heat template structure?06:52
shyambhttps://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/cinder/cinder-volume-pacemaker-puppet.yaml06:52
opendevreviewJiri Podivin proposed openstack/tripleo-heat-templates master: Removing unused imports from requirements  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85894106:52
shyambIf somebody wants to add a new service to pacemaker with active passive mode, how can he do it with minimal steps?06:52
shyambI am looking to add this trilio service as pacemaker resource. https://github.com/shyam-biradar/triliovault-cfg-scripts/blob/dev-stable/5.0/redhat-director-scripts/rhosp16/services/triliovault-wlm-cron.yaml06:56
mariosack chkumar|rover 06:58
*** frenzyfriday is now known as frenzyfriday|ruck07:01
chkumar|roverTengu: will do that , thank you :-)07:03
Tenguchkumar|rover: thanks!07:04
Tengusince there's no molecule run on that patch...07:05
Tengufolks, care to nudge that one into the gate? https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85196207:07
*** yadnesh is now known as yadnesh|afk07:07
opendevreviewJiri Podivin proposed openstack/tripleo-heat-templates master: Refactoring tools as per FIXME comments  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85894807:15
opendevreviewPooja Jadhav proposed openstack/tripleo-ci master: Victoria tear down : multinode and undercloud jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/85393107:23
*** jpena|off is now known as jpena07:26
opendevreviewJuan Larriba proposed openstack/python-tripleoclient stable/wallaby: Support remove and revert actions  https://review.opendev.org/c/openstack/python-tripleoclient/+/85890607:30
opendevreviewPooja Jadhav proposed openstack/tripleo-ci master: Victoria tear down : content provider  https://review.opendev.org/c/openstack/tripleo-ci/+/85606107:44
opendevreviewyatin proposed openstack/openstack-tempest-skiplist master: Remove skips for bug 1793482  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85902708:25
opendevreviewMarios Andreou proposed openstack/tripleo-ci master: Add exclude_containers for zed container build  https://review.opendev.org/c/openstack/tripleo-ci/+/85809208:29
opendevreviewJiri Podivin proposed openstack/tripleo-quickstart-extras master: Completely removing the atop installation task from the undercloud setup  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/83675408:31
*** ysandeep is now known as ysandeep|lunch08:44
opendevreviewJiri Podivin proposed openstack/tripleo-common master: Adding wsgi script  https://review.opendev.org/c/openstack/tripleo-common/+/85905909:25
opendevreviewJiri Podivin proposed openstack/tripleo-heat-templates master: Refactoring tools as per FIXME comments  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85894809:30
opendevreviewJiri Podivin proposed openstack/tripleo-heat-templates master: Ansible configuration of gnocchi httpd  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85888209:30
opendevreviewFernando Díaz Bravo proposed openstack/tripleo-ansible master: bnr - Update on rear scripts to support the ceph bnr on LVM  https://review.opendev.org/c/openstack/tripleo-ansible/+/85852009:35
*** ysandeep|lunch is now known as ysandeep09:50
opendevreviewJiri Podivin proposed openstack/tripleo-common master: Adding wsgi script  https://review.opendev.org/c/openstack/tripleo-common/+/85905910:17
opendevreviewchandan kumar proposed openstack/tripleo-ci master: Drop ANSIBLE_RUNNER_IMAGE from tcib_extra_config  https://review.opendev.org/c/openstack/tripleo-ci/+/85907110:30
*** rlandy|out is now known as rlandy10:38
*** yadnesh|afk is now known as yadnesh10:40
opendevreviewchandan kumar proposed openstack/tripleo-quickstart-extras master: Drop ANSIBLE_RUNNER_IMAGE from tcib_extra_config  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85907210:40
opendevreviewchandan kumar proposed openstack/openstack-tempest-skiplist master: Skip test_create_update_port_with_dns_domain test  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85907410:56
opendevreviewchandan kumar proposed openstack/openstack-tempest-skiplist master: Skip test_create_update_port_with_dns_domain test  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85907410:57
opendevreviewchandan kumar proposed openstack/tripleo-ansible master: [DNM] dockerfile.js removal  https://review.opendev.org/c/openstack/tripleo-ansible/+/85907611:03
opendevreviewchandan kumar proposed openstack/openstack-tempest-skiplist master: Skip test_create_update_port_with_dns_domain test  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85907411:10
opendevreviewMerged openstack/openstack-tempest-skiplist master: Skip test_create_update_port_with_dns_domain test  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85907411:28
opendevreviewSofer Athlan-Guyot proposed openstack/tripleo-heat-templates stable/train: Update, avoid task skipping by directly importing step file.  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/79651811:31
opendevreviewCedric Jeanneret proposed openstack/tripleo-ansible master: We don't need the final drop rule anymore  https://review.opendev.org/c/openstack/tripleo-ansible/+/85328211:32
opendevreviewPooja Jadhav proposed openstack/tripleo-ci master: Victoria tear down : build image/container jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/85763111:32
chkumar|roverTengu: tested here https://review.opendev.org/c/openstack/tripleo-ansible/+/859003/1#message-5446c6b8a46318090c976eb27661a00b225cabfe good to merge it11:48
*** bogdando_ is now known as bogdando11:48
Tengu++11:49
chkumar|roverslagle: for pkg_extras: python*setuptools let me clean this up and move this part to test_deps role11:51
slaglechkumar|rover: ok, that would work11:51
slaglefor some of the other re-use, i'm starting to think we might need another layer on top of molecule11:52
opendevreviewSoniya Murlidhar Vyas proposed openstack/tripleo-quickstart master: Run featureset066 job with tempest-allow  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85284511:52
slagleso that we can reuse the platforms, and customize it without having to copy the whole thing11:52
slaglechkumar|rover: as in here: https://review.opendev.org/c/openstack/tripleo-ansible/+/84215011:53
slaglehave to copy it into every molecule.yml, and copy the whole dockerfile as well11:53
slaglemight need to patch molecule-podman to allow reading a custom dockerfile11:53
slagle*from a common location11:54
slaglemolecule is very half-baked. the use case seems to be "i wrote 1 role and want to test it"11:55
slagledoesn't feel like a test framework. but we could start trying to improve it i guess11:55
opendevreviewJiri Podivin proposed openstack/tripleo-quickstart-extras master: Completely removing the atop installation task from the undercloud setup  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/83675411:58
odyssey4meslagle seems like an old, and common problem - https://github.com/ansible-community/molecule/issues/94211:59
opendevreviewMerged openstack/tripleo-ci master: Remove old ansible versions from zuul jobs  https://review.opendev.org/c/openstack/tripleo-ci/+/85876812:13
opendevreviewMerged openstack/tripleo-ansible master: Mark B&R job as voting  https://review.opendev.org/c/openstack/tripleo-ansible/+/85850712:13
*** frenzyfriday|ruck is now known as frenzyfriday|ruck|food12:26
*** amoralej is now known as amoralej|lunch12:30
owalshslagle: hey, if you are around could you take a look at https://review.opendev.org/c/openstack/tripleo-ansible/+/855975, need to get it backported/downstream12:35
slagleowalsh: ok. was there a bug for this? ifup-ovs should start it.12:37
owalshslagle: https://bugzilla.redhat.com/show_bug.cgi?id=212447312:39
slagleowalsh: https://review.opendev.org/c/openstack/tripleo-ansible/+/855975/9/tripleo_ansible/roles/tripleo_bootstrap/tasks/bootstrap.yml#58 is the task name right there?12:40
owalshslagle: you mean s/Enable/Start and enable/12:41
slagleowalsh: yea i guess. is it not installed by that point?12:42
owalshslagle: https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_bootstrap/tasks/packages.yml#L5112:42
owalshthat's the ceph nodes issue tkajinam was concerned about12:43
slaglethe dpdk node doesn't if up anything ovs?12:45
slagleifup12:45
slaglethis is the exact sort of thing that everyone gripes about tripleo. so before we add more stuff just to do it and across the board, wondering why it's actually needed.12:46
slagleand it'd be good to comment to that effect12:46
slaglecuz this is also the exact type of thing someone will "clean" up later trying to address said gripes12:46
slagle:)12:46
owalshwell with an DIB image the service is enabled so it's started at boot12:48
owalshand in CI it's already enabled before deployment12:49
owalshso this is to get pre-deployed nodes in the same state12:49
owalshturns out nobody is really testing DPDK with pre-deployed12:49
slagleright. just wondering why it's needed12:50
chkumar|roverslagle: let me come up with some pre playbook which will generate the config if custom setting we need to added in the platform section12:50
slagleowalsh: trying to run ovs-vsctl before the ovs interfaces are up? or, they aren't meant to be up. i guess that's the reason?12:51
owalshslagle: I'm guessing it's puppet-vswitch https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/neutron/neutron-ovs-dpdk-agent-container-puppet.yaml#L13512:53
*** yadnesh is now known as yadnesh|away12:56
opendevreviewBogdan Dobrelya proposed openstack/tripleo-common master: Add TRIPLEO_KOLLA_UMASK for custom umask settings  https://review.opendev.org/c/openstack/tripleo-common/+/85893013:01
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates master: Run virtqemud with umask 0027  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85893113:01
opendevreviewBogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Nova: Start nova-compute after its NFS backend is ready  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85000013:06
opendevreviewJuan Larriba proposed openstack/tripleo-ansible stable/wallaby: Mark B&R job as voting  https://review.opendev.org/c/openstack/tripleo-ansible/+/85890713:10
*** frenzyfriday|ruck|food is now known as frenzyfriday|ruck13:11
*** amoralej|lunch is now known as amoralej13:18
chkumar|roverbogdando: please revisit this series https://review.opendev.org/q/topic:tripleo-ansible-ee when you get some time, thank you :-)13:20
opendevreviewVeronika Fisarova proposed openstack/tripleo-validations master: Check the time synchronization and list the NTP servers chrony is trying to access  https://review.opendev.org/c/openstack/tripleo-validations/+/85851013:28
opendevreviewBogdan Dobrelya proposed openstack/tripleo-common stable/wallaby: Update ovmf-amdsev from ovmf-cc to boot UEFI VMs  https://review.opendev.org/c/openstack/tripleo-common/+/85514213:28
opendevreviewJiri Podivin proposed openstack/tripleo-heat-templates master: Implementing basic pre-commit hooks for local testing  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85908513:30
chkumar|roverTengu: https://review.opendev.org/c/openstack/tripleo-ansible/+/858508 this also, thank you :-)13:31
*** pojadhav is now known as pojadhav|afk13:31
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: We need to downgrade in the pre.yml as well  https://review.opendev.org/c/openstack/tripleo-ansible/+/85890513:32
opendevreviewMerged openstack/tripleo-ansible master: Remove unused molecule Dockerfile.j2  https://review.opendev.org/c/openstack/tripleo-ansible/+/85900313:32
Tenguchkumar|rover: sure :)13:32
Tenguchkumar|rover: care to check https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/858868 ? :)13:33
Tenguthat's a "nice to have" for the coming nftables switch. We're ready for next week!13:33
Tengu\o/13:35
opendevreviewJohn Fulton proposed openstack/python-tripleoclient master: get_overcloud_hosts() should not return a list with empty strings  https://review.opendev.org/c/openstack/python-tripleoclient/+/85897713:40
opendevreviewAlfredo Moralejo proposed openstack/tripleo-quickstart master: [DNM] Add temporary repo with ansible-collections-openstack 2.0.0 preview  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85909013:45
opendevreviewAlfredo Moralejo proposed openstack/tripleo-quickstart master: [DNM] Add temporary repo with ansible-collections-openstack 2.0.0 preview  https://review.opendev.org/c/openstack/tripleo-quickstart/+/85909013:46
opendevreviewJiri Podivin proposed openstack/tripleo-common master: Adding wsgi script  https://review.opendev.org/c/openstack/tripleo-common/+/85905914:02
*** dasm|off is now known as dasm14:02
*** ysandeep is now known as ysandeep|out14:02
owalshslagle: either way I'm just making the pre-deployed nodes consistent with DIB. if the OVS logic is bad/wrong then it's wrong for both and should be fixed in a follow up patch by somebody more familiar with the logic14:08
*** pojadhav|afk is now known as pojadhav14:23
opendevreviewyatin proposed openstack/openstack-tempest-skiplist master: Remove skips for bug 1793482  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85902714:51
ykarelrlandy, chkumar|rover please check ^ when you get chance14:53
opendevreviewDamien Ciabrini proposed openstack/tripleo-heat-templates master: WIP HA: minor update with new container image name  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85911014:53
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: Switch cinder-backup default to A/A  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85761415:00
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: Switch cinder-backup default to active-active  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85761415:06
bogdandochkumar|rover: did we land some changes to gating repo management so far?15:16
bogdandofor some reason, zuul reproducer no longer works for me15:16
bogdandorepo setup fails...15:16
bogdandono /opt/gate_repo on subnodes, it wasn't injected it seems15:17
bogdandoand not gate.repo files15:17
*** amoralej is now known as amoralej|off15:37
*** marios is now known as marios|out15:46
cloudnullhappy Friday :D 15:47
cloudnull👋15:48
opendevreviewManojkatari proposed openstack/tripleo-heat-templates master: new tht for Cinder RBD backend  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85707115:52
opendevreviewMerged openstack/openstack-tempest-skiplist master: Remove skips for bug 1793482  https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/85902716:20
*** jpena is now known as jpena|off16:22
fultonjhey cloudnull 16:57
opendevreviewJohn Fulton proposed openstack/tripleo-docs master: Document using tripleo_ceph_client role with standalone Ansible  https://review.opendev.org/c/openstack/tripleo-docs/+/85914216:57
*** rlandy is now known as rlandy|brb16:57
beagleso/ cloudnull17:15
opendevreviewMerged openstack/tripleo-common master: Drop ANSIBLE_RUNNER_IMAGE from tcib_args  https://review.opendev.org/c/openstack/tripleo-common/+/85886417:15
opendevreviewMerged openstack/tripleo-ansible master: Container build: define ARGs after FROM line  https://review.opendev.org/c/openstack/tripleo-ansible/+/85850817:15
opendevreviewMerged openstack/tripleo-quickstart-extras master: Allow console stream for standalone  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85886817:15
*** rlandy|brb is now known as rlandy17:20
opendevreviewJames Slagle proposed openstack/tripleo-ansible master: tripleo_sshd: update for standalone roles  https://review.opendev.org/c/openstack/tripleo-ansible/+/84215017:50
opendevreviewMerged openstack/tripleo-ansible stable/wallaby: Fix typo when nova pool is built and wrong flag during cli creation  https://review.opendev.org/c/openstack/tripleo-ansible/+/85863718:09
opendevreviewMerged openstack/tripleo-ci master: Drop ANSIBLE_RUNNER_IMAGE from tcib_extra_config  https://review.opendev.org/c/openstack/tripleo-ci/+/85907118:09
opendevreviewMerged openstack/tripleo-quickstart-extras master: Drop ANSIBLE_RUNNER_IMAGE from tcib_extra_config  https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/85907218:09
opendevreviewBrent Eagles proposed openstack/tripleo-heat-templates master: Designate: format listen addr properly when ipv6  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85773918:41
opendevreviewMerged openstack/tripleo-heat-templates master: new tht for Cinder NFS backend  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85439818:43
opendevreviewJohn Fulton proposed openstack/tripleo-docs master: Document using tripleo_ceph_client role with standalone Ansible  https://review.opendev.org/c/openstack/tripleo-docs/+/85914220:14
*** dasm is now known as dasm|off21:10
opendevreviewJohn Fulton proposed openstack/tripleo-docs master: Document using tripleo_ceph_client role with standalone Ansible  https://review.opendev.org/c/openstack/tripleo-docs/+/85914221:29
opendevreviewJohn Fulton proposed openstack/tripleo-ansible master: Support standalone variable names in tripleo_ceph_client  https://review.opendev.org/c/openstack/tripleo-ansible/+/85914921:40
opendevreviewMerged openstack/python-tripleoclient master: get_overcloud_hosts() should not return a list with empty strings  https://review.opendev.org/c/openstack/python-tripleoclient/+/85897723:09
opendevreviewMerged openstack/tripleo-heat-templates master: Switch cinder-backup default to active-active  https://review.opendev.org/c/openstack/tripleo-heat-templates/+/85761423:38

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