opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Stop generating certificate requests for disabled networks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848550 | 00:11 |
---|---|---|
opendevreview | Merged openstack/tripleo-heat-templates master: Remove logic to create flavor/keypair in undercloud Nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848694 | 00:42 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-ansible master: wip: deriverd_parameters: Remove node detection by role + flavor https://review.opendev.org/c/openstack/tripleo-ansible/+/848795 | 00:53 |
opendevreview | Miro Tomaska proposed openstack/tripleo-heat-templates stable/train: Neutron: Explicitly set the default hepervisor name https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848492 | 01:03 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Add handling for OVN Multi-RHEL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848208 | 01:06 |
opendevreview | Miro Tomaska proposed openstack/tripleo-heat-templates stable/train: Neutron: Explicitly set the default hepervisor name https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848492 | 01:16 |
*** rlandy is now known as rlandy|out | 01:17 | |
opendevreview | Brendan Shephard proposed openstack/tripleo-ansible master: Replace six.iteritems() with .items() https://review.opendev.org/c/openstack/tripleo-ansible/+/841608 | 01:30 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-common master: Remove heat-docker-agent https://review.opendev.org/c/openstack/tripleo-common/+/848799 | 02:50 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-common master: Remove unused heat-docker-agent https://review.opendev.org/c/openstack/tripleo-common/+/848799 | 02:50 |
*** ysandeep|out is now known as ysandeep|ruck | 03:12 | |
opendevreview | Steve Baker proposed openstack/tripleo-image-elements master: Switch to LVM thin provisioning https://review.opendev.org/c/openstack/tripleo-image-elements/+/847867 | 03:14 |
opendevreview | Merged openstack/tripleo-ansible master: [update] Tell subscription-manager it's not running in a container. https://review.opendev.org/c/openstack/tripleo-ansible/+/848743 | 03:34 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Remove parameters/resources for undercloud Nova https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848695 | 04:06 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Get rid of RemovalPolicies(Mode) https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848699 | 04:06 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Get rid of RemovalPoliciesMode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848699 | 04:14 |
*** raukadah is now known as chandankumar | 04:22 | |
opendevreview | Merged openstack/tripleo-ansible master: [update] Remove obsolete link reference in error message. https://review.opendev.org/c/openstack/tripleo-ansible/+/848750 | 04:25 |
opendevreview | Merged openstack/tripleo-common master: Remove /v2 URL extension https://review.opendev.org/c/openstack/tripleo-common/+/842242 | 04:30 |
*** ysandeep|ruck is now known as ysandeep|ruck|afk | 04:32 | |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Add missing hidden tags for password/credential/private keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847374 | 04:39 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Add missing hidden tag to password/credential/private key https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847374 | 04:45 |
*** beagles_ is now known as beagles | 04:45 | |
beagles | any cores up yet? I have a few patches left that I need to get merged to master and their subsequent backports | 04:47 |
tkajinam | beagles, which ones ? | 04:47 |
beagles | https://review.opendev.org/c/openstack/tripleo-ansible/+/829107 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847804 https://review.opendev.org/c/openstack/tripleo-ansible/+/847803 | 04:48 |
beagles | tkajinam, ^ | 04:48 |
beagles | tkajinam, and basically https://review.opendev.org/q/topic:designate-wallaby .. though I see backend syntax isn't on for haproxy in wallaby argh | 04:54 |
beagles | (thought that was on by default like forever) | 04:54 |
tkajinam | I've voted +2 for master patches maybe bshephar1 or ramishra can check these to vote +A (or -1) | 05:00 |
beagles | tkajinam, ack thanks! | 05:01 |
tkajinam | I need to leave in ~ 30 minutes so I'm afraid I don't have enough time to review wallaby bacports. once we get EMEA folks then we'd have more cores active | 05:03 |
beagles | tkajinam, ack thanks again! | 05:08 |
*** ysandeep|ruck|afk is now known as ysandeep|ruck | 05:08 | |
opendevreview | Brent Eagles proposed openstack/puppet-tripleo master: Designate: add non-backend syntax for miniDNS haproxy support https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 05:13 |
opendevreview | Brent Eagles proposed openstack/puppet-tripleo master: Designate: add non-backend syntax for miniDNS haproxy support https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 05:29 |
opendevreview | Merged openstack/tripleo-ansible master: Fix default tripleo_network_config_template https://review.opendev.org/c/openstack/tripleo-ansible/+/845378 | 05:39 |
opendevreview | Brent Eagles proposed openstack/puppet-tripleo master: Designate: add non-backend syntax for miniDNS haproxy support https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 05:40 |
opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Add missing hidden tags for password/credential/private keys https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847374 | 05:41 |
opendevreview | Merged openstack/tripleo-heat-templates master: Remove unused OS::TripleO::SoftwareDeployment https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848705 | 05:49 |
opendevreview | Brent Eagles proposed openstack/puppet-tripleo master: Designate: add non-backend syntax for miniDNS haproxy support https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 05:51 |
opendevreview | Brent Eagles proposed openstack/puppet-tripleo stable/wallaby: Designate: create proxy ports on external VIP to access miniDNS workers https://review.opendev.org/c/openstack/puppet-tripleo/+/848733 | 05:58 |
opendevreview | Merged openstack/tripleo-common master: Stop generating KeystoneFernetKey[01] https://review.opendev.org/c/openstack/tripleo-common/+/848040 | 06:14 |
chandankumar | :q! | 06:19 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use default config_podman.yml config for podman driver https://review.opendev.org/c/openstack/tripleo-ansible/+/848104 | 06:20 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: [TripleO-Package] Remove Dockerfiles https://review.opendev.org/c/openstack/tripleo-ansible/+/848108 | 06:21 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 06:24 |
beagles | Tengu, could I bother you for some patch reviews when you get a sec starting with https://review.opendev.org/c/openstack/tripleo-ansible/+/829107 ? | 06:35 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Rename molecule jobs to centos-stream https://review.opendev.org/c/openstack/tripleo-ansible/+/848130 | 06:37 |
beagles | 995528 | 06:51 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 06:55 |
Tengu | beagles: sure | 06:57 |
beagles | Tengu, thx! | 06:58 |
opendevreview | Merged openstack/tripleo-ansible master: Remove duplicate entries from .gitignore https://review.opendev.org/c/openstack/tripleo-ansible/+/848460 | 06:58 |
opendevreview | Merged openstack/tripleo-heat-templates master: nova-compute: Refactor hieradata generation https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847531 | 06:58 |
opendevreview | Merged openstack/tripleo-heat-templates master: Glance: Remove unnecessary override of [paste_deploy] config_file https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847524 | 06:58 |
Tengu | beagles: a small question/comment, just to be sure | 07:00 |
Tengu | but otherwise looks good. | 07:01 |
beagles | Tengu, thanks .. yeah the jinja indexing seems to start at 1 for that loop | 07:02 |
opendevreview | Merged openstack/tripleo-ansible master: Replace six.iteritems() with .items() https://review.opendev.org/c/openstack/tripleo-ansible/+/841608 | 07:03 |
Tengu | beagles: 'k. annd... what about firewalling? iirc, that port should be open, shouldn't it? | 07:03 |
beagles | Tengu, oh yes - it's covered by https://review.opendev.org/c/openstack/tripleo-heat-templates/+/829108 | 07:03 |
Tengu | oh, overlooked that in the depends-on | 07:04 |
beagles | Tengu, it's def a weird one where it's a computed port series ... took a few tries to get that worked out | 07:04 |
Tengu | I know your pain - tripleo-lab has some of those fancy features in its internals, it's a proper source of headaches. | 07:04 |
beagles | heh | 07:05 |
Tengu | nudged! | 07:05 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 07:06 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 07:08 |
beagles | Tengu, thanks! | 07:08 |
Tengu | chandankumar: heya! quick question about the "new" molecule support in tripleo-ansible: what's the correct way to get a container launched with systemd instead instead of [what's default now] ? | 07:10 |
chandankumar | Tengu: use ubi9-init | 07:10 |
Tengu | the image? | 07:10 |
chandankumar | Tengu: yes | 07:11 |
Tengu | hmm, where should I set this - Dockerfile is common now? | 07:11 |
Tengu | oh. | 07:11 |
Tengu | image: centos/centos:stream9 ? | 07:11 |
chandankumar | Tengu: https://opendev.org/openstack/tripleo-ansible/src/branch/master/.config/molecule/config_podman.yml | 07:11 |
chandankumar | will take care of that | 07:11 |
chandankumar | Tengu: for example https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_run_cephadm/molecule/default/molecule.yml | 07:12 |
chandankumar | it uses the config_podman.yml config | 07:12 |
Tengu | o_O | 07:13 |
Tengu | how? where? | 07:13 |
Tengu | do I need more coffee? | 07:13 |
chandankumar | Tengu: magic is here https://opendev.org/openstack/tripleo-ansible/src/branch/master/tests/test_molecule.py#L21 | 07:13 |
Tengu | oh, so as long as the driver is set to podman, it's using that config_podman.yml, therefore the ubi9/ubi-init image. | 07:14 |
chandankumar | Tengu: yes correct | 07:14 |
Tengu | ok. so I have another issue then. | 07:14 |
Tengu | aahhh | 07:15 |
Tengu | OK. I get it. | 07:15 |
chandankumar | Tengu: you can override those values | 07:15 |
Tengu | the molecule.yml isn't correct for my new role | 07:15 |
Tengu | https://review.opendev.org/c/openstack/tripleo-ansible/+/848758/4/tripleo_ansible/roles/tripleo_auditd/molecule/default/molecule.yml | 07:15 |
chandankumar | Tengu: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo_collectd/molecule/collectd_connection/molecule.yml | 07:16 |
Tengu | that's not the correct content. afaik I can remove the "platform" section. | 07:16 |
Tengu | let's see. | 07:16 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 07:16 |
chandankumar | Tengu: you can remove from line 5 to line 24 | 07:16 |
Tengu | yep | 07:16 |
chandankumar | it will work | 07:17 |
Tengu | shouldn't we make that the default actually? | 07:17 |
chandankumar | Tengu: delegated driver is default | 07:17 |
Tengu | hmmm. | 07:17 |
chandankumar | https://opendev.org/openstack/tripleo-ansible/src/branch/master/.config/molecule/config.yml | 07:17 |
Tengu | apparently not. | 07:17 |
Tengu | because the _skeleton_ isn't updated with the new things. | 07:17 |
Tengu | chandankumar: https://opendev.org/openstack/tripleo-ansible/src/branch/master/_skeleton_role_/molecule/default/molecule.yml | 07:18 |
Tengu | we shall remove, or just comment, the L5-L25 imho | 07:18 |
Tengu | so that we fallback on the default thingy. | 07:18 |
Tengu | (and maybe the driver block as well) | 07:18 |
Tengu | WDYT? | 07:18 |
Tengu | it's used to provision the content of a new role when we use the role-addition.yml playbook | 07:19 |
chandankumar | Tengu: in skeleton_role, sounds good to remove L5-L25 | 07:19 |
chandankumar | and we can also drop dockerfile form _skeletion_role | 07:20 |
chandankumar | *from | 07:20 |
Tengu | chandankumar: 'k - lemme do a change request for that. | 07:20 |
chandankumar | Tengu: great, by the way thanks for adding env_data role and merging mol config patches | 07:21 |
Tengu | no problem :). Hopefully it will be useful for future debugging. | 07:21 |
Tengu | and if needed, we can add some more gathering. | 07:22 |
chandankumar | Tengu: wallaby backport is ready also https://review.opendev.org/q/topic:unified_mol_config | 07:22 |
Tengu | the role is pretty easy. | 07:22 |
*** amoralej|off is now known as amoralej | 07:22 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: Clean the _skeleton_role_ content https://review.opendev.org/c/openstack/tripleo-ansible/+/848813 | 07:23 |
Tengu | chandankumar: -^^ | 07:23 |
beagles | Tengu, got a couple more if you get a sec https://review.opendev.org/q/topic:designate-byob | 07:24 |
chandankumar | Tengu: looks good, thanks :-) | 07:24 |
Tengu | chandankumar: np | 07:24 |
Tengu | beagles: on it | 07:24 |
beagles | Tengu, fwiw - I need to do a bunch of quickstart work in order to get this testable in u/s CI (stand up some external binds) but we figured it's an "out of the way" feature that we can start doing both u/s and d/s CI work on | 07:25 |
beagles | Tengu, thx | 07:26 |
Tengu | beagles: are you sure the indentation is correct? | 07:27 |
Tengu | I have some doubts, with the spaces heading the {% for ... %} | 07:27 |
Tengu | though, apparently, older blocs have the same display. | 07:28 |
Tengu | for instance https://review.opendev.org/c/openstack/tripleo-ansible/+/847803/4/tripleo_ansible/roles/designate_bind_pool/templates/pools.yaml.j2#47 | 07:28 |
beagles | Tengu, yeah.. seemed to work out in manual testing | 07:30 |
beagles | Tengu, and the old block is covered by CI ... | 07:31 |
* beagles is bracing for all of the headaches that are going to come from admins wanting to link in their own bind's but need to open firewall rules for remote rndc etc | 07:33 | |
beagles | fortunately I think that will be the only pain point - the rest of it works using regular DNS stuff | 07:34 |
Tengu | firewall can be easily done with the ExtraFirewallRules or something like that. | 07:34 |
Tengu | not THAT hard. though, yes, we'll face a surge of BZ due to lack of proper knowledge :) | 07:35 |
beagles | :) | 07:36 |
Tengu | beagles: comments added on both. | 07:36 |
Tengu | sorry :) | 07:36 |
beagles | :) | 07:36 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 07:46 |
Tengu | chandankumar: still some issues, but I suspect it's due to something else than systemd.... adding some debugging.... | 07:46 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: Modify designate bind pool config to match proxied endpoints https://review.opendev.org/c/openstack/tripleo-ansible/+/829107 | 07:47 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-ansible/+/847803 | 07:47 |
beagles | dang | 07:48 |
beagles | Tengu, could I get you to revisit https://review.opendev.org/c/openstack/tripleo-ansible/+/829107 - I inadvertantly rebased | 07:48 |
Tengu | sure | 07:49 |
Tengu | #done :) | 07:49 |
opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates master: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847804 | 07:50 |
beagles | Tengu, thx! | 07:51 |
Tengu | thanks for addressing the comments! | 07:52 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 07:59 |
Tengu | grumpf. | 08:12 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 08:12 |
opendevreview | Brendan Shephard proposed openstack/tripleo-ansible master: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/834992 | 08:20 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible master: Designate: remove obsolete template files from designate_bind_pool https://review.opendev.org/c/openstack/tripleo-ansible/+/848817 | 08:27 |
Tengu | chandankumar: quick question: if I want to add capabilities to the molecule container (cap_add: []), should I just copy the content of the podman_config.yml in my molecule.yml, and set the values I want, or may I just set the particular bits in the platform.0.cap_add ? | 08:32 |
chandankumar | Tengu: you can copy platform section from podman_config.yaml to your molecule.yml and add additional params there | 08:34 |
Tengu | ok | 08:34 |
Tengu | let's see. | 08:35 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 08:35 |
Tengu | auditd service is doing some stuff that are apparently blocked in podman. | 08:35 |
Tengu | "yay". | 08:35 |
Tengu | guess SYS_ADMIN cap is needed. | 08:35 |
chandankumar | Tengu: https://github.com/ansible-community/molecule-podman/blob/main/src/molecule_podman/driver.py#L81 | 08:37 |
chandankumar | this is what I think you are looking for | 08:38 |
Tengu | oh. capabilities. not cap_add | 08:38 |
Tengu | good catch, thanks chandankumar ! | 08:38 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 08:39 |
*** ysandeep|ruck is now known as ysandeep|ruck|lunch | 08:43 | |
beagles | Tengu, are there other cores up and around this hour - I've got a bunch of backports that need looking at but can't bug you for everything | 08:44 |
beagles | it's one of the downsides of component integration side of things... if there's only one person on it, you often end up having to solicit reviews | 08:45 |
Tengu | beagles: I suppose ramishra is around, and you may catch tkajinam and/or bshephar1 (though they are in Japan and Australia, so maybe they are taking a nap ;)) | 08:45 |
beagles | Tengu, thanks | 08:45 |
Tengu | beagles: and later today, you should get North America :) | 08:46 |
Tengu | beagles: btw, I thought you were in North America... ? | 08:46 |
beagles | is slagle on pto this week? | 08:46 |
beagles | Tengu, I am! | 08:46 |
Tengu | beagles: night owl? | 08:46 |
Tengu | slagle isn't on PTO this week afaik. | 08:47 |
Tengu | but he'll come a bit later - in about 3 or 4 hours. | 08:47 |
beagles | Tengu, not really, dog woke me up but I was planning on getting up early to get some EMEA help anyways :) | 08:47 |
beagles | Tengu, ack thanks | 08:47 |
Tengu | heh :). dogs are good wakers. | 08:48 |
Tengu | -.- mine isn't correctly set though, waking me up ~30 minutes before the actual time. | 08:48 |
odyssey4me | Tengu mind giving +w to this backport? https://review.opendev.org/c/openstack/tripleo-ansible/+/848744 | 08:48 |
Tengu | odyssey4me: you don't have core? | 08:48 |
odyssey4me | Tengu also this one https://review.opendev.org/c/openstack/tripleo-ansible/+/848752 | 08:48 |
odyssey4me | Tengu nope :( I don't review often enough, I don't think... | 08:49 |
Tengu | done and done. | 08:50 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Add handling for OVN Multi-RHEL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848208 | 08:50 |
odyssey4me | many thanks | 08:50 |
Tengu | happy to help :) | 08:50 |
beagles | ramishra, if you're around and have sec.. looking for some backport reviews https://review.opendev.org/q/topic:designate-wallaby-neutron-integration | 08:51 |
opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart-extras master: [DNM] Test https://review.opendev.org/c/openstack/tripleo-ci/+/848260 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/848819 | 08:51 |
beagles | (nothing is ever "just one patch") | 08:51 |
ramishra | beagles: sure, in a call atm, will check in few | 08:59 |
beagles | ramishra, cool thanks | 08:59 |
opendevreview | Brendan Shephard proposed openstack/tripleo-heat-templates master: Add handling for OVN Multi-RHEL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848208 | 09:00 |
opendevreview | Bhagyashri Shewale proposed openstack/tripleo-quickstart master: [DNM] test https://review.opendev.org/c/openstack/tripleo-ci/+/848260 https://review.opendev.org/c/openstack/tripleo-quickstart/+/848820 | 09:06 |
beagles | hjensas, you around? where port cleanup code provisioned nodes on overcloud delete these days? | 09:10 |
beagles | hjensas, "port cleanup code for provisioned nodes" | 09:11 |
ramishra | beagles: looking for https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/ansible_plugins/modules/tripleo_overcloud_network_ports.py#L403 | 09:20 |
ramishra | beagles: done! | 09:20 |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Update of OVN controllers as an external task. https://review.opendev.org/c/openstack/tripleo-heat-templates/+/830280 | 09:21 |
beagles | ramishra, thanks and thanks! | 09:29 |
opendevreview | Bhagyashri Shewale proposed openstack/tripleo-heat-templates stable/train: [DNM] test 848260 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848825 | 09:30 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 09:36 |
opendevreview | Brendan Shephard proposed openstack/tripleo-ansible master: Add standalone ovn-controller role https://review.opendev.org/c/openstack/tripleo-ansible/+/845653 | 09:39 |
opendevreview | Bhagyashri Shewale proposed openstack/tripleo-ci master: Create containers and standalone-ipa multinode -options jobs https://review.opendev.org/c/openstack/tripleo-ci/+/848260 | 09:40 |
*** jgilaber|away is now known as jgilaber|testing | 09:42 | |
*** jgilaber|testing is now known as jgilaber|testing|testing | 09:43 | |
*** jgilaber|testing|testing is now known as jgilaber | 09:44 | |
beagles | dciabrin__, just ran into a situation where I needed to add a non-use-backend case for haproxy.pp for wallaby and thought I'd ask the questions, is the use_backend_syntax == False a <= wallaby default? do we expect users to ever force this one way or the other? and finally, does we alter this behind the scenes during upgrades or does it remain whatever it was at the time the upgrade was started? | 09:44 |
beagles | ramishra, if you have a few more seconds could I get some reviews on https://review.opendev.org/q/topic:designate-byob | 09:46 |
*** ysandeep|ruck|lunch is now known as ysandeep|ruck | 09:53 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 10:04 |
hjensas | beagles: it is done in different places. | 10:16 |
hjensas | beagles: "Delete OVN MAC Address ports" here: https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/playbooks/cli-overcloud-delete.yaml#L45-L48 | 10:17 |
hjensas | beagles: node ports is deleted in the "node-unprovision" playbook - https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/playbooks/cli-overcloud-node-unprovision.yaml#L68 | 10:18 |
beagles | hjensas, ah okay, the "Delete OVN MAC Address ports" looks like what I want | 10:19 |
beagles | hjensas, thanks! | 10:19 |
hjensas | beagles: yes, I think thats the place you want to set up something to delete the designate related ports. | 10:19 |
beagles | hjensas, you got it! | 10:19 |
hjensas | beagles: regarding, https://review.opendev.org/c/openstack/python-tripleoclient/+/847194 - not for Train. | 10:21 |
beagles | hjensas, yeah that's what I thought | 10:22 |
*** rlandy|out is now known as rlandy | 10:24 | |
opendevreview | James Slagle proposed openstack/tripleo-ci master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ci/+/848834 | 10:41 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ansible/+/848789 | 10:41 |
fultonj | hey beagles i was on pto yesterday. looks like you got reviews on the patch though https://review.opendev.org/c/openstack/tripleo-ansible/+/829107 | 10:49 |
beagles | fultonj, hey yeah, thanks for checking back ! | 10:50 |
fultonj | np | 10:50 |
beagles | fultonj, don't feel left out though - I have a different one for you :) https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 10:51 |
fultonj | beagles: ok, sure | 10:51 |
beagles | hjensas, is it even possible to use the ovn chassis mac map in train? I saw the comment regarding the static map... | 10:52 |
* beagles needs to re-read the train ovn support | 10:53 | |
*** amoralej is now known as amoralej|lunch | 11:05 | |
beagles | thanks fultonj ! | 11:12 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 11:14 |
fultonj | beagles: np | 11:15 |
*** dviroel|out is now known as dviroel | 11:25 | |
rlandy | Tengu: hi - we are reviewing patches - but https://review.opendev.org/q/topic:unified_mol_config (chandan's patch) needs DF sign off. Can you add it to the DF call tomorrow when you create that section in the meeting notes | 11:26 |
Tengu | rlandy: I've voted on some of them already (not all, one needs some more attention because of unclean backport). I'll push the list. | 11:29 |
beagles | slagle, if you get a sec https://review.opendev.org/q/topic:designate-byob | 11:33 |
beagles | also need some eyes on https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 11:34 |
slagle | beagles: done and done | 11:41 |
beagles | slagle woot tanks! | 11:42 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 11:42 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Use default molecule config for roles having delegated driver https://review.opendev.org/c/openstack/tripleo-ansible/+/847927 | 11:47 |
opendevreview | Rajesh Tailor proposed openstack/tripleo-ansible stable/wallaby: Fix typo from 'sdterr' to 'stderr' https://review.opendev.org/c/openstack/tripleo-ansible/+/848725 | 11:50 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 12:15 |
beagles | fultonj, if you get a sec https://review.opendev.org/c/openstack/puppet-tripleo/+/848733 | 12:17 |
*** rlandy is now known as rlandy|brb | 12:23 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 12:24 |
fultonj | beagles: +2 | 12:25 |
beagles | fultonj, thx | 12:26 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 12:33 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Rename molecule jobs to centos-stream https://review.opendev.org/c/openstack/tripleo-ansible/+/848130 | 12:34 |
opendevreview | Rajesh Tailor proposed openstack/tripleo-ansible stable/wallaby: Fix default tripleo_network_config_template https://review.opendev.org/c/openstack/tripleo-ansible/+/848728 | 12:35 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 12:37 |
*** amoralej|lunch is now known as amoralej | 12:39 | |
opendevreview | Juan Larriba proposed openstack/tripleo-heat-templates stable/wallaby: Remove the Backup and Restore ReaR Heat templates https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848850 | 12:39 |
beagles | slagle, if you get a sec https://review.opendev.org/c/openstack/puppet-tripleo/+/848733 | 12:39 |
*** rlandy|brb is now known as rlandy | 12:42 | |
beagles | bogdando, do you have a moment for https://review.opendev.org/c/openstack/puppet-tripleo/+/848733 ? | 12:56 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 13:05 |
beagles | bogdando, thanks! | 13:28 |
opendevreview | yatin proposed openstack/openstack-tempest-skiplist master: Remove rhos-17 multinode job from skiplist https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/848844 | 13:29 |
jm1 | Tengu, ramishra: all rdo patches for podman collection 1.9.4 has been merged :) | 13:30 |
Tengu | jm1: wonderful, thanks for the hard work! | 13:32 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 13:35 |
*** dasm|off is now known as dasm | 13:39 | |
opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Designate: configure collocation IP on bind and neutron dns integration https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848663 | 13:40 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Rename molecule jobs to centos-stream https://review.opendev.org/c/openstack/tripleo-ansible/+/848130 | 13:45 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 13:48 |
opendevreview | Bhagyashri Shewale proposed openstack/tripleo-ci master: Create containers and standalone-ipa multinode -options jobs https://review.opendev.org/c/openstack/tripleo-ci/+/848847 | 13:53 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Designate: remove unnecessary transitional code for moving extra IP https://review.opendev.org/c/openstack/tripleo-ansible/+/848766 | 13:54 |
opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Designate: specify source address when using rndc https://review.opendev.org/c/openstack/tripleo-ansible/+/848767 | 13:55 |
opendevreview | Merged openstack/tripleo-heat-templates master: Fix release name in release note https://review.opendev.org/c/openstack/tripleo-heat-templates/+/844763 | 13:56 |
opendevreview | chandan kumar proposed openstack/tripleo-ansible stable/wallaby: Use ubi9-init image as base https://review.opendev.org/c/openstack/tripleo-ansible/+/848810 | 13:59 |
chandankumar | bogdando: slagle https://review.opendev.org/c/openstack/tripleo-ansible/+/848104 and https://review.opendev.org/c/openstack/tripleo-ansible/+/848108 are good to go, thanks! | 14:00 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 14:02 |
bogdando | \o/ | 14:03 |
slagle | bogdando: chandankumar : thanks! | 14:04 |
chandankumar | bogdando: slagle thanks :-) | 14:05 |
bogdando | well done all that switching | 14:10 |
bogdando | the code without dockerfiles is beautiful | 14:11 |
opendevreview | David Hill proposed openstack/tripleo-validations master: Add proxy validation for undercloud deployment. https://review.opendev.org/c/openstack/tripleo-validations/+/848871 | 14:16 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 14:21 |
opendevreview | David Hill proposed openstack/python-tripleoclient master: Add proxy validation to deploy and upgrade preflight https://review.opendev.org/c/openstack/python-tripleoclient/+/848872 | 14:23 |
opendevreview | Vijayalakshmi proposed openstack/tripleo-ansible stable/wallaby: Ansible_playbook to select ddp package for NFV node provision https://review.opendev.org/c/openstack/tripleo-ansible/+/848426 | 14:24 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 14:39 |
opendevreview | Merged openstack/tripleo-ansible master: Modify designate bind pool config to match proxied endpoints https://review.opendev.org/c/openstack/tripleo-ansible/+/829107 | 14:41 |
rcastillo|rover | ade_lee: Hi. I'm looking into a failure we're getting in our multinode-ipa job | 14:44 |
rcastillo|rover | xek proposed https://bugs.launchpad.net/tripleo/+bug/1980255/comments/5 as a fix | 14:45 |
rcastillo|rover | but I'm not sure how/if we should even implement that on the CI side | 14:45 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 14:53 |
*** dviroel is now known as dviroel|lunch | 14:59 | |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 15:06 |
opendevreview | Merged openstack/puppet-tripleo master: Designate: add non-backend syntax for miniDNS haproxy support https://review.opendev.org/c/openstack/puppet-tripleo/+/848807 | 15:13 |
opendevreview | Cedric Jeanneret proposed openstack/tripleo-ansible master: New role, replacing puppet-auditd https://review.opendev.org/c/openstack/tripleo-ansible/+/848758 | 15:17 |
beagles | slagle, if you get a sec, this got through the check job https://review.opendev.org/c/openstack/tripleo-ansible/+/847803 | 15:22 |
opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates stable/wallaby: Allow configuring the base port of the mini DNS proxy endpoints https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848878 | 15:24 |
slagle | beagles: +w | 15:27 |
*** ysandeep|ruck is now known as ysandeep|dinner | 15:28 | |
opendevreview | David Hill proposed openstack/tripleo-validations master: Add proxy validation for undercloud deployment. https://review.opendev.org/c/openstack/tripleo-validations/+/848871 | 15:31 |
beagles | slagle, thank once again! | 15:38 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_bootstrap: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/840675 | 15:40 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_kernel: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/841795 | 15:40 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_podman: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/841996 | 15:40 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_logrotate_crond: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/841997 | 15:41 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_sshd: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842150 | 15:41 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_timezone: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842234 | 15:41 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_iscsid: update for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842437 | 15:42 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 15:42 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_nova_compute: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/843659 | 15:45 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Example hiera bindings fixes based on tht data https://review.opendev.org/c/openstack/tripleo-ansible/+/847199 | 15:45 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add logging tasks for common nova services use https://review.opendev.org/c/openstack/tripleo-ansible/+/847028 | 15:45 |
opendevreview | Merged openstack/tripleo-ci master: Add CentOS9 standalone job with FIPS enabled https://review.opendev.org/c/openstack/tripleo-ci/+/847858 | 15:46 |
*** marios is now known as marios|out | 15:54 | |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: tripleo_nova_libvirt: add for standalone roles https://review.opendev.org/c/openstack/tripleo-ansible/+/842152 | 16:00 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Fix nova libvirt role vars mappings and linting https://review.opendev.org/c/openstack/tripleo-ansible/+/847594 | 16:00 |
*** ysandeep|dinner is now known as ysandeep | 16:03 | |
opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Removal of tempest_whitelist variable https://review.opendev.org/c/openstack/tripleo-ci/+/848097 | 16:06 |
*** dviroel|lunch is now known as dviroel | 16:08 | |
*** ysandeep is now known as ysandeep|out | 16:08 | |
opendevreview | Rafael Castillo proposed openstack/openstack-tempest-skiplist master: Revert "Add hard_reboot_after_vol_snap_deletion to skiplist" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/848855 | 16:10 |
*** amoralej is now known as amoralej|off | 16:28 | |
opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: WIP - Add fips jobs for centos 9 https://review.opendev.org/c/openstack/tripleo-ci/+/824479 | 16:32 |
opendevreview | Merged openstack/tripleo-ansible master: Designate: remove obsolete template files from designate_bind_pool https://review.opendev.org/c/openstack/tripleo-ansible/+/848817 | 18:16 |
opendevreview | Merged openstack/tripleo-ansible master: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/834992 | 18:16 |
opendevreview | Merged openstack/tripleo-heat-templates master: Glance: Remove ineffective parameters https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847523 | 18:16 |
opendevreview | Merged openstack/tripleo-ansible master: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-ansible/+/847803 | 18:16 |
opendevreview | Merged openstack/tripleo-heat-templates master: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-heat-templates/+/847804 | 19:04 |
opendevreview | Merged openstack/tripleo-quickstart master: Run featureset056 job with tempest-allow https://review.opendev.org/c/openstack/tripleo-quickstart/+/842354 | 19:04 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible stable/wallaby: Modify designate bind pool config to match proxied endpoints https://review.opendev.org/c/openstack/tripleo-ansible/+/848893 | 19:12 |
opendevreview | Brent Eagles proposed openstack/tripleo-ansible stable/wallaby: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-ansible/+/848894 | 19:13 |
opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates stable/wallaby: Designate: support configuring non-ooo-deployed binds https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848895 | 19:14 |
opendevreview | Rafael Castillo proposed openstack/tripleo-quickstart-extras master: DNM: Testing multinode ipa https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/848896 | 19:17 |
opendevreview | Neil Hanlon proposed openstack/diskimage-builder master: Add Rockylinux 9 build configuration and update jobs for 8 and 9 https://review.opendev.org/c/openstack/diskimage-builder/+/848901 | 19:42 |
opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Remove initial v1 registry entries before installing Ceph https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/848902 | 19:46 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add initial standalone playbooks and inventory for a compute node https://review.opendev.org/c/openstack/tripleo-ansible/+/840509 | 19:58 |
opendevreview | John Fulton proposed openstack/tripleo-quickstart-extras master: Remove initial v1 registry entries before installing Ceph https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/848902 | 20:01 |
opendevreview | James Slagle proposed openstack/tripleo-ansible master: Add docs for standalone roles and playbooks https://review.opendev.org/c/openstack/tripleo-ansible/+/847347 | 20:28 |
opendevreview | James Slagle proposed openstack/tripleo-ansible stable/wallaby: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/848857 | 20:35 |
opendevreview | James Slagle proposed openstack/tripleo-ansible stable/train: Improve connectivity check https://review.opendev.org/c/openstack/tripleo-ansible/+/848904 | 20:37 |
opendevreview | James Slagle proposed openstack/tripleo-ci master: DO-NOT-MERGE: test https://review.opendev.org/c/openstack/tripleo-ci/+/848834 | 20:44 |
*** dviroel is now known as dviroel|out | 20:50 | |
*** dasm is now known as dasm|off | 21:36 | |
*** rlandy is now known as rlandy|biab | 21:50 | |
opendevreview | Miguel Lavalle proposed openstack/tripleo-heat-templates stable/train: [ovn] Don't add conntrack entries for Geneve https://review.opendev.org/c/openstack/tripleo-heat-templates/+/848907 | 22:49 |
*** rlandy|biab is now known as rlandy | 23:01 | |
*** rlandy is now known as rlandy|out | 23:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!