| rdogerrit | rdo-trunk proposed rdoinfo master: Promote CBS tags update for antelope-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/47785 | 00:20 |
|---|---|---|
| rdogerrit | Chandan Kumar proposed config master: [Network Isolation] Add nncp interface during openstack installation https://review.rdoproject.org/r/c/config/+/47954 | 04:32 |
| rdogerrit | Merged config master: [Network Isolation] Add nncp interface during openstack installation https://review.rdoproject.org/r/c/config/+/47954 | 04:38 |
| *** amoralej|off is now known as amoralej | 07:16 | |
| rdogerrit | Marios Andreou proposed rdo-jobs master: DNM testing data plane adoption jobs https://review.rdoproject.org/r/c/rdo-jobs/+/47955 | 08:20 |
| *** jpena|off is now known as jpena | 08:23 | |
| amoralej | jcapitao[m], karolinku[m] after some dummy commits for centos9-antelope we have all the packages in the new components in centos9-master | 08:30 |
| amoralej | i mean, wrt tripleo removal | 08:31 |
| amoralej | but we still have the tripleo component listed in https://trunk.rdoproject.org/centos9-antelope/current/delorean.repo | 08:31 |
| amoralej | it's related to how dlrn finds out the components, which is from the builds database | 08:33 |
| amoralej | https://github.com/softwarefactory-project/DLRN/blob/master/dlrn/utils.py#L340-L349 | 08:33 |
| amoralej | i see two options, modify the db manually to change the components to the new ones for all the packages which stays in centos9-antelope and remove the tripleo ones | 08:35 |
| amoralej | that's the first | 08:35 |
| amoralej | the second one would be to fix dlrn to create the component list from rdoinfo instead of from the actual builds | 08:36 |
| amoralej | the problem with second one are the cases where the component repo still not exist as in bootstrap, we'd have to handle that | 08:37 |
| amoralej | you have any preference? | 08:38 |
| rdogerrit | rdo-trunk proposed openstack/tripleoclient-distgit zed-rdo: python-tripleoclient: failed to build 573a8cad https://review.rdoproject.org/r/c/openstack/tripleoclient-distgit/+/47956 | 08:39 |
| karolinku[m] | will this manual change in db done once now? or again with any new release? | 08:39 |
| amoralej | mmm we'll face similar issue when we remove tripleo from master and zed | 08:39 |
| jcapitao[m] | I prefer to keep querying the db | 08:40 |
| amoralej | would be only for the cases where we need to remove a component from a running dlrn instance | 08:40 |
| amoralej | other option would be to boostrap the instance from scratch | 08:40 |
| amoralej | but i don't like that one | 08:40 |
| amoralej | removing componenets totally is a rare action, but in this case we have to do it in master and antelope | 08:41 |
| amoralej | for tripleo | 08:41 |
| jcapitao[m] | we could create the mysql script and review it before applying | 08:42 |
| amoralej | other option would be to mix both | 08:42 |
| karolinku[m] | question is if changing dlrn handling behavior is worth of effort now | 08:42 |
| amoralej | query the db and see if that component is assigned to any package in rdoinfo | 08:42 |
| amoralej | that would save as of a good part of the db modification | 08:43 |
| amoralej | we wouldn't need to delete old builds, which is something i don't like | 08:43 |
| jcapitao[m] | amoralej: why would we need to delete old builds in second option (i.e to fix dlrn to create the component list from rdoinfo instead of from the actual builds) ? | 08:53 |
| amoralej | no, in that case we wouldn't need to delete them but in that case we'd new to carefully handle the bootstrap | 08:54 |
| amoralej | or other cases | 08:54 |
| amoralej | when there are no builds for a component | 08:54 |
| amoralej | mixing both, extracting the list from the db + checking if any package is associated to it in rdoinfo would actually have the best of both | 08:55 |
| amoralej | i'd say | 08:55 |
| amoralej | wouldn't list empty components and would list only the ones which are currently assigned to the release in rdoinfo | 08:56 |
| amoralej | or the other way around, get the list from rdoinfo and compare to the actual builds in db | 08:56 |
| jcapitao[m] | ok I got the point now | 08:59 |
| amoralej | i think that's the best we can do, we still need to be carefully to force a rebuild of each modified component after merging it in rdoinfo | 09:01 |
| amoralej | or we will have inconsistent current/delorean.repo | 09:01 |
| amoralej | (missing the removed component but packages not in the new ones) | 09:01 |
| amoralej | but still much easier to manage | 09:01 |
| amoralej | note that we don't need to force a rebuild for each moved package, only one per component | 09:02 |
| amoralej | as the list of packages on each component is extracted from rdoinfo | 09:02 |
| jcapitao[m] | yes, much easier | 09:08 |
| jcapitao[m] | I can try to implement it | 09:09 |
| amoralej | i'm working on it ... | 09:09 |
| jcapitao[m] | ok good | 09:09 |
| jcapitao[m] | I'll review it then | 09:09 |
| rdogerrit | Chandan Kumar proposed rdo-jobs master: Add cs9 based crc nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/47957 | 10:03 |
| rdogerrit | Chandan Kumar proposed rdo-jobs master: Add cs9 based crc nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/47957 | 10:08 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: DNM test only https://review.rdoproject.org/r/c/rdoinfo/+/47958 | 10:48 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/rdo-jobs/+/47959 | 11:13 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/rdo-jobs/+/47959 | 11:14 |
| rdogerrit | rdo-trunk rdo-trunk proposed openstack/designate-distgit yoga-rdo: openstack-designate-14.0.2-1 https://review.rdoproject.org/r/c/openstack/designate-distgit/+/47960 | 11:40 |
| rdogerrit | Alfredo Moralejo proposed config master: Fix some rdoinfo jobs after moving master to bobcat tags https://review.rdoproject.org/r/c/config/+/47961 | 11:46 |
| amoralej | karolinku[m], jcapitao[m] ^ if you can check to get rdoinfo-DLRN-check-centos9 running on antelope.yml and bobcat-uc.yml | 11:47 |
| amoralej | karolinku[m], jcapitao[m] https://pypi.org/project/rdopkg/#history we have new release of rdopkg in pypi with the new rdopkg release | 12:00 |
| amoralej | we can start using it in ci to guess the upstream branch | 12:00 |
| rdogerrit | Karolina Kula proposed config master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/config/+/47962 | 12:10 |
| rdogerrit | Karolina Kula proposed rdo-jobs master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/rdo-jobs/+/47959 | 12:11 |
| rdogerrit | Marios Andreou proposed rdo-jobs master: Add failed_when to nested make ansible for data plane adoption jobs https://review.rdoproject.org/r/c/rdo-jobs/+/47955 | 12:30 |
| tkajinam | https://mirror.stream.centos.org/SIGs/9-stream/cloud/x86_64/ | 12:39 |
| tkajinam | do we know what the path in centos sig repository look like for 2023.1 release ? is it openstack-antelope or openstack-2023.1 ? | 12:40 |
| amoralej | openstack-antelope | 12:40 |
| tkajinam | amoralej, thanks ! | 12:40 |
| amoralej | path is aligned with the cbs tags which we chose to keep using friendly names | 12:41 |
| jcapitao[m] | amoralej: ah! I wanted to have https://softwarefactory-project.io/r/c/rdopkg/+/27600 merged before bumping release in PyPI | 12:42 |
| amoralej | that will be a new one :) | 12:43 |
| jcapitao[m] | yeah not a big deal tbh | 12:43 |
| amoralej | i wanted to have the new releases feature asap as i've realized we are still using the wrong branch in the u-c job | 12:43 |
| amoralej | https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/rdoinfo/upper-constraints-update/run.yaml#L31 | 12:45 |
| jcapitao[m] | right, let's fix this | 12:55 |
| jcapitao[m] | and previous patches as well | 12:55 |
| rdogerrit | Merged config master: Fix some rdoinfo jobs after moving master to bobcat tags https://review.rdoproject.org/r/c/config/+/47961 | 12:56 |
| jcapitao[m] | amoralej: are you ? working on fixing https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/rdoinfo/upper-constraints-update/run.yaml#L31 with new rdopkg version ? | 12:58 |
| amoralej | no | 13:00 |
| jcapitao[m] | lemme propose a patch | 13:02 |
| jcapitao[m] | or maybe karolinku if you have a timeslot ? | 13:02 |
| rdogerrit | Alfredo Moralejo proposed config master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/config/+/47962 | 13:03 |
| amoralej | karolinku[m] ^ i guess you need the [m] to get her notified | 13:04 |
| karolinku[m] | nope, it's ok | 13:05 |
| amoralej | so matrix is smarter that i thought :) | 13:05 |
| *** amoralej is now known as amoralej|lunch | 13:06 | |
| jcapitao[m] | I'm using the autocompletion of Matrix/Element client | 13:06 |
| jcapitao[m] | but for the meeting bot it's failing | 13:06 |
| karolinku[m] | I can propose it | 13:12 |
| rdogerrit | Joel Capitao proposed config master: Create antelope branch for dependencies https://review.rdoproject.org/r/c/config/+/47901 | 13:14 |
| jcapitao[m] | karolinku: ack, I'll work on updating rdo-jobs then | 13:23 |
| jcapitao[m] | I mean reverting/updating this https://review.rdoproject.org/r/c/rdo-jobs/+/47878 | 13:24 |
| jcapitao[m] | karolinku: I reviewed https://review.rdoproject.org/r/c/config/+/47962 | 13:24 |
| jcapitao[m] | I put -1 for visibility, it's about extra definition we can remove as well | 13:25 |
| rdogerrit | Karolina Kula proposed config master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/config/+/47962 | 13:29 |
| karolinku[m] | jcapitao, done! | 13:29 |
| jcapitao[m] | thanks ! | 13:35 |
| jcapitao[m] | tkajinam: speaking of release name, do you know what would be the next branch name for p-o-i project ? stable/antelope or stable/2023.1 ? | 13:36 |
| tkajinam | it should be stable/2023.1 | 13:37 |
| tkajinam | unless we request exceptions | 13:37 |
| jcapitao[m] | ok, then we'll have to support it our weirdo jobs | 13:38 |
| jcapitao[m] | https://github.com/rdo-infra/rdo-jobs/blob/ca22912abafeca8daa3415fefa38d871c70da0f9/playbooks/weirdo/dlrn/run.yaml#L123 | 13:38 |
| jcapitao[m] | I'll work on it | 13:39 |
| jcapitao[m] | thanks | 13:39 |
| tkajinam | np and thank you, too ! | 13:39 |
| rdogerrit | Karolina Kula proposed config master: Fix branch name and uc file name in u-c job https://review.rdoproject.org/r/c/config/+/47963 | 13:49 |
| *** amoralej|lunch is now known as amoralej | 14:05 | |
| rdogerrit | Merged config master: Remove tripleo jobs in RDO and rdoinfo https://review.rdoproject.org/r/c/config/+/47962 | 14:24 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Take advantage of the new rdoinfo identifier metadata https://review.rdoproject.org/r/c/rdo-jobs/+/47964 | 14:28 |
| rdogerrit | Joel Capitao proposed config master: Fix branch name and uc file name in u-c job https://review.rdoproject.org/r/c/config/+/47963 | 14:29 |
| jcapitao[m] | amoralej: with karolinku we agreed to work on this feature https://issues.redhat.com/projects/RDO/issues/RDO-114 | 14:38 |
| jcapitao[m] | as you proposed long time ago https://review.rdoproject.org/r/c/gating_scripts/+/45644/comments/729c50b8_df6b1c0c | 14:38 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Take advantage of the new rdoinfo identifier metadata https://review.rdoproject.org/r/c/rdo-jobs/+/47964 | 14:41 |
| amoralej | jcapitao[m], yes, that's second phase :) | 14:43 |
| amoralej | i'd propose rdopkg release -s <status> | 14:44 |
| amoralej | i.e. rdopkg release -s development | 14:44 |
| amoralej | ah, that's the same we put in the ticket :) | 14:44 |
| amoralej | karolinku[m], jcapitao[m] wrt tripleo cleanup from current/delorean.repo and versions.csv https://softwarefactory-project.io/r/c/DLRN/+/27841/ | 14:45 |
| amoralej | with that, we don't need to manipulate anything in the db | 14:45 |
| jcapitao[m] | I'm checking it | 14:58 |
| rdogerrit | Alfredo Moralejo proposed nfvinfo master: Promote ovn22.12-22.12.0-34.el9s to nfv sig testing repo https://review.rdoproject.org/r/c/nfvinfo/+/47965 | 16:26 |
| *** amoralej is now known as amoralej|off | 16:42 | |
| *** jpena is now known as jpena|off | 17:26 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!