| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 00:03 |
|---|---|---|
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 00:04 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 00:09 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 00:16 |
| *** rlandy is now known as rlandy|out | 00:18 | |
| opendevreview | Merged openstack/tripleo-common stable/train: Add excluded group to inventory https://review.opendev.org/c/openstack/tripleo-common/+/859403 | 00:30 |
| opendevreview | Merged openstack/validations-libs stable/1.6: Adding yamllint configuration file and fixing style violations https://review.opendev.org/c/openstack/validations-libs/+/865677 | 00:30 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deploy glance-api-internal services in GlanceApiInternal network https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866415 | 02:03 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deploy glance-api-internal services in GlanceApiInternal network https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866415 | 02:04 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: glance-api: Allow overriding bind_host https://review.opendev.org/c/openstack/puppet-tripleo/+/866416 | 02:09 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Deploy glance-api-internal services in GlanceApiInternal network https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866415 | 02:09 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-quickstart-extras master: Try to recover controller inventory for mixed os deployment. https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/866431 | 02:53 |
| opendevreview | Sofer Athlan-Guyot proposed openstack/tripleo-quickstart master: Add featureset068 for multi-rhel update. https://review.opendev.org/c/openstack/tripleo-quickstart/+/866131 | 02:56 |
| opendevreview | Merged openstack/tripleo-validations stable/wallaby: Removing undercloud-tokenflush validation and related role https://review.opendev.org/c/openstack/tripleo-validations/+/864560 | 03:41 |
| opendevreview | Merged openstack/os-net-config master: remove unicode from code https://review.opendev.org/c/openstack/os-net-config/+/854489 | 03:41 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Add service options to only undercloud.conf https://review.opendev.org/c/openstack/python-tripleoclient/+/866302 | 04:24 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: Add service options to only undercloud.conf https://review.opendev.org/c/openstack/python-tripleoclient/+/866302 | 04:27 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: undercloud: Remove deprecated enable_<service> options https://review.opendev.org/c/openstack/python-tripleoclient/+/866433 | 04:35 |
| *** yadnesh|away is now known as yadnesh | 05:01 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Use ansible variable to detect bootstrap node https://review.opendev.org/c/openstack/tripleo-heat-templates/+/857714 | 05:06 |
| *** marios is now known as marios|ruck | 06:14 | |
| shyam2389 | Hi | 06:40 |
| shyam2389 | Anybody expert on HA resources in tripleo? | 06:40 |
| shyam2389 | https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/cinder/cinder-volume-container-puppet.yaml | 06:40 |
| shyam2389 | Is this call updating cinder pacemaker resource: https://github.com/openstack/tripleo-heat-templates/blob/stable/train/deployment/cinder/cinder-volume-pacemaker-puppet.yaml#L213 | 06:42 |
| opendevreview | Takashi Kajinami proposed openstack/python-tripleoclient master: undercloud: Remove deprecated enable_<service> options https://review.opendev.org/c/openstack/python-tripleoclient/+/866433 | 06:47 |
| *** ysandeep is now known as ysandeep|brb | 07:12 | |
| lmiccini | o/ anyone around that could have a look at https://review.opendev.org/c/openstack/puppet-tripleo/+/863041 please? | 07:20 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-ansible stable/zed: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-ansible/+/866450 | 07:21 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-ansible stable/wallaby: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-ansible/+/866451 | 07:21 |
| tkajinam | lmiccini, I've left a comment | 07:34 |
| lmiccini | tkajinam: we can only have 1 single resource across the cluster, we can't have the usual resource_name-mac-address | 07:35 |
| lmiccini | tkajinam: thank you btw :) | 07:35 |
| tkajinam | hm | 07:36 |
| lmiccini | look at the comment I left yesterday for an example of what it should look like | 07:36 |
| tkajinam | I'm wondering how FencingConfig should look like. If the resource is cluster-wide then having host_mac fields sounds a bit complicated | 07:36 |
| * tkajinam is now looking at the comment and found the example | 07:37 | |
| lmiccini | tkajinam: I intentionally left most of the logic as it is for the other resources to avoid deviating too much from what is the usual flow | 07:38 |
| lmiccini | unfortunately this is a very special case where we can't have multiple copies | 07:38 |
| tkajinam | yeah | 07:41 |
| tkajinam | wondering having separate options such as enable_fence_watchdog and fence_watch_dog level can be simpler | 07:42 |
| lmiccini | I would stick with the usual fencing way tbh, I hope at some point pacemaker will support sbd for remotes and eventually we can get rid of all the complexity | 07:43 |
| tkajinam | yeah and also allow configuring timeout per resource | 07:45 |
| tkajinam | instead of using that cluster-wide property ... | 07:45 |
| lmiccini | yep right now it is a big hammer.. | 07:46 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix GlanceApiInternal network definition https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866165 | 07:56 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: DNM - testing selinux things in CI https://review.opendev.org/c/openstack/tripleo-ansible/+/866347 | 08:05 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 08:05 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: DNM: Try less invasive change of 863041 https://review.opendev.org/c/openstack/puppet-tripleo/+/866436 | 08:06 |
| tkajinam | lmiccini, putting the logic out of that data loop might be simpler. this still accepts the same data structure. we can merge the logic into the existing look when we get per-node watchdog resource ^^^ | 08:08 |
| tkajinam | *the existing loop* | 08:09 |
| lmiccini | tkajinam: are you concerned with the "$watchdog_devices = local_fence_devices('fence_watchdog', $all_devices)" specifically? | 08:10 |
| tkajinam | no. more about making the logic in the $data.each |$items| loop complicated | 08:10 |
| tkajinam | if we need the different treatment for fence_watchdog device then adding separate logic would be cleaner, IMO | 08:11 |
| tkajinam | lmiccini, see https://review.opendev.org/c/openstack/puppet-tripleo/+/866436 | 08:12 |
| lmiccini | tkajinam: ack I see what you mean | 08:17 |
| lmiccini | I will give it a go | 08:17 |
| tkajinam | lmiccini, just updated my comment in your patch. I don't block it because of my "preference". however if that version makes sense to you then feel free pull that implementation. | 08:18 |
| tkajinam | I'll keep my -2 on mine to keep your review go. | 08:19 |
| lmiccini | tkajinam: sure. I'll test it, the fewer lines of code the better. | 08:20 |
| tkajinam | lmiccini, thanks ! | 08:21 |
| *** pojadhav is now known as pojadhav|lunch | 08:21 | |
| *** frenzy_friday|rover is now known as frenzy_friday | 08:33 | |
| *** jpena|off is now known as jpena | 08:38 | |
| *** pojadhav|lunch is now known as pojadhav| | 08:43 | |
| *** pojadhav| is now known as pojadhav | 08:43 | |
| *** yadnesh is now known as yadnesh|afk | 08:49 | |
| *** ysandeep|brb is now known as ysandeep | 08:51 | |
| *** amoralej|off is now known as amoralej | 08:54 | |
| marios|ruck | ade_lee: o/ can you please have a look at https://bugs.launchpad.net/tripleo/+bug/1998211 when you next get some time | 09:16 |
| ade_lee | marios|ruck, so what is different about fs64? | 09:18 |
| ade_lee | marios|ruck, and has it ever worked - or is this brand new? | 09:18 |
| ade_lee | and is it passing on other branches? | 09:19 |
| opendevreview | Daniel Bengtsson proposed openstack/tripleo-ansible master: Remove old podman files. https://review.opendev.org/c/openstack/tripleo-ansible/+/865514 | 09:20 |
| marios|ruck | ade_lee: so relatively new for zed (build history is not great there https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp_1supp-featureset064-zed ) .. it is ipa with custom naming (eg overcloud1 vs overcloud) | 09:20 |
| marios|ruck | ade_lee: wallaby is not too bad https://review.rdoproject.org/zuul/builds?job_name=periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp_1supp-featureset064-wallaby | 09:21 |
| marios|ruck | ade_lee: master is similar to zed but it fails on different issue (installing from galaxy) so unrelated e.g. https://logserver.rdoproject.org/openstack-periodic-integration-main/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp_1supp-featureset064-master/078d752/logs/supplemental/home/cloud-user/ipa_prep.sh.log.txt.gz | 09:21 |
| marios|ruck | ade_lee: 'master is similar to zed' i meant the build history is not pretty | 09:22 |
| ade_lee | marios|ruck, are we using secure rbac rules? | 09:23 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Changing mount parameter of podman task to volume https://review.opendev.org/c/openstack/tripleo-validations/+/866438 | 09:27 |
| marios|ruck | ade_lee: not on this job | 09:32 |
| opendevreview | Karthik S proposed openstack/os-net-config master: Avoid reconfiguring SR-IOV devices during stack update https://review.opendev.org/c/openstack/os-net-config/+/866312 | 09:33 |
| odyssey4me | hey folks - we have a backport ready to go whenever someone has a chance to review it - https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859435 | 09:44 |
| opendevreview | Antonio Romito proposed openstack/tripleo-heat-templates master: Pacemaker cleanup during scale_down task for Compute nodes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865445 | 09:53 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 09:56 |
| *** yadnesh|afk is now known as yadnesh | 10:00 | |
| *** ysandeep is now known as ysandeep|afk | 10:01 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865631 | 10:17 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 10:18 |
| opendevreview | Oliver Walsh proposed openstack/tripleo-ansible master: WIP: fix ceph_spec_bootstrap inventory parsing https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 10:47 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Remove standalone-upgrade-ffu from gate until it is stable to vote https://review.opendev.org/c/openstack/tripleo-ci/+/866441 | 10:51 |
| *** dviroel_ is now known as dviroel | 11:05 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: DNM - testing selinux things in CI https://review.opendev.org/c/openstack/tripleo-ansible/+/866347 | 11:06 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 11:07 |
| *** rlandy|out is now known as rlandy|rover | 11:10 | |
| *** ysandeep|afk is now known as ysandeep | 11:17 | |
| opendevreview | Fernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes https://review.opendev.org/c/openstack/tripleo-ansible/+/862795 | 11:19 |
| opendevreview | Fernando Díaz Bravo proposed openstack/tripleo-ansible master: Only do snapshot on inactive nodes https://review.opendev.org/c/openstack/tripleo-ansible/+/862795 | 11:22 |
| opendevreview | Oliver Walsh proposed openstack/python-tripleoclient master: Use mergable inventory format for openstack tripleo deploy https://review.opendev.org/c/openstack/python-tripleoclient/+/866447 | 11:49 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: [wallaby-only][ffwd] Delete OVNDBs pacemaker resource https://review.opendev.org/c/openstack/tripleo-heat-templates/+/861154 | 12:33 |
| opendevreview | Antonio Romito proposed openstack/tripleo-heat-templates master: Pacemaker cleanup during scale_down task for Compute nodes https://review.opendev.org/c/openstack/tripleo-heat-templates/+/865445 | 12:36 |
| *** frenzy_friday is now known as frenzy_friday|food | 12:54 | |
| opendevreview | Fernando Díaz Bravo proposed openstack/tripleo-ansible master: Block deletion of existing snapshots executing the creation command https://review.opendev.org/c/openstack/tripleo-ansible/+/866449 | 12:57 |
| opendevreview | Fernando Díaz Bravo proposed openstack/tripleo-ansible master: Block deletion of existing snapshots executing the creation command https://review.opendev.org/c/openstack/tripleo-ansible/+/866449 | 12:58 |
| *** amoralej is now known as amoralej|off | 13:06 | |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Fix password file path message https://review.opendev.org/c/openstack/python-tripleoclient/+/861308 | 13:10 |
| *** frenzy_friday|food is now known as frenzy_friday | 13:13 | |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: DNM - testing selinux things in CI https://review.opendev.org/c/openstack/tripleo-ansible/+/866347 | 13:21 |
| opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Configure ansible-galaxy to use caching proxies https://review.opendev.org/c/openstack/tripleo-ansible/+/866031 | 13:21 |
| opendevreview | Fernando Díaz Bravo proposed openstack/python-tripleoclient master: bnr - Deprecation note https://review.opendev.org/c/openstack/python-tripleoclient/+/866334 | 13:22 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/zed: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866455 | 13:31 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/zed: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866455 | 13:32 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/wallaby: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866456 | 13:32 |
| opendevreview | Luis Tomas Bolivar proposed openstack/tripleo-heat-templates stable/wallaby: Add support for new ovn-bgp-agent config flag https://review.opendev.org/c/openstack/tripleo-heat-templates/+/866456 | 13:32 |
| opendevreview | Luca Miccini proposed openstack/puppet-tripleo master: Enable fence_watchdog configuration in stonith topology https://review.opendev.org/c/openstack/puppet-tripleo/+/863041 | 13:58 |
| *** dasm|off is now known as dasm | 14:14 | |
| opendevreview | Jiri Podivin proposed openstack/validations-common stable/1.6: DNM testing new doc job overrides for stable branches https://review.opendev.org/c/openstack/validations-common/+/866343 | 15:20 |
| *** ysandeep is now known as ysandeep|out | 15:31 | |
| *** dviroel is now known as dviroel|lunhc | 15:43 | |
| *** dviroel|lunhc is now known as dviroel | 16:19 | |
| *** yadnesh is now known as yadnesh|away | 16:26 | |
| *** marios|ruck is now known as marios|out | 16:50 | |
| opendevreview | Merged openstack/tripleo-upgrade stable/train: Ensure playbook fails on any error https://review.opendev.org/c/openstack/tripleo-upgrade/+/865986 | 17:16 |
| *** jpena is now known as jpena|off | 17:31 | |
| *** dviroel is now known as dviroel|afk | 17:58 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Filter excluded nodes from ansible delegates https://review.opendev.org/c/openstack/tripleo-heat-templates/+/859435 | 19:18 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Handle child groups when building ceph spec from inventory https://review.opendev.org/c/openstack/tripleo-ansible/+/866503 | 21:04 |
| opendevreview | Merged openstack/tripleo-ansible stable/train: Ensure english locale when running the command for ovs checks. https://review.opendev.org/c/openstack/tripleo-ansible/+/832305 | 21:19 |
| owalsh | fultonj: FYI https://review.opendev.org/c/openstack/tripleo-ansible/+/866410 | 21:39 |
| fultonj | owalsh: yeah, i saw that after i got mine working | 21:39 |
| fultonj | i commented in it | 21:39 |
| owalsh | ack, CI isn't happy though but that looks like a bug in the cento ansible-core rpm maybe | 21:40 |
| fultonj | thanks for 866410 let me see about using ansible.inventory.manager in my patch | 21:40 |
| fultonj | i was trying to make my patch look around the inventory more intelligently but by using ansible.inventory.manager i might just get a smarter inventory parses | 21:41 |
| fultonj | e.g. "more intelligently" -> handle child groups | 21:41 |
| fultonj | parser* | 21:42 |
| owalsh | yeah, it should work with any hierarchy | 21:42 |
| fultonj | it's late for you owalsh | 21:42 |
| owalsh | reinstalling fedora :-) | 21:42 |
| * fultonj has to go get his kid from school. have a good weekend | 21:43 | |
| owalsh | just got IRC done, going to quit now. Have a good weekend! | 21:43 |
| fultonj | o/ | 21:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!