| *** pmannidi is now known as pmannidi|brb | 01:11 | |
| *** pmannidi|brb is now known as pmannidi | 01:28 | |
| *** rlandy|rover|bbl is now known as rlandy|rover | 01:46 | |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates stable/ussuri: Enable keystone_authtoken/memcach_use_advanced_pool for Sahara https://review.opendev.org/c/openstack/tripleo-heat-templates/+/806908 | 04:21 |
|---|---|---|
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible stable/wallaby: Create /etc/cni/net.d if it does not exist https://review.opendev.org/c/openstack/tripleo-ansible/+/806909 | 05:06 |
| *** pmannidi is now known as pmannidi|Lunch | 05:07 | |
| *** pmannidi|Lunch is now known as pmannidi | 05:41 | |
| *** ysandeep|out is now known as ysandeep | 06:16 | |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: undercloud-debug validation refactor https://review.opendev.org/c/openstack/tripleo-validations/+/805852 | 06:31 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: DNM test https://review.opendev.org/c/openstack/tripleo-ansible/+/807027 | 06:32 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Add CephRbdTrashPurgeInterval parameter https://review.opendev.org/c/openstack/tripleo-heat-templates/+/805987 | 06:53 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-ci master: WIP/DNM - Add fips job https://review.opendev.org/c/openstack/tripleo-ci/+/798948 | 06:55 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 06:56 |
| *** amoralej|off is now known as amoralej | 07:05 | |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: undercloud-debug validation documentation improvement https://review.opendev.org/c/openstack/tripleo-validations/+/805852 | 07:23 |
| *** jpena|off is now known as jpena | 07:39 | |
| opendevreview | Merged openstack/tripleo-repos master: Make ansible-test sanity voting https://review.opendev.org/c/openstack/tripleo-repos/+/805173 | 07:46 |
| opendevreview | mbu proposed openstack/tripleo-common stable/train: [Train-only] Upload validation container in swift https://review.opendev.org/c/openstack/tripleo-common/+/806878 | 07:52 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: undercloud-debug validation documentation improvement https://review.opendev.org/c/openstack/tripleo-validations/+/805852 | 08:02 |
| opendevreview | Merged openstack/puppet-tripleo stable/wallaby: Make sure there is a space after debug in virtlogd_wrapper https://review.opendev.org/c/openstack/puppet-tripleo/+/806902 | 08:10 |
| *** ykarel is now known as ykarel|lunch | 08:13 | |
| *** ysandeep is now known as ysandeep|lunch | 08:16 | |
| jpodivin | I have a ... question regarding the openstack upper constraints. From the way I undestand it, the maximum versions of the packages are listed there, and if you try to install something you are supposed to use -c with the `pip` to make sure you respect them. Correct? | 08:17 |
| jpodivin | But here I am installing molecule, which depends on pluggy <1.0 and >=0.7.1. Which fails becuase 1.0.0 is in the upper constraints? | 08:18 |
| opendevreview | Grzegorz Grasza proposed openstack/tripleo-heat-templates stable/wallaby: Environment for switching to using IPs for memcached https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807051 | 08:37 |
| opendevreview | Grzegorz Grasza proposed openstack/tripleo-heat-templates stable/victoria: Environment for switching to using IPs for memcached https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807052 | 08:38 |
| opendevreview | Grzegorz Grasza proposed openstack/tripleo-heat-templates stable/ussuri: Environment for switching to using IPs for memcached https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807053 | 08:38 |
| opendevreview | Grzegorz Grasza proposed openstack/tripleo-heat-templates stable/train: Environment for switching to using IPs for memcached https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807054 | 08:38 |
| jpodivin | Does anyone know what is the proper way to solve this? Because from where I stand, the req constraints have to be either updated, or replaced with globals | 08:39 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Setting constraints to global rather than upper https://review.opendev.org/c/openstack/tripleo-validations/+/807046 | 08:41 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Allow to pass the release version as parameter https://review.opendev.org/c/openstack/python-tripleoclient/+/807070 | 08:49 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Replace undercloud-setup IPA part with specific tasks https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/805951 | 09:00 |
| *** ysandeep|lunch is now known as ysandeep | 09:12 | |
| *** chem is now known as Guest6061 | 09:19 | |
| opendevreview | chandan kumar proposed openstack/python-tripleoclient master: [tcib]Added python-version parameter https://review.opendev.org/c/openstack/python-tripleoclient/+/807086 | 09:20 |
| chandankumar | Tengu: ^^ | 09:20 |
| chandankumar | we got both covered | 09:20 |
| chandankumar | now going to use in tripleo common patch | 09:20 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Allow to pass the release version as parameter https://review.opendev.org/c/openstack/python-tripleoclient/+/807070 | 09:21 |
| Tengu | chandankumar: ah. ok. I just forced the type to be an int for the "realses" -^^ | 09:21 |
| Tengu | but it's more cosmetics. | 09:22 |
| Tengu | and shouldn't conflict. | 09:22 |
| chandankumar | yes | 09:22 |
| Tengu | chandankumar: wondering if there wouldn't be a better way than adding N new parameters though. | 09:22 |
| Tengu | chandankumar: what about something like "--tcib-environment foo=bar,zoo=topia" ? | 09:23 |
| Tengu | that would make the whole thing far, far more generic, extensible and easy. | 09:23 |
| Tengu | wdyt? | 09:24 |
| Tengu | or --tcib-extras | 09:24 |
| chandankumar | Tengu: that also sounds good :-) | 09:24 |
| Tengu | chandankumar: lemme do another one with this specific thing. | 09:24 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Role description auto-extraction https://review.opendev.org/c/openstack/tripleo-validations/+/807092 | 09:26 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Setting constraints to global rather than upper https://review.opendev.org/c/openstack/tripleo-validations/+/807046 | 09:28 |
| opendevreview | Merged openstack/tripleo-ci master: Remove unused c8s nodesets https://review.opendev.org/c/openstack/tripleo-ci/+/803865 | 09:30 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Revert "Disable postcopy for ovn/ovs-dpdk" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/806529 | 09:30 |
| Tengu | chandankumar: almost done with my change - just want to make it robust enough.. | 09:36 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/wallaby: Revert "Disable postcopy for ovn/ovs-dpdk" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807055 | 09:40 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/victoria: Revert "Disable postcopy for ovn/ovs-dpdk" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807056 | 09:40 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/ussuri: Revert "Disable postcopy for ovn/ovs-dpdk" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807057 | 09:40 |
| opendevreview | Martin Schuppert proposed openstack/tripleo-heat-templates stable/train: Revert "Disable postcopy for ovn/ovs-dpdk" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807058 | 09:41 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: DNM test https://review.opendev.org/c/openstack/tripleo-ansible/+/807027 | 09:41 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Allow to pass arbitrary parameters to TCIB image config https://review.opendev.org/c/openstack/python-tripleoclient/+/807098 | 09:41 |
| Tengu | chandankumar: -^^ WDYT? | 09:41 |
| Tengu | chandankumar: ONE thing is enforced: "tcib_" prefix must be set to the keys, just to ensure we're doing what we're thinking we're doing. | 09:42 |
| chandankumar | Tengu: looks good, need to consume that first and see how it goes | 09:43 |
| chandankumar | let me go with tcib_extras | 09:44 |
| Tengu | chandankumar: np - lemme know how it goes. then we'll need to get Alex and cloudnull approval :) | 09:44 |
| chandankumar | it will open more posibility with in tcib tool | 09:45 |
| Tengu | yup. that's the goal. | 09:45 |
| Tengu | without too many issues hopefully. | 09:45 |
| Tengu | we'll see. | 09:45 |
| Tengu | but extending it like that sounds like a good idea :) | 09:45 |
| Tengu | taking a break now, back in ~1h | 09:46 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: WIP - Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 09:47 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Parameter to set api policy to limit live migration to role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/802027 | 09:48 |
| opendevreview | Juan Badia Payno proposed openstack/tripleo-quickstart-extras master: Create overcloud_baremetal_deploy.yaml file properly https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797452 | 09:53 |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: [WIP]Override default network_data/vip yaml for downstream bm jobs https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/806724 | 09:54 |
| *** ykarel|lunch is now known as ykarel | 09:56 | |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: DNM test https://review.opendev.org/c/openstack/tripleo-ansible/+/807027 | 10:03 |
| opendevreview | Rajesh Tailor proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 10:13 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations master: Role description auto-extraction https://review.opendev.org/c/openstack/tripleo-validations/+/807092 | 10:15 |
| jpodivin | ade_lee: so I took a look at that certmonger installation, tried to introduce it in the fips role, but it looks like it will have to go in the heat template after all. | 10:18 |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/puppet-tripleo/+/803911 | 10:24 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-ci master: WIP/DNM - Add fips job https://review.opendev.org/c/openstack/tripleo-ci/+/798948 | 10:25 |
| opendevreview | Rajesh Tailor proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 10:28 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-ci master: DNM https://review.opendev.org/c/openstack/tripleo-ci/+/806930 | 10:30 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: WIP - Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 10:31 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-ci master: DNM https://review.opendev.org/c/openstack/tripleo-ci/+/806930 | 10:32 |
| *** jcapitao is now known as jcapitao_lunch | 10:50 | |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Allow to pass arbitrary parameters to TCIB image config https://review.opendev.org/c/openstack/python-tripleoclient/+/807098 | 10:52 |
| opendevreview | Marios Andreou proposed openstack/tripleo-repos master: DNM testing for d/stream disable ssl get-hash https://review.opendev.org/c/openstack/tripleo-repos/+/807117 | 10:58 |
| opendevreview | Marios Andreou proposed openstack/tripleo-quickstart master: Use tripleo.repos.get_hash in repo-setup role https://review.opendev.org/c/openstack/tripleo-quickstart/+/791486 | 10:58 |
| Tengu | chandankumar: I think we can drop our other patches adding the release and python-version - wdyt? the one allowing to add arbitrary params sounds better. | 10:59 |
| chandankumar | Tengu: yes, sure | 11:05 |
| Tengu | hmm | 11:13 |
| Tengu | chandankumar: is the tripleo-ci-centos-8-undercloud-upgrade job broken? | 11:13 |
| Tengu | seems to get a steady trend of failures lately, and we're missing most of the logs that might help debugging the failure. | 11:14 |
| *** ysandeep is now known as ysandeep|afk | 11:15 | |
| chandankumar | Tengu: https://zuul.openstack.org/builds?job_name=tripleo-ci-centos-8-undercloud-upgrade looks pretty green with few failure | 11:16 |
| chandankumar | Tengu: can you share one specific example? | 11:17 |
| Tengu | chandankumar: weird... https://review.opendev.org/c/openstack/tripleo-ansible/+/806753 failed, but I don't see where the actual logs are while looking at https://5271630870d153a2fe1e-be6253c0e82f1539fed391a5717e06a0.ssl.cf1.rackcdn.com/806753/1/check/tripleo-ci-centos-8-undercloud-upgrade/f663c9c/logs/index.html | 11:17 |
| Tengu | chandankumar: same for that one: https://review.opendev.org/c/openstack/tripleo-ansible/+/806837 | 11:19 |
| *** dviroel|out is now known as dviroel|ruck | 11:19 | |
| Tengu | chandankumar: maybe it was just a glitch, but the 2 last negative votes from zuul I've checked were linked to that very job | 11:19 |
| chandankumar | Tengu: Run ansible playbook to collect logs got failed in above shared logs | 11:20 |
| chandankumar | ERROR: Collect logs timed out | 11:21 |
| Tengu | that won't help | 11:21 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: Fix molecule job for tripleo_ha_wrapper https://review.opendev.org/c/openstack/tripleo-ansible/+/807027 | 11:26 |
| chandankumar | Tengu: bandini new containers are built now with latest tag c75dfdf78316ba67245f1455633edea2 or https://trunk.rdoproject.org/centos9-master-preview/consistent/delorean.repo.md5 and also contains tripleo-ansible cni fix and rexml fix | 11:37 |
| bandini | chandankumar: awesome, on it | 11:37 |
| *** jpena is now known as jpena|lunch | 11:38 | |
| *** rlandy is now known as rlandy|rover | 11:39 | |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: WIP - Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 11:43 |
| opendevreview | Vijayalakshmi proposed openstack/os-net-config master: Update sriov config service to handle nic partitioned PF https://review.opendev.org/c/openstack/os-net-config/+/806561 | 11:44 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Added pre from centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 11:49 |
| chandankumar | Tengu: hello | 11:50 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 11:50 |
| Tengu | chandankumar: long time no talk ;). what can I do? | 11:51 |
| Tengu | chandankumar: hmmmm what will be setting the actual tcib_release? | 11:51 |
| chandankumar | Tengu: yes, it's too long, please have a look at this https://review.opendev.org/c/openstack/tripleo-common/+/800580 and https://review.opendev.org/c/openstack/tripleo-ci/+/800577/33/roles/build-containers/templates/tripleo-build.sh.j2 | 11:52 |
| Tengu | ah! | 11:52 |
| Tengu | chandankumar: separator is a coma - it will fail I'd say. | 11:52 |
| rlandy|rover | odyssey4me: hi ... would like to chat to someone in upgrades about https://trello.com/c/ulHXnS4Y/2065-cixlp1940844tripleociproa-centos-8-standalone-upgrade-ussuri-failing-after-upgrade-router-ip-address-unreachable | 11:53 |
| rlandy|rover | basically we need to restart openvswitch after the upgrade | 11:53 |
| Tengu | chandankumar: we can probably modify my patch to allow passing that --tcib-extras multiple times, it might be cleaner? can't really say... | 11:53 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Added pre from centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 11:54 |
| rlandy|rover | adding to it our files would not be the best way | 11:54 |
| Tengu | rlandy|rover: ooooh. | 11:54 |
| rlandy|rover | Tengu: hey | 11:54 |
| Tengu | rlandy|rover: heya :). that card will probably help someone soon I think :) | 11:54 |
| chandankumar | Tengu: want to give it a spin this explemetation then modify? | 11:55 |
| chandankumar | *implementation | 11:56 |
| Tengu | chandankumar: as you want - I'm jumping from topic to topic right now -.-' | 11:56 |
| * chandankumar takes a dive and will share the result | 11:56 | |
| rlandy|rover | Tengu: actually | 11:56 |
| rlandy|rover | the other issue is the .ssh permission folder | 11:56 |
| Tengu | rlandy|rover: just to ensure I understand: openvswitch needs to be restarted once the actual upgrade of tripleo/osp is over, leading to a second, short service outage? | 11:56 |
| rlandy|rover | I saw you commented on that | 11:56 |
| Tengu | rlandy|rover: yeah, saw it, saw the patch proposal... and I'm not happy with that proposal | 11:57 |
| rlandy|rover | Tengu: k - let's talk that one over in a sec | 11:57 |
| Tengu | that .ssh thing? | 11:57 |
| rlandy|rover | going back to the upgrade | 11:57 |
| rlandy|rover | for openvswitch | 11:57 |
| Tengu | ok. openvswitch | 11:57 |
| rlandy|rover | there is an incompatibility between ovn | 11:57 |
| Tengu | chandankumar: thanks! sorry for abandonning you with a crapy code change ;) | 11:58 |
| Tengu | rlandy|rover: between versions? that can happen.. | 11:58 |
| rlandy|rover | and the version of openvswitch with | 11:58 |
| rlandy|rover | the version installed with train | 11:58 |
| rlandy|rover | is too old | 11:58 |
| rlandy|rover | ussuri brings in the right version with rdo-openvswitch | 11:58 |
| rlandy|rover | and ir does get installed | 11:59 |
| rlandy|rover | but the service needs a restart | 11:59 |
| Tengu | ok. "service" as in systemd unit for a host-installed service, or "service" as in "one of the many containers" ? | 11:59 |
| rlandy|rover | as in systemctl restart | 12:00 |
| rlandy|rover | so systemd | 12:00 |
| Tengu | ok. is openvswitch pushing anything in the logs that might help detecting that need? | 12:00 |
| rlandy|rover | logs? | 12:01 |
| rlandy|rover | we know the issue | 12:02 |
| rlandy|rover | ie: can't reach instance ip | 12:02 |
| Tengu | yeah - I'm wondering how we can detect such a restart has to happen and apply that in tripleo-heat-templates | 12:02 |
| rlandy|rover | Tengu: slaweq didn't want that in THT | 12:02 |
| Tengu | i.e. using a lame "service: state=restarted when=CONDITION" | 12:02 |
| Tengu | ah | 12:02 |
| rlandy|rover | because that is not a user thing | 12:02 |
| Tengu | that's... | 12:02 |
| rlandy|rover | we need this for CI upgrades | 12:03 |
| Tengu | not only CI I'd say... | 12:03 |
| Tengu | apparently it might also be a thing within OSP during FFU - but it has to be investigated a bit more | 12:03 |
| rlandy|rover | there is an upgrades module to do this | 12:03 |
| Tengu | yeah, tripleo-upgrade | 12:04 |
| Tengu | still.... I've seen this exact same behaviour earlier today with a downstream FFU job. | 12:04 |
| Tengu | i.e. instance launched, but floating IP isn't answering. | 12:04 |
| Tengu | so I'm not THAT sure it's only within CI. | 12:05 |
| rlandy|rover | train -> ussuri has this issue | 12:06 |
| rlandy|rover | but correct - this could show up elsewhere | 12:06 |
| Tengu | train -> wallaby probably has the same, if we could get it upstream as well :) | 12:06 |
| Tengu | so that's why I think it should be within t-h-t, or something actual operator leverage during the upgrade. | 12:07 |
| Tengu | but that's just my guts feeling ;) | 12:08 |
| opendevreview | mbu proposed openstack/python-tripleoclient stable/train: [Train-Only] Run Validations outside of mistral for the deployment https://review.opendev.org/c/openstack/python-tripleoclient/+/807125 | 12:09 |
| opendevreview | Rajesh Tailor proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 12:12 |
| *** ysandeep|afk is now known as ysandeep | 12:13 | |
| *** jcapitao_lunch is now known as jcapitao | 12:21 | |
| *** amoralej is now known as amoralej|lunch | 12:27 | |
| opendevreview | Rajesh Tailor proposed openstack/puppet-tripleo master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/puppet-tripleo/+/803911 | 12:27 |
| opendevreview | Bhagyashri Shewale proposed openstack/tripleo-repos master: WIP - Add module to generate CentOS Compose repos https://review.opendev.org/c/openstack/tripleo-repos/+/804306 | 12:28 |
| opendevreview | mbu proposed openstack/python-tripleoclient stable/train: [Train-Only] Run Validations outside of mistral for the deployment https://review.opendev.org/c/openstack/python-tripleoclient/+/807125 | 12:34 |
| bandini | chandankumar:am i doing something wrong here? https://paste.opendev.org/show/808540/ | 12:39 |
| *** jpena|lunch is now known as jpena | 12:40 | |
| chandankumar | bandini: it got chnaged | 12:40 |
| bandini | chandankumar: also c75dfdf78316ba67245f1455633edea2 does not work. what's the right hash i should be using? | 12:41 |
| chandankumar | bandini: use this tag c75dfdf78316ba67245f1455633edea2 | 12:41 |
| bandini | chandankumar: i am not getting love with that has either? https://paste.opendev.org/show/808541/ | 12:42 |
| chandankumar | bandini: so correct hash is this 14ba5932ccde964e82a6100f108fa62e | 12:44 |
| bandini | ack lemme try that one | 12:44 |
| chandankumar | bandini: https://logserver.rdoproject.org/53/18953/101/check/tripleo-build-containers-stream9-development/03be925/job-output.txt | 12:44 |
| chandankumar | trunk.registry.rdoproject.org/tripleomastercentos9/openstack-manila-share 14ba5932ccde964e82a6100f108fa62e 1a1265bbbea8 5 minutes ago 707 MB | 12:44 |
| bandini | got you | 12:44 |
| chandankumar | bandini: need to switch to current-tripleo | 12:45 |
| bandini | chandankumar: ack, let me know when that tag is available and i will change my scripts | 12:45 |
| chandankumar | bandini: sure | 12:45 |
| chandankumar | bandini: for current situation go with above tag | 12:46 |
| bandini | perfect, that one seems to work | 12:46 |
| *** frenzy_friday is now known as anbanerj|ruck | 12:46 | |
| opendevreview | Sandeep Yadav proposed openstack/tripleo-heat-templates stable/wallaby: Revert "Enable CephDashboard for scenario001" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807064 | 12:47 |
| *** dviroel|ruck is now known as dviroel | 12:48 | |
| jpodivin | I wanted to ask, what is the proper way to reference openstack/requirements constraints? I looked in the doc for requirements project https://docs.openstack.org/project-team-guide/dependency-management.htm but I'm not sure I got it right in regards to actual pip usage | 12:49 |
| jpodivin | Should we use upper-constraints.txt or global-requirements.txt | 12:49 |
| dviroel | hi Tengu , saw your comments on https://review.opendev.org/c/openstack/python-tripleoclient/+/806993 | 12:50 |
| dviroel | and yes, another solution would be guarantee that .ssh folder has heat-user as owner | 12:51 |
| dviroel | it seems that this patch from cloud-init is now changing the folder ownership to root | 12:51 |
| dviroel | but even with .ssh onwership set to root, other jobs are running just fine | 12:52 |
| dviroel | it is a problem for 'sed -i', thats why i was only modifying that specific point | 12:53 |
| Tengu | dviroel: ok.. well, in any cases, ensuring it belongs to the actual user is always a better thing :/. | 12:53 |
| Tengu | I didn't -2 it, so it might be OK to merge, but I still have some concerns when rights/ownerships are wrong.. | 12:54 |
| dviroel | Tengu: have the same feeling, not sure why cloud-init is going to another way | 12:54 |
| Tengu | because cloud doesn't care about rights/ownership? free for all and all the things? :D | 12:54 |
| Tengu | (oh, sorry, we're not Friday yet...) | 12:54 |
| dviroel | lol | 12:54 |
| cloudnull | chandankumar Tengu, re: tcib-extras -- LGTM. if that helps unblock us for future +2 from me . | 12:55 |
| Tengu | cloudnull: cool! | 12:55 |
| Tengu | cloudnull: so you're ok with all the patch and the need to prefix and all? that's too easy :) | 12:56 |
| dviroel | cloudnull: hey o/ - can you also take a look on what Tengu and I were talking about? | 12:57 |
| dviroel | cloudnull: https://review.opendev.org/c/openstack/python-tripleoclient/+/806993 | 12:57 |
| dviroel | cloudnull: have more thoughts about it :) | 12:57 |
| Tengu | :) | 12:57 |
| cloudnull | Tengu to be fair the only ones I reviewed are found here - https://review.opendev.org/q/topic:%22tcib%252Fextra-params%22+(status:open%20OR%20status:merged) - so yes I'm OK with all those patches :P P | 12:58 |
| opendevreview | Ananya proposed openstack/openstack-tempest-skiplist master: Revert "Update skip TestSnapshotPattern and TestVolumeBootPattern" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/806794 | 12:59 |
| cloudnull | dviroel sure | 12:59 |
| Tengu | cloudnull: *all the patch content - sorry :) | 12:59 |
| cloudnull | I knew what you meant 😛 | 12:59 |
| Tengu | sorry, I'm having a hard time this week ;). | 13:00 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: Revert "[train-only] Adding ForceNoTsx flag" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807065 | 13:00 |
| opendevreview | David Vallee Delisle proposed openstack/python-tripleoclient stable/train: Revert "[train-only] post stack creation tsx validation" https://review.opendev.org/c/openstack/python-tripleoclient/+/807066 | 13:00 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates stable/train: Revert "[train-only] Adding ForceNoTsx flag" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807065 | 13:06 |
| opendevreview | David Vallee Delisle proposed openstack/python-tripleoclient stable/train: Revert "[train-only] post stack creation tsx validation" https://review.opendev.org/c/openstack/python-tripleoclient/+/807066 | 13:08 |
| cloudnull | dviroel one question, but otherwise seems fine. | 13:08 |
| Tengu | chandankumar: keeping my patch as WIP, I have some doubts I did it right with the argparse thing. IIRC there's a builtin type for "coma separated list". | 13:08 |
| cloudnull | it would be neat if we could port that functionality into an ansible playbook | 13:08 |
| cloudnull | but that's beyond the scope of that review | 13:09 |
| dviroel | cloudnull: yeah, would be great | 13:13 |
| dviroel | cloudnull: i'm currently trying to unblock train with this fix/workaround | 13:13 |
| cloudnull | ++ | 13:14 |
| dviroel | cloudnull: if we all agree with the solution, i can update the patch based on your comments, thanks | 13:15 |
| opendevreview | Christophe Fontaine proposed openstack/tripleo-ansible master: tripleo_ovs_dpdk: tripleo_ovs_dpdk_socket_memory is now optional https://review.opendev.org/c/openstack/tripleo-ansible/+/807138 | 13:15 |
| Tengu | chandankumar: would you rather get multiple --tcib-extras, or is the current "comma separated list" OK ? | 13:18 |
| Tengu | cloudnull: -^ same question. I'm... well. unable to chose. | 13:18 |
| cloudnull | multiple args | 13:18 |
| *** amoralej|lunch is now known as amoralej | 13:19 | |
| cloudnull | if we have so many args that it becomes clumsy they then I'd say we should make a new template | 13:19 |
| Tengu | cloudnull: ok. just have to remember how python exposes it, but iirc, it's a simple list. | 13:20 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-ansible master: Check whether an HA resource already exists explicitly https://review.opendev.org/c/openstack/tripleo-ansible/+/806610 | 13:22 |
| cloudnull | yeah, nargs+ it just becomes a list | 13:23 |
| chandankumar | Tengu: in meeting will ping back | 13:24 |
| opendevreview | Cedric Jeanneret proposed openstack/python-tripleoclient master: Allow to pass arbitrary parameters to TCIB image config https://review.opendev.org/c/openstack/python-tripleoclient/+/807098 | 13:24 |
| Tengu | cloudnull: iirc "action='append'" is OK -^^ | 13:24 |
| cloudnull | ++ | 13:26 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 13:26 |
| Tengu | chandankumar: updated the patch to take multiple --tcib-extras, makes more sense, and will be more readable in the end. Should have thought about it first -.-'. | 13:26 |
| Tengu | sorry for the back'n'forth | 13:26 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 13:34 |
| opendevreview | David Vallee Delisle proposed openstack/puppet-tripleo master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/puppet-tripleo/+/803911 | 13:36 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-role-container-registry master: Use ansible-core 2.11 for tests https://review.opendev.org/c/openstack/ansible-role-container-registry/+/805982 | 13:39 |
| opendevreview | Bogdan Dobrelya proposed openstack/paunch stable/train: Wrap stopping podman -t with systemd timeouts https://review.opendev.org/c/openstack/paunch/+/807149 | 13:40 |
| opendevreview | Gregory Thiemonge proposed openstack/tripleo-ci master: WIP Testing nested virt for scenario010-standalone job https://review.opendev.org/c/openstack/tripleo-ci/+/799958 | 13:52 |
| ade_lee | jpodivin, where do you think the change will have to go? | 13:52 |
| jpodivin | ade_lee:Tried putting the dependency in the enable fips role. That didn't work, so that leaves the haproxy heat template. haproxy-public-tls-certmonger.yaml | 14:04 |
| chandankumar | Tengu: logs from previous run https://logserver.rdoproject.org/53/18953/101/check/tripleo-build-containers-stream9-development/26e996e/logs/build.log | 14:05 |
| chandankumar | not working | 14:05 |
| chandankumar | I am missing some wiring stuff with tcib | 14:05 |
| opendevreview | amolkahat proposed openstack/tripleo-operator-ansible master: Added Ephemeral Heat CLI in overcloud_deploy role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/806047 | 14:05 |
| Tengu | chandankumar: probably due to the space actually. | 14:06 |
| chandankumar | let me update that and try again | 14:07 |
| ade_lee | jpodivin, interesting -- I'm surprised that the dependency didn't work in the enable_fips role -- I think that runs before the others, right? | 14:07 |
| Tengu | chandankumar: but with the new one, you just put --tcib-extras tcib_release=9 --tcib-extras tcib_python_version=... and | 14:07 |
| jpodivin | ade_lee: it does, | 14:07 |
| chandankumar | aye sir | 14:07 |
| jpodivin | It's odd, logs seem to indicate it was initiated. | 14:08 |
| ade_lee | jpodivin, how did you add the dependency? | 14:08 |
| jpodivin | ade_lee: Depends-On: .... | 14:08 |
| opendevreview | chandan kumar proposed openstack/tripleo-ci master: Added pre from centos 9 container builds https://review.opendev.org/c/openstack/tripleo-ci/+/800577 | 14:08 |
| jpodivin | ade_lee: https://review.opendev.org/c/zuul/zuul-jobs/+/807031 https://review.opendev.org/c/openstack/tripleo-ci/+/798948 | 14:09 |
| ade_lee | jpodivin, ah ok - I see it | 14:09 |
| ade_lee | in zuul-jobs | 14:09 |
| jpodivin | kept it as a DNM, you know, because shot in the dark | 14:10 |
| ade_lee | yup - so is that version available? did the log show it being installed? | 14:10 |
| Tengu | chandankumar: leaving for today - lemme know if it's working and all. see you tomorrow! | 14:11 |
| chandankumar | Tengu: sure, see ya tomorrow, thanks for all the help today :-) | 14:11 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-role-container-registry master: Use ansible-core 2.11 for tests https://review.opendev.org/c/openstack/ansible-role-container-registry/+/805982 | 14:12 |
| jpodivin | ade_lee: I think so, grepping the logs. | 14:12 |
| ade_lee | jpodivin, which logs are you looking at? | 14:13 |
| jpodivin | the job-output.txt | 14:13 |
| ade_lee | jpodivin, yup -which job? | 14:13 |
| jpodivin | ade_lee: I think multinode-ipa, but I've downloaded them so it's on local, I'll try to find the hash | 14:14 |
| opendevreview | Douglas Viroel proposed openstack/python-tripleoclient stable/train: Fix edit authorized_keys file in-place https://review.opendev.org/c/openstack/python-tripleoclient/+/806993 | 14:14 |
| jpodivin | ade_lee: the status of the TASK is changed ... so it didn't fail. | 14:15 |
| jpodivin | ade_lee: there it is https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a09/798948/6/check/tripleo-ci-centos-8-standalone-on-multinode-ipa/a091fd4/job-output.txt | 14:16 |
| jpodivin | ade_lee: could be that it got installed on different host. But that's just a guess | 14:17 |
| ade_lee | jpodivin, so your dependency is 0.79-13 | 14:18 |
| jpodivin | yes, and the dnf logs on the undercloud confirm it was installed. | 14:19 |
| ade_lee | jpodivin, the needed version is 0.79-13-3 | 14:19 |
| * jpodivin facepalm | 14:19 | |
| ade_lee | there is a 0.79-13-2 which will not work | 14:19 |
| opendevreview | alexey-mr proposed openstack/os-net-config master: Allow linux tap to use routes https://review.opendev.org/c/openstack/os-net-config/+/807160 | 14:19 |
| jpodivin | that must be it. I'll update the patch rerun the tests. | 14:20 |
| ade_lee | jpodivin, the question is whether that version is available upstream .. | 14:20 |
| ade_lee | hopefully it is | 14:20 |
| jpodivin | Hm, well there is a higher version available on my fedora, but I don't know about rhel | 14:20 |
| jpodivin | ade_lee: sorry about that. | 14:23 |
| opendevreview | Brendan Shephard proposed openstack/tripleo-docs master: Update paunch tips and tricks https://review.opendev.org/c/openstack/tripleo-docs/+/803129 | 14:25 |
| opendevreview | chandan kumar proposed openstack/tripleo-common master: DNM: Temp patch to test c9 container builds https://review.opendev.org/c/openstack/tripleo-common/+/800580 | 15:01 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Fail validation if pacemaker service is not active https://review.opendev.org/c/openstack/tripleo-validations/+/806904 | 15:05 |
| opendevreview | Merged openstack/tripleo-docs master: Update paunch tips and tricks https://review.opendev.org/c/openstack/tripleo-docs/+/803129 | 15:23 |
| sshnaidm | dpeacock, rlandy|rover do you need me in call as well? (about ci?) | 15:24 |
| rlandy|rover | sshnaidm: yes pls | 15:28 |
| opendevreview | Merged openstack/tripleo-validations stable/ussuri: Fail validation if pacemaker service is not active https://review.opendev.org/c/openstack/tripleo-validations/+/806905 | 15:31 |
| opendevreview | Merged openstack/tripleo-validations stable/train: Fail validation if pacemaker service is not active https://review.opendev.org/c/openstack/tripleo-validations/+/806906 | 15:31 |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Remove name parameter from cell export https://review.opendev.org/c/openstack/python-tripleoclient/+/804330 | 15:31 |
| opendevreview | Merged openstack/puppet-pacemaker master: Add support for fence_kubevirt fencing agent https://review.opendev.org/c/openstack/puppet-pacemaker/+/806924 | 15:31 |
| opendevreview | Merged openstack/puppet-tripleo master: Extend fencing to hosts using fence_kubevirt agent. https://review.opendev.org/c/openstack/puppet-tripleo/+/806926 | 15:31 |
| jpodivin | zbr: hi there do you have a minute ? | 15:36 |
| opendevreview | Martin Schuppert proposed openstack/puppet-tripleo stable/wallaby: Extend fencing to hosts using fence_kubevirt agent. https://review.opendev.org/c/openstack/puppet-tripleo/+/807067 | 15:38 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-docs master: Document adding files into TCIB build environment https://review.opendev.org/c/openstack/tripleo-docs/+/807173 | 15:43 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-docs master: Document adding files into TCIB build environment https://review.opendev.org/c/openstack/tripleo-docs/+/807173 | 15:45 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM https://review.opendev.org/c/openstack/validations-common/+/807174 | 15:48 |
| jpodivin | ade_lee: eh, we might have a bit of problem, I tried both versions of the certmonger available to me, in fedora, neither of them seem to be in the repos used by CI | 15:50 |
| opendevreview | Merged openstack/openstack-tempest-skiplist master: Revert "Update skip TestSnapshotPattern and TestVolumeBootPattern" https://review.opendev.org/c/openstack/openstack-tempest-skiplist/+/806794 | 15:56 |
| mwhahaha | rlandy|rover: did we file a bz against cloud-init for that permission problem? | 16:15 |
| rlandy|rover | mwhahaha: not yet | 16:15 |
| rlandy|rover | will do | 16:15 |
| mwhahaha | we should because i hit it on my own env | 16:15 |
| mwhahaha | the folder is root, but the key file is user | 16:15 |
| rlandy|rover | stream update | 16:15 |
| mwhahaha | yea | 16:16 |
| mwhahaha | probably should be urgent | 16:16 |
| rlandy|rover | yep - doing that now | 16:16 |
| rlandy|rover | dviroel: ^^ fyi | 16:17 |
| dviroel | ok | 16:18 |
| mwhahaha | it just doesn't affect us after switched to ansible because i think we become: true for setup | 16:18 |
| mwhahaha | but i hit it on a basic cloud-init run (no tripleo) | 16:18 |
| dviroel | rlandy|rover: this is probably the patch that cause the issue https://git.centos.org/rpms/cloud-init/blob/0f12fbd6d9d3f58c1b8c68ef8f6f1e5f2cbe40c8/f/SOURCES/ci-Stop-copying-ssh-system-keys-and-check-folder-permis.patch | 16:18 |
| mwhahaha | clearly it does not check folder permissions correctly :D | 16:19 |
| dviroel | lol | 16:24 |
| rlandy|rover | mwhahaha: dviroel: https://bugzilla.redhat.com/show_bug.cgi?id=2000664 | 16:28 |
| rlandy|rover | you can add/edit that | 16:28 |
| rlandy|rover | you are both added to bug cc list | 16:29 |
| mwhahaha | k | 16:29 |
| mwhahaha | nice standalone installs are broken with the basics | 16:30 |
| * mwhahaha dies a bit more inside | 16:30 | |
| mwhahaha | [Errno 2] No such file or directory: '/home/stack/tripleo-heat-installer-templates/environments/templates/net_config_bridge.j2' | 16:30 |
| mwhahaha | anyone? | 16:31 |
| rlandy|rover | release? master? | 16:31 |
| mwhahaha | yea | 16:31 |
| mwhahaha | i found it | 16:32 |
| mwhahaha | our neutron-ovs-dvr.yaml is bad | 16:32 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Fix getting service_name https://review.opendev.org/c/openstack/tripleo-ansible/+/806083 | 16:32 |
| rlandy|rover | no trace in jobs | 16:32 |
| mwhahaha | i'll file a bug later | 16:35 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-repos master: [WIP] Add molecule jobs to test the modules https://review.opendev.org/c/openstack/tripleo-repos/+/806879 | 16:40 |
| *** jpena is now known as jpena|off | 16:47 | |
| *** ysandeep is now known as ysandeep|out | 16:47 | |
| *** amoralej is now known as amoralej|off | 16:49 | |
| opendevreview | Damien Ciabrini proposed openstack/tripleo-heat-templates master: DNM quick override to deploy mariadb on centos9 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807181 | 16:51 |
| *** sshnaidm is now known as sshnaidm|off | 17:01 | |
| bandini | nothing provides genisoimage needed by python3-ironicclient-4.8.0-0.20210902073338.b5df386.el9.noarch | 17:22 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Enable Ceph RGW public port into firewall when using SSL https://review.opendev.org/c/openstack/tripleo-heat-templates/+/806893 | 17:33 |
| opendevreview | Merged openstack/puppet-tripleo master: Remove duplicate spec file https://review.opendev.org/c/openstack/puppet-tripleo/+/805328 | 17:33 |
| opendevreview | Merged openstack/tripleo-common master: Fix awk expression https://review.opendev.org/c/openstack/tripleo-common/+/806549 | 17:33 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Environment for switching to using IPs for memcached https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807051 | 18:25 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Environment for switching to using IPs for memcached https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807052 | 18:25 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 18:29 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Create /etc/cni/net.d if it does not exist https://review.opendev.org/c/openstack/tripleo-ansible/+/806909 | 18:36 |
| opendevreview | Merged openstack/ansible-role-container-registry master: Use ansible-core 2.11 for tests https://review.opendev.org/c/openstack/ansible-role-container-registry/+/805982 | 18:37 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 19:02 |
| opendevreview | David Vallee Delisle proposed openstack/tripleo-heat-templates master: Enable support for Libvirt modular daemons https://review.opendev.org/c/openstack/tripleo-heat-templates/+/803915 | 19:11 |
| dviroel | cloudnull: chandankumar rlandy|rover https://review.opendev.org/c/openstack/python-tripleoclient/+/806993 need more votes here, passing ci already, thanks | 19:24 |
| rlandy|rover | dviroel: thank you - voted | 19:25 |
| lbragstad | cloudnull curious if you think my latest comment here is correct based on our last conversation? https://review.opendev.org/c/openstack/tripleo-heat-templates/+/781571 | 19:28 |
| opendevreview | Merged openstack/puppet-tripleo master: Glance: Add support for cinder_os_region_name https://review.opendev.org/c/openstack/puppet-tripleo/+/805540 | 20:01 |
| opendevreview | Merged openstack/puppet-tripleo master: Remove unused variables in tripleo::profile::base::gnocchi https://review.opendev.org/c/openstack/puppet-tripleo/+/805509 | 20:01 |
| opendevreview | Michele Baldessari proposed openstack/tripleo-heat-templates master: DNM quick override to deploy mariadb on centos9 https://review.opendev.org/c/openstack/tripleo-heat-templates/+/807181 | 20:05 |
| cloudnull | lbragstad re: remaining services, yes that looks right | 20:14 |
| rlandy|rover | cloudnull: hey - could we get your vote/w+ on https://review.opendev.org/c/openstack/python-tripleoclient/+/806993 | 20:34 |
| rlandy|rover | to clear train line | 20:35 |
| *** dviroel is now known as dviroel|out | 21:01 | |
| opendevreview | Brendan Shephard proposed openstack/tripleo-docs master: Update undercloud install for EL8 https://review.opendev.org/c/openstack/tripleo-docs/+/806236 | 21:18 |
| *** bnemec is now known as bnemec-pto | 21:45 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!