| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Bump rdoinfo caracal-uc tags to latest upper-constraints or latest Puppetfile https://review.rdoproject.org/r/c/rdoinfo/+/52009 | 00:22 |
|---|---|---|
| rdogerrit | rdo-trunk rdo-trunk proposed deps/python-pyroute2 c9s-caracal-rdo: Update to 0.7.12 https://review.rdoproject.org/r/c/deps/python-pyroute2/+/52010 | 00:33 |
| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Promote CBS tags update for antelope-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/51427 | 00:36 |
| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Promote CBS tags update for zed-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/52011 | 00:37 |
| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Promote CBS tags update for wallaby-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/51282 | 00:38 |
| rdogerrit | Merged openstack/tempest-distgit rpm-master: Remove sahara-tests and murano-tests https://review.rdoproject.org/r/c/openstack/tempest-distgit/+/52000 | 06:17 |
| rdogerrit | rdo-trunk rdo-trunk proposed puppet/puppet-sahara-distgit rpm-master: puppet-sahara: failed to build 50b3fc6 https://review.rdoproject.org/r/c/puppet/puppet-sahara-distgit/+/52012 | 06:57 |
| rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Remove sahara, murano and senlin clients from requirements https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/51996 | 07:50 |
| rdogerrit | Alfredo Moralejo proposed openstack/openstackclient-distgit rpm-master: Remove sahara, murano and senlin clients from requirements https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/51997 | 07:55 |
| rdogerrit | Joel Capitao proposed deps/python-pyroute2 c9s-caracal-rdo: Update to 0.7.12 https://review.rdoproject.org/r/c/deps/python-pyroute2/+/52010 | 07:59 |
| rdogerrit | Merged openstack/kolla-distgit antelope-rdo: openstack-kolla-16.4.0-1 https://review.rdoproject.org/r/c/openstack/kolla-distgit/+/52006 | 08:07 |
| rdogerrit | Merged openstack/kolla-distgit bobcat-rdo: openstack-kolla-17.3.0-1 https://review.rdoproject.org/r/c/openstack/kolla-distgit/+/52005 | 08:07 |
| rdogerrit | Alfredo Moralejo proposed openstack/mistral-extra-distgit rpm-master: Remove muranoclient and senlinclient from requirements https://review.rdoproject.org/r/c/openstack/mistral-extra-distgit/+/51995 | 08:14 |
| rdogerrit | Merged openstack/rally-openstack-distgit rpm-master: Remove sahara, murano and senlin clients from requirements https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/51996 | 08:15 |
| jcapitao[m] | hello folks | 08:17 |
| jcapitao[m] | may you review https://review.opendev.org/q/I812a90d668ecee95b9cada5818d66ac8cead0a35 in order to speed unblocking of antelope p-o-i promotion | 08:17 |
| amoralej | i just have +1 in those | 08:18 |
| amoralej | tkajinam ^ if you have a while | 08:18 |
| rdogerrit | Merged rdoinfo master: Pin the packages falling back to master in Bobcat https://review.rdoproject.org/r/c/rdoinfo/+/51667 | 08:20 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for zed-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/52011 | 08:20 |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo caracal-uc tags to latest upper-constraints or latest Puppetfile https://review.rdoproject.org/r/c/rdoinfo/+/52009 | 08:20 |
| jcapitao[m] | ack thanks | 08:20 |
| jcapitao[m] | amoralej: wrt https://review.rdoproject.org/r/q/topic:inactive-removal - after the merge of all patches in https://review.rdoproject.org/r/q/topic:inactive-removal , I think we can send rdoinfo patches | 08:21 |
| tkajinam | amoralej, I've added +2. if these are stuck for some time then I'll do +A | 08:21 |
| amoralej | thanks | 08:21 |
| tkajinam | jcapitao[m], thanks for these backports. It has been blocked by multiple centos/puppet problems but these are now green. we should be able to merge these soon, hopefully... | 08:21 |
| jcapitao[m] | amoralej: maybe separated patch for each project (with clients, libs, tempest, etc) ? | 08:22 |
| jcapitao[m] | tkajinam: hopefully, thanks for the heads-up | 08:22 |
| amoralej | tkajinam, sent one for senlin iirc, we can do one for each project, yep | 08:22 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: Remove Senlin from Caracal https://review.rdoproject.org/r/c/rdoinfo/+/51910 | 08:23 |
| jcapitao[m] | right | 08:23 |
| rdogerrit | rdo-trunk rdo-trunk proposed openstack/collectd-gnocchi-distgit bobcat-rdo: python-collectd-gnocchi-1.7.1-1 https://review.rdoproject.org/r/c/openstack/collectd-gnocchi-distgit/+/52013 | 08:24 |
| rdogerrit | Merged openstack/packstack-distgit rpm-master: Remove puppet-sahara requirement https://review.rdoproject.org/r/c/openstack/packstack-distgit/+/52002 | 08:24 |
| jcapitao[m] | FYI I'm preparing the sahara rdoinfo patch | 08:25 |
| rdogerrit | Merged deps/python-pyroute2 c9s-caracal-rdo: Update to 0.7.12 https://review.rdoproject.org/r/c/deps/python-pyroute2/+/52010 | 08:26 |
| rdogerrit | Merged openstack/heat-distgit rpm-master: Remove saharaclient from requirements https://review.rdoproject.org/r/c/openstack/heat-distgit/+/51999 | 08:27 |
| tkajinam | jcapitao[m] amoralej, I'd agree that per-project approach may be simpler. | 08:31 |
| tkajinam | jcapitao[m], one thing you may have to be aware of is that heat may be dependent on saharaclient and monsacaclient now | 08:31 |
| tkajinam | we deprecated the related features this cycle and will remove these in next cycle. so you might have to leave clients in Caracal and drop these in Dalmatian | 08:32 |
| tkajinam | or remove these from heat's dependency though it may need some tweaks in unit tests | 08:32 |
| jcapitao[m] | hmmm https://review.rdoproject.org/r/c/openstack/heat-distgit/+/51999 | 08:34 |
| jcapitao[m] | we don't run unit tests during the build | 08:35 |
| jcapitao[m] | lemme check | 08:35 |
| jcapitao[m] | looks like we've never run unit tests for Heat | 08:39 |
| jcapitao[m] | if we turn on, it will fail, you're right https://github.com/openstack/heat/blob/9da45ab44bb985c633d246ba9132f1ccc5831cfd/heat/tests/clients/test_clients.py#L28 | 08:40 |
| rdogerrit | Merged openstack/collectd-gnocchi-distgit bobcat-rdo: python-collectd-gnocchi-1.7.1-1 https://review.rdoproject.org/r/c/openstack/collectd-gnocchi-distgit/+/52013 | 08:47 |
| rdogerrit | Joel Capitao proposed openstack/tobiko-distgit bobcat-rdo: Dummy commit to force rebuild https://review.rdoproject.org/r/c/openstack/tobiko-distgit/+/52014 | 09:11 |
| rdogerrit | Joel Capitao proposed openstack/os-faults-distgit bobcat-rdo: Dummy commit to force rebuild https://review.rdoproject.org/r/c/openstack/os-faults-distgit/+/52015 | 09:20 |
| jcapitao[m] | amoralej karolinku : as tkajinam said above saharaclient is still used in heat https://github.com/openstack/heat/blob/master/heat/engine/clients/os/sahara.py | 09:51 |
| amoralej | yeah, but i guess it's just optional dep? | 09:52 |
| jcapitao[m] | it fails in %check | 09:52 |
| amoralej | i mean, it's in requirements.txt | 09:52 |
| jcapitao[m] | I enabled back tests locally | 09:52 |
| amoralej | but it's imported only when used ? | 09:52 |
| amoralej | anyway, we can keep saharaclient for a while, but i think its real usage is contained in the plugin | 09:53 |
| jcapitao[m] | I don't think so | 09:54 |
| jcapitao[m] | it's failing during discovery | 09:54 |
| jcapitao[m] | for the other clients it's ok | 09:54 |
| amoralej | which discovery ? | 09:54 |
| amoralej | tests discovery ? | 09:54 |
| amoralej | or actual functional ? | 09:55 |
| jcapitao[m] | yep | 09:55 |
| jcapitao[m] | test discovery | 09:55 |
| amoralej | could you paste logs ? | 09:55 |
| jcapitao[m] | https://paste.centos.org/view/546e2d14 | 09:55 |
| tkajinam | we might be able to just remove the test files and that may be enough | 09:55 |
| amoralej | so, in unit tests, it's not optional, it's testing the sahara client plugin | 09:56 |
| jcapitao[m] | yeah I can remove it in heat/tests/common.py" | 09:56 |
| amoralej | but in real runtime, i think it's optional | 09:56 |
| amoralej | said that, maintaining saharaclient in caracal is not a big problem, i guess | 09:56 |
| amoralej | but the point is that, if sahara server is not running in the cluster, there is not much that heat can do it with it | 09:57 |
| amoralej | so the plugin is useless ... | 09:57 |
| tkajinam | unless keeping sharaclient causes big problem then I may suggest leaving it now (without tweaking heat for now). Once we remove sahara support from heat in 2024.2 then you can drop saharaclient | 09:57 |
| tkajinam | yeah | 09:57 |
| tkajinam | I was dropping the functionality in this cycle but there has been long discussion about "inactive is not retired" kind of arguments | 09:58 |
| jcapitao[m] | ok +1 to follow upstream | 09:58 |
| amoralej | wfm | 10:00 |
| amoralej | the same happens with senlinclient ? | 10:00 |
| amoralej | there are tests for senlin https://github.com/openstack/heat/blob/9da45ab44bb985c633d246ba9132f1ccc5831cfd/heat/tests/clients/test_senlin_client.py | 10:01 |
| jcapitao[m] | for senlin it's ok as I remove this test file | 10:02 |
| amoralej | ok | 10:03 |
| rdogerrit | Joel Capitao proposed openstack/heat-distgit rpm-master: Revert "Remove saharaclient from requirements" https://review.rdoproject.org/r/c/openstack/heat-distgit/+/51879 | 10:04 |
| rdogerrit | Alfredo Moralejo proposed openstack/os-diff-distgit rpm-master: Add os-diff specfile for rdo-package https://review.rdoproject.org/r/c/openstack/os-diff-distgit/+/51560 | 10:17 |
| rdogerrit | Merged openstack/mistral-extra-distgit rpm-master: Remove muranoclient and senlinclient from requirements https://review.rdoproject.org/r/c/openstack/mistral-extra-distgit/+/51995 | 10:19 |
| rdogerrit | Merged openstack/openstackclient-distgit rpm-master: Remove sahara, murano and senlin clients from requirements https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/51997 | 10:20 |
| rdogerrit | Joel Capitao proposed openstack/heat-distgit rpm-master: Enable tests https://review.rdoproject.org/r/c/openstack/heat-distgit/+/52016 | 10:21 |
| rdogerrit | Joel Capitao proposed rdoinfo master: Remove Sahara from Caracal https://review.rdoproject.org/r/c/rdoinfo/+/52017 | 10:30 |
| rdogerrit | Merged openstack/tobiko-distgit bobcat-rdo: Dummy commit to force rebuild https://review.rdoproject.org/r/c/openstack/tobiko-distgit/+/52014 | 10:31 |
| rdogerrit | Merged openstack/os-diff-distgit rpm-master: Add os-diff specfile for rdo-package https://review.rdoproject.org/r/c/openstack/os-diff-distgit/+/51560 | 10:35 |
| rdogerrit | Joel Capitao proposed rdoinfo master: Remove Murano from Caracal https://review.rdoproject.org/r/c/rdoinfo/+/52018 | 10:37 |
| rdogerrit | Merged rdoinfo master: Remove Senlin from Caracal https://review.rdoproject.org/r/c/rdoinfo/+/51910 | 10:46 |
| amoralej | tosky, https://review.rdoproject.org/r/c/rdoinfo/+/52017 bye old sahara days :) | 11:21 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Manage branch name change to unmaintained releases in weirdo jobs https://review.rdoproject.org/r/c/rdo-jobs/+/51921 | 11:36 |
| rdogerrit | Alfredo Moralejo proposed rdo-infra/weirdo master: DNM test only https://review.rdoproject.org/r/c/rdo-infra/weirdo/+/52019 | 11:39 |
| rdogerrit | Joel Capitao proposed rdoinfo master: Remove Murano from Caracal https://review.rdoproject.org/r/c/rdoinfo/+/52018 | 11:39 |
| rdogerrit | Joel Capitao proposed rdoinfo master: Remove Murano from Caracal https://review.rdoproject.org/r/c/rdoinfo/+/52018 | 11:40 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Manage branch name change to unmaintained releases in weirdo jobs https://review.rdoproject.org/r/c/rdo-jobs/+/51921 | 11:49 |
| rdogerrit | Alfredo Moralejo proposed rdo-jobs master: Use upstream_branch to detect branch name in dlrn jobs https://review.rdoproject.org/r/c/rdo-jobs/+/52021 | 12:00 |
| tosky | amoralej: eh :/ | 12:02 |
| rdogerrit | Merged openstack/heat-distgit rpm-master: Revert "Remove saharaclient from requirements" https://review.rdoproject.org/r/c/openstack/heat-distgit/+/51879 | 12:04 |
| rdogerrit | Karolina Kula proposed config master: Create caracal-rdo branch for libs and clients https://review.rdoproject.org/r/c/config/+/52022 | 12:19 |
| rdogerrit | Karolina Kula proposed config master: Create caracal-rdo branch for libs and clients https://review.rdoproject.org/r/c/config/+/52022 | 12:30 |
| *** gthiemon1e is now known as gthiemonge | 13:36 | |
| amoralej | jcapitao[m], karolinku[m] https://review.rdoproject.org/r/c/rdo-jobs/+/51921 is working, tested in https://review.rdoproject.org/r/c/testproject/+/52020 and https://review.rdoproject.org/r/c/rdo-infra/weirdo/+/52019 | 14:10 |
| jcapitao[m] | yes I just merged it | 14:10 |
| amoralej | good | 14:11 |
| jcapitao[m] | but for https://review.rdoproject.org/r/c/rdo-jobs/+/52021 | 14:11 |
| amoralej | thanks | 14:11 |
| jcapitao[m] | did you test it somewhere ? | 14:11 |
| amoralej | nop | 14:11 |
| amoralej | too easy :) | 14:11 |
| jcapitao[m] | :D | 14:12 |
| jcapitao[m] | for some reason mirroring of tobiko distgit to GH does not work | 14:15 |
| jcapitao[m] | https://github.com/rdo-packages/tobiko-distgit | 14:15 |
| jcapitao[m] | https://github.com/rdo-packages/tobiko-distgit/tree/bobcat-rdo | 14:15 |
| amoralej | only tobiko ? | 14:15 |
| amoralej | or all projects? | 14:15 |
| jcapitao[m] | dunno yet | 14:16 |
| amoralej | probably all | 14:16 |
| jcapitao[m] | looks like all | 14:16 |
| amoralej | https://github.com/rdo-packages/mistral-extra-distgit | 14:17 |
| jcapitao[m] | revert of commit heat does not appear | 14:17 |
| amoralej | there was some changes related to users in gh rdo orgs, lemme ask | 14:17 |
| jcapitao[m] | ack | 14:18 |
| jcapitao[m] | karolinku: looks like https://review.rdoproject.org/r/c/rdo-infra/ansible-role-dlrn/+/51998 didn't solve the issue of swift FTBFS | 14:25 |
| karolinku[m] | yes, im on it. looks that configs are taken from different places, for master and rest | 14:25 |
| jcapitao[m] | you mean the best=0 entry ? | 14:28 |
| karolinku[m] | enitre source of dlrn config | 14:30 |
| jcapitao[m] | so ~/dlrn/scripts/centos9-bobcat.cfg was updated in Trunk | 14:38 |
| jcapitao[m] | I see the change | 14:38 |
| jcapitao[m] | but it does not appear in logs https://logserver.rdoproject.org/76/51976/2/check/DLRN-rpmbuild-centos9/c74bb05/job-output.txt | 14:38 |
| jcapitao[m] | so I suspect there is cache mechanism enabled | 14:39 |
| amoralej | dlrn jobs in ci do not use the config from ansible role, but some configs in dlrn repo itself | 14:39 |
| amoralej | https://github.com/softwarefactory-project/DLRN/blob/master/scripts/centos-stream-9.cfg | 14:40 |
| jcapitao[m] | hmm | 14:43 |
| jcapitao[m] | i see data/dlrn-5.cfg contains the change | 14:44 |
| jcapitao[m] | but not data/dlrn-1.cfg | 14:44 |
| amoralej | where ? | 14:44 |
| jcapitao[m] | in centos9-bobcat trunk | 14:44 |
| amoralej | that's fine | 14:44 |
| jcapitao[m] | yes | 14:44 |
| jcapitao[m] | but swift job run selected dlrn-1 | 14:44 |
| amoralej | it's because we have different parameters in the variables | 14:44 |
| amoralej | for cents9-master | 14:45 |
| amoralej | karolinku[m] is sending the patch :) | 14:45 |
| amoralej | we have two different issues | 14:46 |
| amoralej | job failing in CI, and build failing in rdo-trunk | 14:46 |
| amoralej | build failing in rdo-trunk should be fixed now and it should work fine after recheck (dlrn recheck) | 14:47 |
| amoralej | should be fixed with karolinku[m] patch | 14:47 |
| amoralej | but, the ci job in https://review.rdoproject.org/r/c/openstack/swift-distgit/+/51976 does not use the config from ansible-role-dlrn | 14:47 |
| amoralej | but the one in https://github.com/softwarefactory-project/DLRN/blob/master/scripts/centos-stream-9.cfg | 14:47 |
| amoralej | that's wrt bobcat | 14:48 |
| jcapitao[m] | what jobs were failing in CI ? | 14:48 |
| karolinku[m] | sf infra patch: https://softwarefactory-project.io/r/c/software-factory/sf-infra/+/31019 | 14:48 |
| amoralej | i mean DLRN-rpmbuild-centos9 https://review.rdoproject.org/r/c/openstack/swift-distgit/+/51976?tab=change-view-tab-header-zuul-results-summary | 14:48 |
| jcapitao[m] | hmm but it's still failing in DLRN-rpmbuild-centos9 | 14:50 |
| amoralej | yes because https://github.com/softwarefactory-project/DLRN/blob/master/scripts/centos-stream-9.cfg has best=1 | 14:50 |
| jcapitao[m] | yes | 14:50 |
| rdogerrit | Marios Andreou proposed rdo-jobs master: Adds overcloud os-net-config and ssh setup for new data plane adoption job https://review.rdoproject.org/r/c/rdo-jobs/+/51916 | 14:50 |
| jcapitao[m] | but which patch should fix this ? | 14:51 |
| amoralej | a nonexisting yet one :) | 14:51 |
| amoralej | there is no patch for that sent | 14:51 |
| amoralej | but there is patch for the FTBFS in trunk.r.o | 14:51 |
| jcapitao[m] | okk I got it now | 14:52 |
| -opendevstatus- NOTICE: Jobs that fail due to being unable to resolve mirror.dfw.rackspace.opendev.org can be rechecked. This error was an unexpected side effect of some nodepool configuration changes which have been reverted. | 16:55 | |
| rdogerrit | Joel Capitao proposed openstack/kolla-distgit antelope-rdo: Dummy commit to force rebuild https://review.rdoproject.org/r/c/openstack/kolla-distgit/+/52023 | 17:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!