| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Bump rdoinfo zed tags to latest upper-constraints or latest Puppetfile https://review.rdoproject.org/r/c/rdoinfo/+/51657 | 00:51 |
|---|---|---|
| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Promote CBS tags update for antelope-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/51427 | 01:02 |
| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Promote CBS tags update for zed-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/51658 | 01:03 |
| rdogerrit | rdo-trunk rdo-trunk proposed rdoinfo master: Promote CBS tags update for wallaby-8s-testing https://review.rdoproject.org/r/c/rdoinfo/+/51282 | 01:04 |
| rdogerrit | Merged rdoinfo master: Promote CBS tags update for zed-9s-testing https://review.rdoproject.org/r/c/rdoinfo/+/51658 | 06:29 |
| rdogerrit | Merged rdoinfo master: Bump rdoinfo zed tags to latest upper-constraints or latest Puppetfile https://review.rdoproject.org/r/c/rdoinfo/+/51657 | 06:35 |
| rdogerrit | rdo-trunk rdo-trunk proposed openstack/openstackclient-distgit zed-rdo: python-openstackclient-6.0.1-1 https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/51659 | 06:38 |
| rdogerrit | rdo-trunk rdo-trunk proposed openstack/oslo-messaging-distgit zed-rdo: python-oslo-messaging-14.0.3-1 https://review.rdoproject.org/r/c/openstack/oslo-messaging-distgit/+/51660 | 06:38 |
| rdogerrit | rdo-trunk rdo-trunk proposed openstack/sushy-distgit zed-rdo: python-sushy-4.3.4-1 https://review.rdoproject.org/r/c/openstack/sushy-distgit/+/51661 | 06:38 |
| rdogerrit | Merged openstack/oslo-messaging-distgit zed-rdo: python-oslo-messaging-14.0.3-1 https://review.rdoproject.org/r/c/openstack/oslo-messaging-distgit/+/51660 | 07:22 |
| rdogerrit | Merged openstack/sushy-distgit zed-rdo: python-sushy-4.3.4-1 https://review.rdoproject.org/r/c/openstack/sushy-distgit/+/51661 | 07:22 |
| rdogerrit | Merged openstack/openstackclient-distgit zed-rdo: python-openstackclient-6.0.1-1 https://review.rdoproject.org/r/c/openstack/openstackclient-distgit/+/51659 | 07:23 |
| rdogerrit | Chandan Kumar proposed rdo-jobs master: Drop centos-9-crc-single-node-centos-9-standalone nodeset https://review.rdoproject.org/r/c/rdo-jobs/+/51662 | 07:53 |
| amoralej | jcapitao[m], wrt the issue with manila tempest in antelope, maybe we could start by reverting the manila plugin update we did in https://review.rdoproject.org/r/c/rdoinfo/+/51422 | 08:08 |
| jcapitao[m] | I found the root cause | 08:08 |
| amoralej | what's it? | 08:08 |
| jcapitao[m] | https://review.opendev.org/c/openstack/manila-tempest-plugin/+/888618 | 08:08 |
| jcapitao[m] | since this commit it's failing | 08:08 |
| jcapitao[m] | https://review.opendev.org/q/topic:%22bp/allow-locking-shares-against-deletion%22+project:openstack/manila need to be backported in stable/antelope | 08:09 |
| jcapitao[m] | but there is too much conflicts .. | 08:09 |
| jcapitao[m] | IMHO the change is too big to be backported | 08:09 |
| amoralej | and actually, that may go agains stable policy rules | 08:10 |
| amoralej | dunno | 08:10 |
| jcapitao[m] | so I think we need to pin manila plugin to 8d4a2de | 08:10 |
| amoralej | you found any way to test it? | 08:10 |
| amoralej | with DNM i mean | 08:10 |
| jcapitao[m] | the commit before the failing one | 08:10 |
| jcapitao[m] | yeah with my DNM patches from yesterday | 08:10 |
| amoralej | propose it and we will ask someone for reviews | 08:11 |
| jcapitao[m] | the pinning ? | 08:11 |
| amoralej | explain it in the commit message | 08:11 |
| amoralej | yes | 08:11 |
| amoralej | btw, i think we discussed about running p-o-i in u-c updates | 08:11 |
| jcapitao[m] | I was writing my investigation result in https://issues.redhat.com/projects/RDO/issues/RDO-206 | 08:11 |
| amoralej | u-c bumps, i mean | 08:11 |
| jcapitao[m] | yeah | 08:11 |
| jcapitao[m] | at least for antelope... | 08:12 |
| amoralej | ok, but please add some explanation in the commit message too :) | 08:12 |
| jcapitao[m] | we would have caught it | 08:12 |
| amoralej | yep | 08:12 |
| jcapitao[m] | yeah sure, no worries for the commit message ;) | 08:12 |
| amoralej | btw, is there any "official" way to reference jira tickets in commit messages? | 08:13 |
| amoralej | we may keep using the usual "Resolves:" etc ... | 08:13 |
| jcapitao[m] | don't think so | 08:13 |
| amoralej | although there is no automation | 08:13 |
| jcapitao[m] | yeah.. | 08:13 |
| jcapitao[m] | there is for LP because there is automation | 08:13 |
| amoralej | yes, but i think it'd be good to agree in some common way | 08:14 |
| amoralej | karolinku[m] wrt link jira tickets in commit messages | 08:15 |
| amoralej | ^ | 08:15 |
| karolinku[m] | why Resolves: is bad idea? | 08:23 |
| karolinku[m] | long time ago I was using quite nice integratorin for Jira and bitbucket | 08:26 |
| karolinku[m] | maybe there is some plugins to integrate with gerrit | 08:27 |
| amoralej | how worked it? | 08:27 |
| amoralej | using ticket name in commit subject or something like that? | 08:27 |
| amoralej | https://support.atlassian.com/jira-software-cloud/docs/reference-issues-in-your-development-work/ | 08:27 |
| amoralej | that's what i've found | 08:27 |
| karolinku[m] | for gerrit rewiew you had to added some hook name, sth like "Resolves" and it automatically pin it to jira ticket | 08:27 |
| rdogerrit | Joel Capitao proposed rdoinfo master: Pin manila tempest plugin in Antelope https://review.rdoproject.org/r/c/rdoinfo/+/51663 | 08:28 |
| amoralej | https://marketplace.atlassian.com/apps/1210874/gerrit-code-review-for-jira?tab=overview&hosting=server | 08:30 |
| amoralej | that's one | 08:30 |
| amoralej | but dunno if that's available | 08:30 |
| jcapitao[m] | karolinku: you mean it adds the link in the review ? or it update the JIRA ticket with the review URL ? | 08:30 |
| karolinku[m] | second option | 08:30 |
| amoralej | that'd be cool, actually, that's other question i had :) | 08:31 |
| amoralej | we can link manually | 08:31 |
| amoralej | for that plugin, i.e. https://github.com/MeetMe/jira-gerrit-plugin/blob/master/README.md | 08:31 |
| amoralej | they use "Bug:" or "Issue:" footer | 08:31 |
| amoralej | or symply Look for the issue key in the commit message | 08:32 |
| jcapitao[m] | I think infra is using "Issue" | 08:32 |
| amoralej | Issue wfm | 08:35 |
| amoralej | i think, as soon as we agree on something, is good | 08:35 |
| karolinku[m] | are we able to install plugins? | 08:36 |
| jcapitao[m] | I hope so | 08:37 |
| amoralej | i guess nop | 08:37 |
| jcapitao[m] | https://review.rdoproject.org/r/c/rdoinfo/+/51663 | 08:37 |
| jcapitao[m] | if you can review ^ | 08:37 |
| jcapitao[m] | to unblock antelope promotion | 08:38 |
| amoralej | let's wait for tosky review | 08:38 |
| jcapitao[m] | d/s are waiting some changes | 08:38 |
| amoralej | jcapitao[m], https://bugs.launchpad.net/manila/+bug/2051926 is related? | 08:44 |
| amoralej | apparently it comes from discussion | 08:44 |
| amoralej | https://meetings.opendev.org/irclogs/%23rdo/%23rdo.2024-02-01.log.html#t2024-02-01T08:40:45 | 08:45 |
| jcapitao[m] | looks like another issue | 08:47 |
| amoralej | no, i think it's related | 08:47 |
| amoralej | they are proposing to skip tests if the manila server do not provide a certain microviersion | 08:48 |
| amoralej | which is the issue with those tests for manila | 08:48 |
| rdogerrit | mathieu bultel 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 | 08:51 |
| jcapitao[m] | amoralej: ok it's related https://review.opendev.org/c/openstack/manila/+/887115/13/manila/api/openstack/api_version_request.py#208 | 08:54 |
| amoralej | yep | 08:55 |
| jcapitao[m] | from the failing patch I found https://review.opendev.org/c/openstack/manila/+/887115 | 08:55 |
| rdogerrit | mathieu bultel 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 | 08:55 |
| amoralej | so, i guess, until they include by-microversion skipping in the tempest plugin, the best option is to revert, let's see what manila or tempest guys say | 08:56 |
| jcapitao[m] | agree | 08:56 |
| jcapitao[m] | I checked the manila LP yesterday and didn't find out the bug reported by Takashi .. | 08:57 |
| jcapitao[m] | nvm, good to practise troubleshooting skills | 08:59 |
| amoralej | yes, this wasn't an easy one :) | 09:00 |
| amoralej | matbu, ^ i left some comments in the review | 09:06 |
| rdogerrit | Alfredo Moralejo proposed config master: Add Zuul definitions for os-diff https://review.rdoproject.org/r/c/config/+/51664 | 09:23 |
| rdogerrit | Merged config master: Add Zuul definitions for os-diff https://review.rdoproject.org/r/c/config/+/51664 | 09:29 |
| tosky_ | so, about that manila promotion issue | 09:32 |
| tosky_ | didn't we say it's a problem with the promotion jobs? I propose to disable the job which fails | 09:32 |
| amoralej | the job or the test ? | 09:32 |
| tosky_ | amoralej, jcapitao[m] , tkajinam ^^ | 09:32 |
| amoralej | well, imo it's not job issue, it's plugin compatibility issue, tbh, but if we want to add some skip rule, i'm good with it | 09:33 |
| tosky_ | it's the fact that the plugin is default to max microversion and expecting users to set it | 09:33 |
| amoralej | so, it can be configured? sorry, i missed that part | 09:34 |
| tosky_ | which may be wrong, yes, but that means that the job should be adapted as it is asking manila for something which is not supported | 09:34 |
| tkajinam | I've prepared the required changes to puppet-tempest to set min/max micro version and ideally we can merge it and backport it to stable branches to set the appropriate versions to workaround the current problem with the plugin | 09:34 |
| tkajinam | https://review.opendev.org/c/openstack/puppet-tempest/+/907399 | 09:34 |
| tosky_ | https://meetings.opendev.org/irclogs/%23openstack-manila/%23openstack-manila.2024-02-01.log.html#t2024-02-01T15:10:24 | 09:35 |
| tkajinam | however especially the backport part is broken because of the nasty libvirt bug which is not yet fixed after a week. I really don't want to update every stable branch and then revert the change so am waiting until the fix is released | 09:35 |
| tosky_ | ok, so maybe we can just forcibly skip that test? | 09:35 |
| tosky_ | but I'm against reverting an otherwise working plugin to an previous commit just for a configuration issue | 09:36 |
| amoralej | ok, please comment in the review | 09:36 |
| amoralej | apparently it's failing in setup so all jobs are failing, i'd say | 09:37 |
| amoralej | iiuc https://logserver.rdoproject.org/openstack-periodic-daily2/review.rdoproject.org/rdoinfo/master/weirdo-antelope-promote-puppet-scenario005-centos-stream-9/5f46097/logs/weirdo-project/logs/testr_results.html | 09:38 |
| amoralej | tkajinam, wrt the libvirt issue, your workaround in p-o-i is not fixing all the puppet jobs ? | 09:38 |
| jcapitao[m] | I'm ok with skipping the jobs as workaround as well | 09:39 |
| tkajinam | I've merged the change to exclude the bad libvirt in master but haven't yet backported it to stable branches | 09:39 |
| tkajinam | am hoping to get a fix instead of patching 4 stable branches and then revert these all | 09:39 |
| amoralej | ah, ok, that's the issue | 09:39 |
| jcapitao[m] | makes sense | 09:40 |
| amoralej | i read somewhere that the fix will be in the next compose | 09:40 |
| amoralej | lemme check | 09:40 |
| jcapitao[m] | new libvirt should land in mirror this week | 09:40 |
| jcapitao[m] | hopefully | 09:40 |
| tkajinam | yea that's why I understood from the update in jira | 09:40 |
| tkajinam | so I'd leave it now unless you need urgent requirement to avoid forcing promotion | 09:41 |
| tkajinam | you *have* | 09:41 |
| jcapitao[m] | I think we can wait | 09:41 |
| amoralej | antelope promotion is 14 days old, but if there is path to get the proper fix, i'm fine | 09:42 |
| amoralej | 10.0.0-2 is in now https://composes.stream.centos.org/production/latest-CentOS-Stream/compose/AppStream/x86_64/os/ we will see | 09:43 |
| rdogerrit | Marios Andreou proposed rdo-jobs master: Adding new data-plane-adoption-osp17-multinode-to-extracted-crc https://review.rdoproject.org/r/c/rdo-jobs/+/51227 | 09:47 |
| *** tosky_ is now known as tosky | 09:48 | |
| rdogerrit | Marihan Girgis proposed config master: Add infrawatch/functional-test to zuul https://review.rdoproject.org/r/c/config/+/51665 | 09:53 |
| rdogerrit | Marihan Girgis proposed config master: Add infrawatch/functional-test to zuul https://review.rdoproject.org/r/c/config/+/51666 | 10:04 |
| rdogerrit | Joel Capitao proposed puppet/puppet-concat-distgit antelope-rdo: puppet-concat-7.3.3-1 https://review.rdoproject.org/r/c/puppet/puppet-concat-distgit/+/50769 | 11:31 |
| rdogerrit | Joel Capitao proposed puppet/puppet-concat-distgit antelope-rdo: puppet-concat-7.3.3-1 https://review.rdoproject.org/r/c/puppet/puppet-concat-distgit/+/50769 | 11:31 |
| rdogerrit | Merged puppet/puppet-concat-distgit antelope-rdo: puppet-concat-7.3.3-1 https://review.rdoproject.org/r/c/puppet/puppet-concat-distgit/+/50769 | 11:45 |
| rdogerrit | Merged rdo-infra/ansible-role-dlrn master: Switch from RSA review.r.o key fingerprint to ECDSA https://review.rdoproject.org/r/c/rdo-infra/ansible-role-dlrn/+/48076 | 12:57 |
| rdogerrit | Merged openstack/manila-distgit yoga-rdo: openstack-manila-14.2.0-1 https://review.rdoproject.org/r/c/openstack/manila-distgit/+/51643 | 13:13 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: Pin the packages falling back to master in Bobcat https://review.rdoproject.org/r/c/rdoinfo/+/51667 | 14:21 |
| amoralej | jcapitao[m], ^ wrt pinning branchless packages in bobcat | 14:34 |
| jcapitao[m] | amoralej: ack, I'm reviewing with my script | 14:43 |
| amoralej | ack | 14:43 |
| jcapitao[m] | also, could you review https://softwarefactory-project.io/r/c/DLRN/+/30442 when you have a chance please | 14:44 |
| jcapitao[m] | I had to fix a dep issue as well https://softwarefactory-project.io/r/c/DLRN/+/30825 | 14:45 |
| rdogerrit | Marios Andreou proposed rdo-jobs master: Adding new data-plane-adoption-osp17-multinode-to-extracted-crc https://review.rdoproject.org/r/c/rdo-jobs/+/51227 | 14:58 |
| rdogerrit | Joel Capitao proposed rdo-jobs master: Use opendev git forge instead of GH https://review.rdoproject.org/r/c/rdo-jobs/+/51668 | 14:58 |
| jcapitao[m] | amoralej: ^ to be added as depends-on in your patch | 14:59 |
| rdogerrit | Joel Capitao proposed rdoinfo master: Pin the packages falling back to master in Bobcat https://review.rdoproject.org/r/c/rdoinfo/+/51667 | 15:01 |
| jcapitao[m] | I added it | 15:01 |
| spotz[m] | How'd the clean up go this morning/afternoon? | 15:05 |
| amoralej | spotz[m], we rescheduled for tomorrow beacuse of availability issues | 15:08 |
| spotz[m] | amoralej: same time? | 15:09 |
| amoralej | i'll add you to the invite | 15:09 |
| amoralej | although it will be in the middle of the night for you :) | 15:09 |
| spotz[m] | Yeah I'll be asleep:) | 15:10 |
| spotz[m] | I maybed in case a dog wakes me up but don't count on it | 15:12 |
| amoralej | jcapitao[m], i will remove openstack-macros | 15:17 |
| jcapitao[m] | okk | 15:17 |
| jcapitao[m] | looks like we don't have distgit | 15:17 |
| amoralej | yes | 15:17 |
| jcapitao[m] | wondering how we manage when bootstrapping | 15:18 |
| amoralej | it's special case | 15:18 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: Pin the packages falling back to master in Bobcat https://review.rdoproject.org/r/c/rdoinfo/+/51667 | 15:18 |
| rdogerrit | Joel Capitao proposed rdo-infra/releng master: DNM Script to list projects commit built not in release branch https://review.rdoproject.org/r/c/rdo-infra/releng/+/51632 | 15:50 |
| jcapitao[m] | amoralej: I left a comment in https://issues.redhat.com/browse/RDO-160 | 15:55 |
| amoralej | so, what are the ones you miss? | 16:00 |
| amoralej | ah, i see why i missed some | 16:01 |
| jcapitao[m] | the ones missing are tcib | 16:02 |
| jcapitao[m] | edpm-image-builder | 16:02 |
| amoralej | yes, those two | 16:02 |
| amoralej | we need to specify main | 16:02 |
| amoralej | we define source-branch at root level | 16:02 |
| jcapitao[m] | ah you grep on master ? | 16:02 |
| amoralej | no, i grep on "Falling back" | 16:03 |
| amoralej | those two have source-branch: main defined in base, so they directly try that repo and do not fallback | 16:03 |
| jcapitao[m] | okk | 16:03 |
| amoralej | for puppet-archive, i see it's pinned | 16:03 |
| jcapitao[m] | yeah my script should not show it | 16:04 |
| jcapitao[m] | it's a false positive | 16:04 |
| jcapitao[m] | and python-os-client-config as well | 16:04 |
| rdogerrit | Alfredo Moralejo proposed rdoinfo master: Pin the packages falling back to master in Bobcat https://review.rdoproject.org/r/c/rdoinfo/+/51667 | 16:07 |
| amoralej | jcapitao[m], ^ updated tcib and edpm-image-builder | 16:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!