rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Bump rdoinfo bobcat-uc tags to latest upper-constraints https://review.rdoproject.org/r/c/rdoinfo/+/48945 | 00:16 |
---|---|---|
rdogerrit | rdo-trunk rdo-trunk proposed deps/python-greenlet c9s-bobcat-rdo: Update to 2.0.2 https://review.rdoproject.org/r/c/deps/python-greenlet/+/48946 | 00:26 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 04:30 |
rdogerrit | Rabi Mishra proposed config master: Add infra-operator to github projects https://review.rdoproject.org/r/c/config/+/48947 | 05:14 |
rdogerrit | Merged config master: Add infra-operator to github projects https://review.rdoproject.org/r/c/config/+/48947 | 05:51 |
*** amoralej|off is now known as amoralej | 06:13 | |
rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: [WIP] [DNM] Early attempt to create new adoption job https://review.rdoproject.org/r/c/rdo-jobs/+/48878 | 06:49 |
rdogerrit | Alfredo Moralejo proposed openstack/edpm-image-builder-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/edpm-image-builder-distgit/+/48948 | 06:51 |
rdogerrit | Marios Andreou proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 06:59 |
rdogerrit | Alfredo Moralejo proposed openstack/ironic-staging-drivers-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ironic-staging-drivers-distgit/+/48949 | 07:05 |
rdogerrit | Alfredo Moralejo proposed openstack/edpm-image-builder-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/edpm-image-builder-distgit/+/48948 | 07:07 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 07:10 |
rdogerrit | Alfredo Moralejo proposed openstack/ironic-staging-drivers-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ironic-staging-drivers-distgit/+/48949 | 07:16 |
rdogerrit | Alfredo Moralejo proposed openstack/edpm-image-builder-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/edpm-image-builder-distgit/+/48948 | 07:19 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 07:19 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 07:22 |
rdogerrit | Alfredo Moralejo proposed openstack/ironic-staging-drivers-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ironic-staging-drivers-distgit/+/48949 | 07:26 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 07:29 |
rdogerrit | Alfredo Moralejo proposed openstack/edpm-image-builder-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/edpm-image-builder-distgit/+/48948 | 07:29 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 07:29 |
jcapitao[m] | amoralej: I left a comment in https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 07:30 |
jcapitao[m] | I reused your piece of code in reno | 07:30 |
amoralej | thanks jcapitao[m], if that works i'm preparing a patch to the releng script | 07:31 |
jcapitao[m] | ack | 07:31 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 07:31 |
amoralej | jcapitao[m], you hit the same issue or just for consistency? | 07:32 |
jcapitao[m] | yes, missing doc/requirements.txt | 07:32 |
amoralej | jcapitao[m], onother that i found is https://review.rdoproject.org/r/c/openstack/ironic-staging-drivers-distgit/+/48949/3/openstack-ironic-staging-drivers.spec#77 | 07:33 |
amoralej | some packages use UPPER_CONSTRAINTS_FILE instead of TOX_CONSTRAINTS_FILE | 07:33 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 07:41 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 07:44 |
jcapitao[m] | ack for CONSTRAINTS_FILE | 07:46 |
jcapitao[m] | I didn't hit that issue so far | 07:46 |
rdogerrit | Alfredo Moralejo proposed openstack/edpm-image-builder-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/edpm-image-builder-distgit/+/48948 | 07:50 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 07:51 |
rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: [WIP] [DNM] Early attempt to create new adoption job https://review.rdoproject.org/r/c/rdo-jobs/+/48878 | 07:52 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 07:58 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 07:59 |
jcapitao[m] | I hit a special case where sphinx extras are set up | 08:00 |
jcapitao[m] | https://github.com/openstack/reno/blob/81587f616f17904336cdc431e25c42b46cd75b8f/tox.ini#L13 | 08:00 |
jcapitao[m] | but looks like only reno project sets it up like this | 08:00 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 08:08 |
amoralej | jcapitao[m], that's installing its sphinx extra requirements | 08:09 |
amoralej | defined in https://github.com/openstack/reno/blob/81587f616f17904336cdc431e25c42b46cd75b8f/setup.cfg#L35 | 08:09 |
amoralej | automatic BR doesn't know how to handle that? | 08:09 |
amoralej | that will force us to add sphinx BR manually i guess | 08:10 |
jcapitao[m] | yeah I guess | 08:10 |
jcapitao[m] | ValueError: Requirement '.[sphinx]' from tox --print-deps-only: py39 is invalid. | 08:10 |
jcapitao[m] | https://logserver.rdoproject.org/51/48951/2/check/DLRN-rpmbuild-centos9/8e2c54b/job-output.txt | 08:10 |
amoralej | yeap, it worked ... | 08:11 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 08:12 |
amoralej | jcapitao[m], karolinku[m] wrt excluded runtime, we need to exclude watcherclient in rally, i.e. but we will need more, i've done this | 08:16 |
amoralej | https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950/6/openstack-rally-plugins.spec#61 | 08:16 |
amoralej | i was wondering if we should add macros by default in the spec as we do with exclued_brs although it's empty | 08:17 |
amoralej | or maybe just add something like this only when needed | 08:17 |
amoralej | wdyt ? | 08:17 |
amoralej | or just one sed per req... dunno | 08:19 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-openstack-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-openstack-distgit/+/48950 | 08:19 |
amoralej | ^ now with a macro for the list of reqs to ban | 08:20 |
karolinku[m] | I thought of creating list of exclusions to script, but... | 08:21 |
karolinku[m] | we will discover them as we process packages, to it is not much faster | 08:21 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 08:22 |
rdogerrit | Alfredo Moralejo proposed openstack/stevedore-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/48952 | 08:24 |
jcapitao[m] | IMHO I prefer the use of excluded_reqs | 08:24 |
jcapitao[m] | it'd ease automation in the future | 08:24 |
rdogerrit | Joel Capitao proposed openstack/requestsexceptions-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/requestsexceptions-distgit/+/48973 | 08:27 |
rdogerrit | Joel Capitao proposed openstack/rsd-lib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rsd-lib-distgit/+/48974 | 08:28 |
rdogerrit | Joel Capitao proposed openstack/sphinx-feature-classification-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/sphinx-feature-classification-distgit/+/48975 | 08:29 |
rdogerrit | Joel Capitao proposed openstack/stackviz-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stackviz-distgit/+/48976 | 08:30 |
karolinku[m] | will there be any future for such transformation?:) | 08:34 |
jcapitao[m] | good question :) | 08:36 |
jcapitao[m] | it's difficult to foresee | 08:36 |
karolinku[m] | will we use it for deps...? | 08:37 |
karolinku[m] | btw, im not against developing the script, just asking:) | 08:38 |
jcapitao[m] | for the deps yes maybe | 08:38 |
jcapitao[m] | or cyclic reqs | 08:38 |
jcapitao[m] | amoralej: btw wrt cyclic reqs, if we already project on bootstrapping CS10, should we handle now ? | 08:40 |
karolinku[m] | if we will add exclusions automatically, there is a chance that we will exlude sth what is needed (not it case of missing package ofc) | 08:41 |
amoralej | yes, for new OSs | 08:41 |
amoralej | lemme send my idea | 08:41 |
amoralej | jcapitao[m], wrt cyclic, yes we will still need it to bootstran new versions of centos or of python | 08:41 |
amoralej | and in some weird cases we may have issues with specific versions requirements | 08:42 |
amoralej | so we may still need repo_bootstrap | 08:42 |
amoralej | we need to be able to boostrap from scratch | 08:42 |
rdogerrit | Alfredo Moralejo proposed rdo-infra/releng master: Some improvements to the upgrade_spec.sh https://review.rdoproject.org/r/c/rdo-infra/releng/+/48977 | 08:46 |
amoralej | karolinku[m], jcapitao[m] ^ | 08:46 |
amoralej | so, when needed (only), we may run upgrade_spec.sh --add-exclude-reqs after --all | 08:46 |
amoralej | with that, at least we have an homogeneous way to implement although only applied when needed | 08:47 |
rdogerrit | Alfredo Moralejo proposed openstack/stevedore-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/48952 | 08:50 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 08:52 |
rdogerrit | Alfredo Moralejo proposed openstack/cliff-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/cliff-distgit/+/48978 | 08:53 |
jcapitao[m] | amoralej: I meant if we should already handle the bootstrapping case now ? | 08:53 |
jcapitao[m] | for example in reno | 08:53 |
amoralej | imo yes | 08:54 |
jcapitao[m] | about your comments on stevedore and cliff | 08:54 |
amoralej | better now than when we need to bootstrap it | 08:54 |
amoralej | or we will be able to find the boostrap order when we need to | 08:54 |
jcapitao[m] | yeah but it's hard to spot all the cyclic req | 08:54 |
amoralej | well, at least the ones that we find | 08:55 |
amoralej | i don't expect many tbh | 08:55 |
amoralej | most of them are already included with repo_bootstrap | 08:56 |
amoralej | the reno case is special | 08:56 |
amoralej | because i'm afraid it will be automatically added as a new BR to many packages | 08:56 |
amoralej | i've seen that in DLRN-rpmbuild-centos9-compare-builddeps jobs in some of the tests | 08:56 |
jcapitao[m] | did I choose the only special case out there for my first package ? xD | 08:56 |
amoralej | yes you did :) | 08:57 |
amoralej | unless we ban it from BRs | 08:57 |
amoralej | which actually i though about | 08:57 |
amoralej | jcapitao[m], i.e. https://logserver.rdoproject.org/63/48363/4/check/DLRN-rpmbuild-centos9-compare-builddeps/b342f34/job-output.txt | 08:59 |
amoralej | oslo-lo | 08:59 |
jcapitao[m] | right | 08:59 |
rdogerrit | Merged rdo-infra/releng master: Some improvements to the upgrade_spec.sh https://review.rdoproject.org/r/c/rdo-infra/releng/+/48977 | 09:02 |
rdogerrit | Alfredo Moralejo proposed openstack/stevedore-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/48952 | 09:02 |
jcapitao[m] | amoralej: ^ let's see if it installs reno | 09:03 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 09:27 |
rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: [WIP] [DNM] Early attempt to create new adoption job https://review.rdoproject.org/r/c/rdo-jobs/+/48878 | 09:31 |
rdogerrit | Alfredo Moralejo proposed openstack/stevedore-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/48952 | 10:11 |
rdogerrit | Alfredo Moralejo proposed openstack/cliff-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/cliff-distgit/+/48978 | 10:13 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 10:19 |
rdogerrit | Karolina Kula proposed openstack/hacking-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/hacking-distgit/+/48980 | 11:13 |
karolinku[m] | amoralej, jcapitao[m]: do you think is it worth to add possiblity of parsing list of packages to process? I mean autmated pull, modify and push changes for this list? | 11:16 |
amoralej | i think we must manually before review | 11:18 |
amoralej | sorry, i mean, we must manually review it before sending it | 11:18 |
amoralej | all the packages i have sent so far have required some manual tweak | 11:19 |
rdogerrit | Karolina Kula proposed openstack/hardware-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/hardware-distgit/+/48981 | 11:20 |
rdogerrit | Enrique Vallespi Gil proposed config master: Add testing kerberos roles for promote-hash role and tripleo-upstream ci-scripts. https://review.rdoproject.org/r/c/config/+/48982 | 11:22 |
amoralej | karolinku[m], said that, i've created a 5 lines script to do all what you said but sending it :) | 11:24 |
karolinku[m] | and you didn't share! LD | 11:27 |
karolinku[m] | :D | 11:27 |
rdogerrit | Enrique Vallespi Gil proposed config master: Add testing kerberos roles for promote-hash role and tripleo-upstream ci-scripts. https://review.rdoproject.org/r/c/config/+/48982 | 11:28 |
rdogerrit | Douglas Viroel proposed rdo-jobs master: Adds FIPS nodesets for IBM BM3 cloud https://review.rdoproject.org/r/c/rdo-jobs/+/48902 | 11:28 |
amoralej | karolinku[m], it's just 5 lines! :) | 11:29 |
rdogerrit | Enrique Vallespi Gil proposed config master: Add testing kerberos roles for promote-hash role and tripleo-upstream ci-scripts. https://review.rdoproject.org/r/c/config/+/48982 | 11:30 |
rdogerrit | Enrique Vallespi Gil proposed config master: Add testing kerberos roles for promote-hash role and tripleo-upstream ci-scripts. https://review.rdoproject.org/r/c/config/+/48982 | 11:31 |
rdogerrit | Alfredo Moralejo proposed openstack/cliff-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/cliff-distgit/+/48978 | 11:33 |
rdogerrit | Karolina Kula proposed openstack/heat-translator-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/heat-translator-distgit/+/48983 | 11:35 |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 11:38 |
rdogerrit | Alfredo Moralejo proposed openstack/stevedore-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stevedore-distgit/+/48952 | 11:38 |
rdogerrit | Alfredo Moralejo proposed openstack/cliff-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/cliff-distgit/+/48978 | 11:44 |
amoralej | karolinku[m], we should make sure there is always a blank space between last BR and next section | 11:46 |
karolinku[m] | ok | 11:46 |
karolinku[m] | just another sed needed | 11:47 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-distgit/+/48984 | 11:48 |
rdogerrit | Joel Capitao proposed openstack/stackviz-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/stackviz-distgit/+/48976 | 11:50 |
rdogerrit | Joel Capitao proposed openstack/rsd-lib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rsd-lib-distgit/+/48974 | 11:56 |
jcapitao[m] | amoralej karolinku : I'm hitting issue with rpmlint "obsolete-not-provided" | 11:58 |
jcapitao[m] | I don't think this warning is relevant for us | 11:58 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-distgit/+/48984 | 11:58 |
jcapitao[m] | I meant for RDO | 11:58 |
jcapitao[m] | https://review.rdoproject.org/zuul/build/0db16e121d524e7cba9a2dcc5a0d82ed | 11:58 |
*** amoralej is now known as amoralej|lunch | 11:58 | |
jcapitao[m] | wdyt ? | 11:59 |
rdogerrit | Karolina Kula proposed openstack/hacking-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/hacking-distgit/+/48980 | 12:06 |
rdogerrit | Karolina Kula proposed openstack/ironic-lib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ironic-lib-distgit/+/48985 | 12:15 |
rdogerrit | Karolina Kula proposed openstack/keystoneauth1-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/keystoneauth1-distgit/+/48986 | 12:19 |
rdogerrit | Joel Capitao proposed rdoinfo master: Retire stackviz project https://review.rdoproject.org/r/c/rdoinfo/+/48987 | 12:22 |
rdogerrit | Merged rdo-jobs master: Adds FIPS nodesets for IBM BM3 cloud https://review.rdoproject.org/r/c/rdo-jobs/+/48902 | 12:24 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Add rpmlint filters for rsd-lib https://review.rdoproject.org/r/c/rdo-jobs/+/48988 | 12:34 |
rdogerrit | Joel Capitao proposed openstack/rsd-lib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rsd-lib-distgit/+/48974 | 12:34 |
rdogerrit | Merged openstack/hardware-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/hardware-distgit/+/48981 | 12:35 |
*** amoralej|lunch is now known as amoralej | 12:36 | |
rdogerrit | Chandan Kumar proposed rdo-jobs master: Added Retry getting k8s resource https://review.rdoproject.org/r/c/rdo-jobs/+/48932 | 12:36 |
rdogerrit | Karolina Kula proposed openstack/keystonemiddleware-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/keystonemiddleware-distgit/+/48989 | 12:37 |
amoralej | jcapitao[m], where are those obsolete coming from? | 12:38 |
amoralej | that may be some obsolete that we don't need anymore? | 12:38 |
jcapitao[m] | it's the other way around | 12:39 |
jcapitao[m] | rpmlint want us to provides obsoletes | 12:39 |
jcapitao[m] | we're hitting the issue for sphinx-feature-classification | 12:40 |
jcapitao[m] | and rsd-lib | 12:40 |
jcapitao[m] | and probably more coming | 12:40 |
amoralej | btw, there seems to be some issue with compare-files in some cases https://logserver.rdoproject.org/85/48985/1/check/DLRN-rpmbuild-centos9-compare-files/a80bb48/job-output.txt | 12:41 |
jcapitao[m] | I'll check out why | 12:43 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Add rpmlint filters for sphinx-feature-classification https://review.rdoproject.org/r/c/rdo-jobs/+/48990 | 12:43 |
rdogerrit | Joel Capitao proposed openstack/sphinx-feature-classification-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/sphinx-feature-classification-distgit/+/48975 | 12:43 |
rdogerrit | Joan Francesc Gilabert proposed rdo-jobs master: [WIP] [DNM] Early attempt to create new adoption job https://review.rdoproject.org/r/c/rdo-jobs/+/48878 | 12:44 |
amoralej | jcapitao[m], see rpm -q --obsoletes -p https://logserver.rdoproject.org/75/48975/1/check/DLRN-rpmbuild-centos9/7d5ef43/buildset/centos9/current/python3-sphinx-feature-classification-1.1.1-0.20230623083350.f8909fc.el9.noarch.rpm | 12:46 |
amoralej | it obsoletes python39-sphinx-feature-classification | 12:46 |
amoralej | i think that's the issue | 12:47 |
amoralej | what creates that? | 12:47 |
rdogerrit | Alfredo Moralejo proposed openstack/rally-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rally-distgit/+/48984 | 12:49 |
jcapitao[m] | ah | 12:49 |
rdogerrit | Marios Andreou proposed rdo-jobs master: Configure OVN database IP address for database backup https://review.rdoproject.org/r/c/rdo-jobs/+/48929 | 12:50 |
rdogerrit | Alfredo Moralejo proposed openstack/automaton-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/automaton-distgit/+/48992 | 12:51 |
jcapitao[m] | and from trunk | 12:52 |
jcapitao[m] | rpm -q --obsoletes -p https://trunk.rdoproject.org/centos9-master/component/common/95/ae/95aea5e50c80770f6ddeccffd2c9fff9a4f70dbf_291f5aa6/python3-sphinx-feature-classification-1.1.0-0.20210921151659.d3d6887.el9.noarch.rpm | 12:52 |
jcapitao[m] | python-sphinx-feature-classification < 1.1.0-0.20210921151659.d3d6887.el9 | 12:52 |
jcapitao[m] | amoralej: what I understand is that rpmlint wants an explicit obsoleted in SPEC file ? | 12:54 |
amoralej | usually, when you obsolete something you also provides it | 12:54 |
amoralej | it's part of upgrade path usual implementation | 12:55 |
amoralej | i thing obsolete-not-provided means, it's obsoleting a package but it does not have a provides for it | 12:55 |
amoralej | it can be a problem or not, depending on the case | 12:56 |
jcapitao[m] | ok we doesn't provide python39-sphinx-feature-classification | 12:57 |
jcapitao[m] | but python3.9-* | 12:57 |
jcapitao[m] | python3.9dist | 12:57 |
jcapitao[m] | python3dist | 12:57 |
jcapitao[m] | as it's handles automatically, I don't think this "obsolete-not-provided " condition is necessary | 12:58 |
jcapitao[m] | s/handles/handled/ | 12:58 |
rdogerrit | Alfredo Moralejo proposed openstack/automaton-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/automaton-distgit/+/48992 | 13:11 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Do not filter on reponame https://review.rdoproject.org/r/c/rdo-jobs/+/48993 | 13:12 |
jcapitao[m] | amoralej: ^ to fix the issue of the compare-files job | 13:13 |
rdogerrit | Joel Capitao proposed rdo-jobs master: Add rpmlint filters for requestsexceptions https://review.rdoproject.org/r/c/rdo-jobs/+/48994 | 13:15 |
amoralej | jcapitao[m], wrt obsolete issue, so the automation provides python3.9- and obsoletes python39- right? | 13:15 |
jcapitao[m] | yes | 13:15 |
amoralej | but we have the same issue before the patch | 13:15 |
amoralej | right? | 13:15 |
amoralej | s/issue/behaviour/ | 13:15 |
amoralej | i.e. | 13:16 |
amoralej | pm -q --obsoletes -p https://trunk.rdoproject.org/centos9-master/component/common/72/e8/72e81d7d846a8e306fd8d98e2f15270a8a537620_622acb7a/python3-cliff-4.3.0-0.20230606074914.72e81d7.el9.noarch.rpm | 13:16 |
jcapitao[m] | yes | 13:16 |
amoralej | rpm -q --provides -p https://trunk.rdoproject.org/centos9-master/component/common/72/e8/72e81d7d846a8e306fd8d98e2f15270a8a537620_622acb7a/python3-cliff-4.3.0-0.20230606074914.72e81d7.el9.noarch.rpm | 13:16 |
jcapitao[m] | dunno why we didn't hit the issue before | 13:16 |
jcapitao[m] | that's the same behavior for cliff | 13:17 |
amoralej | note that rpmlint is just comparing rpmlint | 13:17 |
amoralej | from old and new | 13:17 |
amoralej | i see last build from sphinx-feature-classification in trunk is 20210921151659 | 13:18 |
amoralej | so probably the macros have changed | 13:18 |
amoralej | since then | 13:18 |
jcapitao[m] | yeah but I mean in previous build | 13:18 |
jcapitao[m] | it must have compared with previous one | 13:19 |
amoralej | i'm not sure why it obsoletes python39... but yes, seems good to ignore | 13:19 |
jcapitao[m] | it was relevant when the job was done manually | 13:19 |
jcapitao[m] | but now .. | 13:20 |
jcapitao[m] | https://review.rdoproject.org/r/c/rdo-jobs/+/48994 | 13:20 |
jcapitao[m] | it's my 3rd patch to ignore them | 13:20 |
amoralej | wrt https://review.rdoproject.org/r/c/rdo-jobs/+/48993 some packages may not be noarch ... | 13:21 |
amoralej | mmm there may be some way to not split the output in two lines ? | 13:22 |
jcapitao[m] | yeah that was my interrogation | 13:22 |
amoralej | well, tbh, probably all of them are noarch. We have some arched but only in deps, iirc | 13:22 |
jcapitao[m] | I tried but haven't found a way | 13:22 |
amoralej | what about using repoquery instead | 13:23 |
amoralej | you have some dnf command example with a logs repo? | 13:24 |
jcapitao[m] | PARENT_JOB_REPO=https://logserver.rdoproject.org/85/48985/1/check/DLRN-rpmbuild-centos9/9179a83/buildset/centos9/current/ | 13:24 |
jcapitao[m] | dnf -q --releasever 9 --installroot=/tmp/root --disablerepo='*' --enablerepo='DLRN-rpmbuild-centos9' --repofrompath=DLRN-rpmbuild-centos9,$PARENT_JOB_REPO list available | 13:24 |
amoralej | dnf repoquery --disablerepo='*' --enablerepo='DLRN-rpmbuild-centos9' --repofrompath=DLRN-rpmbuild-centos9,$REPOPATH -q | 13:27 |
amoralej | jcapitao[m], ^ | 13:27 |
amoralej | is that what you need ? | 13:27 |
amoralej | mmm we need to remove the epoch i guess | 13:28 |
jcapitao[m] | repoquery -q --releasever 9 --installroot=/tmp/root --disablerepo='*' --enablerepo='DLRN-rpmbuild-centos9' --repofrompath=DLRN-rpmbuild-centos9,$PARENT_JOB_REPO | grep -v -e "src$" | rev | cut -d- -f3- | re | 13:28 |
amoralej | you want the package name only, right? | 13:28 |
amoralej | better to use --qf | 13:28 |
amoralej | but we need to remove the src ... | 13:29 |
jcapitao[m] | yeah .. | 13:29 |
amoralej | dnf repoquery --disablerepo='*' --enablerepo='DLRN-rpmbuild-centos9' --repofrompath=DLRN-rpmbuild-centos9,$REPOPATH --arch noarch --qf %{name} | 13:29 |
jcapitao[m] | ah! | 13:30 |
amoralej | dnf repoquery --disablerepo='*' --enablerepo='DLRN-rpmbuild-centos9' --repofrompath=DLRN-rpmbuild-centos9,$REPOPATH --arch noarch --arch x86_64 --qf %{name} | 13:30 |
amoralej | just in case we have some arched one | 13:30 |
jcapitao[m] | missing -q | 13:30 |
jcapitao[m] | perfect | 13:30 |
amoralej | and if we want nvr (without epoch) in some case --nvr instead of --qf ... | 13:31 |
amoralej | remember you need to install the dnf-plugins-core for repoquery | 13:31 |
jcapitao[m] | we need the name only in order to pull from trunk after | 13:32 |
jcapitao[m] | ah right | 13:32 |
jcapitao[m] | thanks for the reminder | 13:32 |
jcapitao[m] | amoralej: on my CS8 container, dnf repoquery works out-of-the-box | 13:35 |
jcapitao[m] | I don't have dnf-plugins-core installed | 13:35 |
jcapitao[m] | but repoquery is provided by yum-utils | 13:36 |
jcapitao[m] | let's go with "dnf repoquery" | 13:36 |
rdogerrit | Karolina Kula proposed openstack/heat-translator-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/heat-translator-distgit/+/48983 | 13:36 |
rdogerrit | Joel Capitao proposed rdo-jobs master: List packages with dnf repoquery https://review.rdoproject.org/r/c/rdo-jobs/+/48993 | 13:37 |
amoralej | doesn't yum-utills pulls the plugins core ? | 13:38 |
rdogerrit | Joel Capitao proposed openstack/requestsexceptions-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/requestsexceptions-distgit/+/48973 | 13:38 |
amoralej | ah right! repoquery is not in that plugins package | 13:39 |
amoralej | actually it's part of dnf itself | 13:40 |
rdogerrit | Merged rdo-jobs master: Add rpmlint filters for rsd-lib https://review.rdoproject.org/r/c/rdo-jobs/+/48988 | 13:46 |
rdogerrit | Merged rdo-jobs master: Add rpmlint filters for sphinx-feature-classification https://review.rdoproject.org/r/c/rdo-jobs/+/48990 | 13:47 |
rdogerrit | Merged rdo-jobs master: Add rpmlint filters for requestsexceptions https://review.rdoproject.org/r/c/rdo-jobs/+/48994 | 13:47 |
amoralej | jcapitao[m], i left a comment in https://review.rdoproject.org/r/c/rdo-jobs/+/48993 | 13:47 |
rdogerrit | Alfredo Moralejo proposed openstack/castellan-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/castellan-distgit/+/48995 | 13:55 |
rdogerrit | Alfredo Moralejo proposed openstack/ceilometermiddleware-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ceilometermiddleware-distgit/+/48996 | 13:57 |
rdogerrit | Joel Capitao proposed rdo-jobs master: List packages with dnf repoquery https://review.rdoproject.org/r/c/rdo-jobs/+/48993 | 14:02 |
rdogerrit | Alfredo Moralejo proposed openstack/cinderlib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/cinderlib-distgit/+/48997 | 14:03 |
jcapitao[m] | amoralej: right, I was too fast | 14:03 |
jcapitao[m] | last PS should be good | 14:03 |
amoralej | :) | 14:03 |
amoralej | let's test it with a depends-on | 14:03 |
rdogerrit | Joel Capitao proposed rdo-jobs master: List packages with dnf repoquery https://review.rdoproject.org/r/c/rdo-jobs/+/48993 | 14:03 |
rdogerrit | Joel Capitao proposed openstack/rsd-lib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rsd-lib-distgit/+/48974 | 14:04 |
rdogerrit | Alfredo Moralejo proposed openstack/castellan-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/castellan-distgit/+/48995 | 14:04 |
jcapitao[m] | testing with ^ | 14:04 |
amoralej | cool | 14:05 |
jcapitao[m] | amoralej: karolinku : do you think we can ignore -test subpkg in script easily ? | 14:07 |
amoralej | you mean to not remove the requires for -tests subpackages? | 14:07 |
jcapitao[m] | yes | 14:07 |
amoralej | i couldn't figure out a clean way | 14:08 |
amoralej | the only way would be to find line numbers for the starting and end of the -tests block and ignore that range from the sed for Requires | 14:08 |
jcapitao[m] | yep somehting like this | 14:09 |
rdogerrit | Alfredo Moralejo proposed openstack/collectd-gnocchi-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/collectd-gnocchi-distgit/+/48998 | 14:11 |
jcapitao[m] | amoralej: karolinku: note that testrepository and os-testr were replaced by stestr long time ago | 14:12 |
jcapitao[m] | so the removal of this BRs spotted by the job are ok | 14:12 |
amoralej | yep, actually, that shows some differences in buildreqs job | 14:13 |
jcapitao[m] | I hitted this for some EPEL9 build | 14:13 |
amoralej | jcapitao[m], so you think that we should also update requires for -tests to reflect it? | 14:13 |
amoralej | i'm just leaving them as-is, tbh i didn't check what is there at all | 14:14 |
jcapitao[m] | if we move them to extra we won't have to | 14:14 |
jcapitao[m] | yes let's handle them after | 14:15 |
rdogerrit | Alfredo Moralejo proposed openstack/castellan-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/castellan-distgit/+/48995 | 14:15 |
amoralej | that was my thought | 14:15 |
jcapitao[m] | there is also mock spotted as missing | 14:18 |
jcapitao[m] | most of the projects have moved to unittest.mock | 14:18 |
rdogerrit | Alfredo Moralejo proposed openstack/castellan-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/castellan-distgit/+/48995 | 14:25 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 14:29 |
rdogerrit | Alfredo Moralejo proposed openstack/collectd-gnocchi-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/collectd-gnocchi-distgit/+/48998 | 14:31 |
rdogerrit | Joel Capitao proposed openstack/rsd-lib-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/rsd-lib-distgit/+/48974 | 14:32 |
rdogerrit | Alfredo Moralejo proposed openstack/castellan-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/castellan-distgit/+/48995 | 14:34 |
rdogerrit | Joel Capitao proposed openstack/reno-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951 | 14:38 |
jcapitao[m] | amoralej: https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951/comments/bf399afd_491fc1bd | 14:41 |
jcapitao[m] | I think we are good for reno no ? | 14:42 |
rdogerrit | Alfredo Moralejo proposed openstack/ceilometermiddleware-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ceilometermiddleware-distgit/+/48996 | 14:42 |
rdogerrit | Joel Capitao proposed openstack/sphinx-feature-classification-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/sphinx-feature-classification-distgit/+/48975 | 14:44 |
amoralej | jcapitao[m], did you try with with_docs 1 ? | 14:45 |
jcapitao[m] | yes it's failing | 14:45 |
amoralej | ack | 14:45 |
jcapitao[m] | amoralej: tested with https://review.rdoproject.org/r/c/openstack/reno-distgit/+/48951/4..5 | 14:47 |
jcapitao[m] | the failure is https://review.rdoproject.org/zuul/build/0de05df6007a4e729bff1ee769e36fa8 | 14:47 |
jcapitao[m] | dulwich can't find a git repo | 14:47 |
jcapitao[m] | dulwich aims to load a git repo in python object | 14:48 |
amoralej | ok, good to keep disabled | 14:48 |
rdogerrit | Alfredo Moralejo proposed openstack/collectd-gnocchi-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/collectd-gnocchi-distgit/+/48998 | 14:48 |
amoralej | jcapitao[m], karolinku[m] i think it has been a good first day of spec updates :) | 15:00 |
amoralej | i'll start reviewing the ones passing ci on monday | 15:00 |
jcapitao[m] | yeah :) | 15:01 |
jcapitao[m] | we were optimistic in our time estimation x) | 15:02 |
amoralej | yep :) | 15:03 |
amoralej | anyway, i also think we will be faster as go progress | 15:03 |
rdogerrit | Alfredo Moralejo proposed openstack/ironic-staging-drivers-distgit rpm-master: Switch to pyproject-rpm-macros https://review.rdoproject.org/r/c/openstack/ironic-staging-drivers-distgit/+/48949 | 15:06 |
rdogerrit | Merged rdo-jobs master: List packages with dnf repoquery https://review.rdoproject.org/r/c/rdo-jobs/+/48993 | 15:09 |
*** amoralej is now known as amoralej|off | 17:15 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!