| rdogerrit | Jaromír Wysoglad proposed rdo-jobs master: Add telemetry tests to adoption job https://review.rdoproject.org/r/c/rdo-jobs/+/54044 | 07:00 |
|---|---|---|
| rdogerrit | Merged rdo-infra/weirdo master: Stop removing and disabling NetworkManager https://review.rdoproject.org/r/c/rdo-infra/weirdo/+/55030 | 07:00 |
| rdogerrit | Merged rdo-infra/weirdo master: Remove tasks specific for CentOS Stream 8 https://review.rdoproject.org/r/c/rdo-infra/weirdo/+/55031 | 07:00 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Create propose-packstack-modules-update job https://review.rdoproject.org/r/c/rdo-jobs/+/52078 | 07:05 |
| rdogerrit | Daniel Pawlik proposed config master: Retry executing oc command instead of pause https://review.rdoproject.org/r/c/config/+/55038 | 07:25 |
| rdogerrit | Daniel Pawlik proposed config master: Retry executing oc command instead of pause https://review.rdoproject.org/r/c/config/+/55038 | 07:25 |
| rdogerrit | Merged config master: Retry executing oc command instead of pause https://review.rdoproject.org/r/c/config/+/55038 | 07:29 |
| amoralej | jcapitao[m], karolinku[m] may I get reviews on https://review.rdoproject.org/r/c/rdoinfo/+/55033 ? | 07:35 |
| jcapitao[m] | sure | 07:37 |
| jcapitao[m] | tracked with https://issues.redhat.com/browse/RDO-384 | 07:39 |
| jcapitao[m] | amoralej: they were packaged already in the past ? | 07:40 |
| amoralej | I don't think so | 07:41 |
| jcapitao[m] | ok | 07:41 |
| amoralej | i did some local builds and apparently they do not need any additional requirement | 07:41 |
| jcapitao[m] | I'll do the license check | 07:41 |
| jcapitao[m] | or karolinku do you want to take care ? | 07:42 |
| amoralej | i can take care of the initial specs, etc | 07:43 |
| karolinku[m] | license check? not really 😅 | 07:44 |
| jcapitao[m] | well, there is a command to check the license ofc :) | 07:49 |
| jcapitao[m] | I'll do it | 07:50 |
| karolinku[m] | rly? | 07:59 |
| karolinku[m] | fedora one? | 07:59 |
| jcapitao[m] | yes fedora one "licensecheck" | 08:00 |
| jcapitao[m] | karolinku: https://review.rdoproject.org/r/c/rdoinfo/+/55033 | 08:05 |
| jcapitao[m] | so basically it's just running the command (cf my comment in the JIRA) | 08:05 |
| jcapitao[m] | and to ensure they are all open-source | 08:06 |
| jcapitao[m] | I'll let you +W if it's ok for you too | 08:07 |
| rdogerrit | Joel Capitao proposed config master: Create propose-packstack-modules-update job https://review.rdoproject.org/r/c/config/+/55040 | 08:42 |
| rdogerrit | Joel Capitao proposed config master: Create propose-packstack-modules-update job https://review.rdoproject.org/r/c/config/+/55040 | 08:51 |
| rdogerrit | Ananya Banerjee proposed rdo-jobs master: Adds periodic-adoption-multinode-to-crc-no-ceph-rollback job to criteria https://review.rdoproject.org/r/c/rdo-jobs/+/55041 | 09:21 |
| rdogerrit | Merged rdoinfo master: Add python-watcherclient and openstack-watcher-ui https://review.rdoproject.org/r/c/rdoinfo/+/55033 | 11:27 |
| rdogerrit | Karolina Kula proposed openstack/openstackclient-distgit dalmatian-rdo: Revert added patches https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55032 | 11:54 |
| rdogerrit | Karolina Kula proposed openstack/openstackclient-distgit rpm-master: Revert added patches https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55042 | 11:58 |
| rdogerrit | Merged rdo-jobs master: Refines the if else condition for cifmw_test_operator_tempest_image_tag https://review.rdoproject.org/r/c/rdo-jobs/+/55034 | 12:14 |
| rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: Reboot compute nodes as workaround for adoption https://review.rdoproject.org/r/c/rdo-jobs/+/55043 | 12:21 |
| rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: Install systemd-container in standalone jobs https://review.rdoproject.org/r/c/rdo-jobs/+/55044 | 12:29 |
| rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: Install systemd-container in standalone jobs https://review.rdoproject.org/r/c/rdo-jobs/+/55044 | 12:30 |
| jcapitao[m] | karolinku: wrt openstackclient patch, actually the problem is the lua condition | 12:50 |
| jcapitao[m] | I just realized | 12:50 |
| rdogerrit | Joel Capitao proposed openstack/openstackclient-distgit rpm-master: Apply the patches strictly below 7.1.3 version https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55045 | 12:51 |
| rdogerrit | Joel Capitao proposed openstack/openstackclient-distgit dalmatian-rdo: Apply the patches strictly below 7.1.3 version https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55046 | 12:52 |
| rdogerrit | Merged openstack/openstackclient-distgit rpm-master: Remove some uneeded BRs for Fedora https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/54996 | 13:13 |
| rdogerrit | rdo-trunk rdo-trunk proposed config master: Create project for watcherclient https://review.rdoproject.org/r/c/config/+/55047 | 13:14 |
| rdogerrit | rdo-trunk rdo-trunk proposed config master: Create project info for watcherclient in RDO https://review.rdoproject.org/r/c/config/+/55048 | 13:14 |
| rdogerrit | rdo-trunk rdo-trunk proposed config master: Add Zuul definitions for watcherclient https://review.rdoproject.org/r/c/config/+/55049 | 13:14 |
| rdogerrit | rdo-trunk rdo-trunk proposed config master: Create project for watcher-dashboard https://review.rdoproject.org/r/c/config/+/55050 | 13:14 |
| rdogerrit | rdo-trunk rdo-trunk proposed config master: Create project info for watcher-dashboard in RDO https://review.rdoproject.org/r/c/config/+/55051 | 13:14 |
| rdogerrit | rdo-trunk rdo-trunk proposed config master: Add Zuul definitions for watcher-dashboard https://review.rdoproject.org/r/c/config/+/55052 | 13:14 |
| karolinku[m] | which problem? I run https://review.rdoproject.org/r/c/rdoinfo/+/55018 depending on my patch | 13:18 |
| rdogerrit | Merged config master: Create project for watcherclient https://review.rdoproject.org/r/c/config/+/55047 | 13:22 |
| rdogerrit | Merged config master: Create project info for watcherclient in RDO https://review.rdoproject.org/r/c/config/+/55048 | 13:22 |
| rdogerrit | Joel Capitao proposed config master: Create project for watcher-dashboard https://review.rdoproject.org/r/c/config/+/55050 | 13:23 |
| rdogerrit | Joel Capitao proposed config master: Create project info for watcher-dashboard in RDO https://review.rdoproject.org/r/c/config/+/55051 | 13:23 |
| rdogerrit | Joel Capitao proposed config master: Add Zuul definitions for watcherclient https://review.rdoproject.org/r/c/config/+/55049 | 13:23 |
| rdogerrit | Joel Capitao proposed config master: Add Zuul definitions for watcher-dashboard https://review.rdoproject.org/r/c/config/+/55052 | 13:23 |
| jcapitao[m] | karolinku: yes but the problem by bumping the version in distgit is that we would update in CloudSIG (buildsys-tags) first before Trunk (tags), but we should not | 13:25 |
| jcapitao[m] | with that https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55046 the DLRN job in rdoinfo patch will work, as the patches won't be applied | 13:26 |
| jcapitao[m] | the packstack and p-o-i jobs will then run | 13:26 |
| jcapitao[m] | if all goes ok, we'll merge it, and then automation will propose patch to update in CloudSIG | 13:27 |
| jcapitao[m] | this is the proper order | 13:27 |
| rdogerrit | Merged config master: Create project for watcher-dashboard https://review.rdoproject.org/r/c/config/+/55050 | 13:54 |
| rdogerrit | Merged config master: Create project info for watcher-dashboard in RDO https://review.rdoproject.org/r/c/config/+/55051 | 13:55 |
| karolinku[m] | jcapitao: but are those patch really needed for all version below 7.1.3, or just 7.1.2 ? | 13:58 |
| jcapitao[m] | just 7.1.2 | 13:59 |
| jcapitao[m] | but the patches adding those patches were backported down to dalmatian only | 13:59 |
| karolinku[m] | ? | 14:02 |
| karolinku[m] | sorry, can you rephrase? I understand that your patch will apply it all version below 7.1.3, all releases | 14:03 |
| jcapitao[m] | ok lemme bring more context | 14:04 |
| jcapitao[m] | so when preparing dalmatian release, we hit issues with openstackclient | 14:04 |
| jcapitao[m] | some changes were needed upstream | 14:05 |
| jcapitao[m] | https://review.opendev.org/c/openstack/python-openstackclient/+/930887?usp=search | 14:06 |
| jcapitao[m] | and https://review.opendev.org/c/openstack/python-openstackclient/+/931031 | 14:06 |
| jcapitao[m] | we didn't want to wait official release of 7.1.3 because we wanted to unblock the situation | 14:07 |
| jcapitao[m] | so we applied those patches at distgit level | 14:07 |
| jcapitao[m] | https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/54830 | 14:07 |
| jcapitao[m] | https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/54853 | 14:07 |
| jcapitao[m] | and backported them in dalmatian-rdo branch only | 14:07 |
| jcapitao[m] | https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/54668 | 14:08 |
| jcapitao[m] | and https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/54901 | 14:08 |
| jcapitao[m] | we added the lua condition in order to not apply the patches when bumping to 7.1.3, but the condition was not correct (i.e it should be strict <) | 14:09 |
| jcapitao[m] | so the situation we are hitting is because of this incorrect condition | 14:10 |
| karolinku[m] | but since it's merged upstream, do we still need patches? | 14:11 |
| jcapitao[m] | that's the tricky part | 14:12 |
| jcapitao[m] | we won't need those patches once https://review.rdoproject.org/r/c/rdoinfo/+/55018 merged | 14:12 |
| jcapitao[m] | but to be able to merge it, we need https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55046 | 14:13 |
| jcapitao[m] | in order to test 7.1.3 in p-o-i and packstack | 14:13 |
| jcapitao[m] | and then, if it pass CI, we'll be able to merge your patch https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55042 | 14:13 |
| jcapitao[m] | and https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55032 | 14:14 |
| karolinku[m] | understood | 14:15 |
| karolinku[m] | merging lua fix | 14:16 |
| jcapitao[m] | 👍️ | 14:17 |
| rdogerrit | Merged config master: Add Zuul definitions for watcherclient https://review.rdoproject.org/r/c/config/+/55049 | 14:18 |
| rdogerrit | Merged config master: Add Zuul definitions for watcher-dashboard https://review.rdoproject.org/r/c/config/+/55052 | 14:18 |
| amoralej | I'd say replication to github is broken, the just created distgits for watcherclient and dashboard do not exist in github.com/rdo-packages | 14:19 |
| jcapitao[m] | amoralej: lemme check the Gerrit logs | 14:22 |
| amoralej | maybe related to mail change too | 14:22 |
| jcapitao[m] | yeah that's my guess too | 14:22 |
| amoralej | mmm openstackclient was updated 5 minutes ago, it seems | 14:23 |
| jcapitao[m] | hmm | 14:23 |
| amoralej | or maybe github ui is missleading, i'm not sure | 14:24 |
| jcapitao[m] | https://github.com/rdo-packages/openstackclient-distgit/commit/4162c3e60a9ce7e2fc74e200d6e3760709a5a309 | 14:24 |
| jcapitao[m] | it's because when you submit a commit in Gerrit, even though it's not merged, the commit is available in Github | 14:25 |
| jcapitao[m] | so the sync is ok | 14:25 |
| amoralej | mmm | 14:25 |
| jcapitao[m] | but for new projects, it's not apparently | 14:25 |
| rdogerrit | Merged openstack/openstackclient-distgit dalmatian-rdo: Apply the patches strictly below 7.1.3 version https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55046 | 14:25 |
| rdogerrit | Merged openstack/openstackclient-distgit rpm-master: Apply the patches strictly below 7.1.3 version https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/55045 | 14:25 |
| amoralej | maybe the problem is in github repo creation, i'm not sure where that is done | 14:26 |
| amoralej | if it's gerrit itself or part of some of the ci jobs | 14:26 |
| jcapitao[m] | amoralej: I see in logs that Gerrit fails to replicate | 14:30 |
| jcapitao[m] | looks like we need to update Gerrit config with new email | 14:30 |
| amoralej | it's weird that keeps working for openstackclient ... | 14:30 |
| jcapitao[m] | I'm checking the Gerrit conf | 14:31 |
| jcapitao[m] | amoralej: replication is working for other project | 14:32 |
| jcapitao[m] | Replication to git@github.com:rdo-infra/review.rdoproject.org-config.git completed in 14427ms, 15014ms delay, 0 retries [CONTEXT PLUGIN="replication" pushOneId="367f28ab" request="SSH" ] | 14:32 |
| amoralej | the error may be because it does not exist | 14:32 |
| jcapitao[m] | but not for the new repo | 14:32 |
| jcapitao[m] | looks like | 14:33 |
| jcapitao[m] | Missing repository created; retry replication to git@github.com:rdo-packages/watcherclient-distgit.git [CONTEXT pushOneId="2f8550ef" ] | 14:33 |
| amoralej | jcapitao[m], https://review.rdoproject.org/zuul/builds?job_name=config-distgit-mirror-github&project=config two jobs failed .... | 14:33 |
| amoralej | i think that's the actual issue | 14:33 |
| amoralej | https://review.rdoproject.org/zuul/build/f30cb3bb5ade4cecb114550ec1f100c3 | 14:34 |
| amoralej | yeah, likely related to the mail change | 14:34 |
| amoralej | we'll need to rerun those | 14:34 |
| jcapitao[m] | lemme check the error before rerunning | 14:36 |
| jcapitao[m] | looks like our token is no longer valid | 14:40 |
| rdogerrit | Luigi Toscano proposed rdo-jobs master: DNM WIP Run swift migration tests post-adoption https://review.rdoproject.org/r/c/rdo-jobs/+/55053 | 14:54 |
| rdogerrit | Alfredo Moralejo proposed openstack/watcherclient-distgit rpm-master: Initial version of spec file. https://review.rdoproject.org/r/c/openstack/watcherclient-distgit/+/55055 | 15:12 |
| rdogerrit | Alfredo Moralejo proposed openstack/watcher-dashboard-distgit rpm-master: Initial version of spec file https://review.rdoproject.org/r/c/openstack/watcher-dashboard-distgit/+/55056 | 15:14 |
| rdogerrit | Francesco Pantano proposed rdo-jobs master: Run TripleO prerequisites before running the adoption test suite https://review.rdoproject.org/r/c/rdo-jobs/+/55057 | 15:16 |
| rdogerrit | Francesco Pantano proposed rdo-jobs master: Run TripleO prerequisites before running the adoption test suite https://review.rdoproject.org/r/c/rdo-jobs/+/55057 | 15:17 |
| rdogerrit | Joel Capitao proposed config master: Update rdo-trunk Github token https://review.rdoproject.org/r/c/config/+/55058 | 15:22 |
| rdogerrit | Joel Capitao proposed config master: Update rdo-trunk Github token https://review.rdoproject.org/r/c/config/+/55058 | 15:23 |
| rdogerrit | Merged config master: Update rdo-trunk Github token https://review.rdoproject.org/r/c/config/+/55058 | 15:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!