| *** ykarel_ is now known as ykarel | 06:46 | |
| opendevreview | Takashi Kajinami proposed openstack/releases master: Retire puppet-qdr: Mark the deliverables as retired https://review.opendev.org/c/openstack/releases/+/907955 | 07:13 |
|---|---|---|
| tkajinam | idk if anyone would try to fix all these broken jobs after unmaintained transition https://review.opendev.org/q/topic:%22create-yoga%22 | 08:05 |
| frickler | tkajinam: well that's up to the unmaintained-core people. and if things stay broken, that's IMO a good indicator that stuff should get EOLed instead | 08:17 |
| tkajinam | yeah | 08:17 |
| elodilles | indeed that's gonna be a challenge :D | 08:18 |
| tkajinam | it may give us a good filter for inactive projects indeed. | 08:22 |
| tkajinam | elodilles, I'm asking this here because you are not in #heat channel, but do you still need stable/victoria branch of heat repos ? | 08:25 |
| frickler | elodilles: see https://review.opendev.org/c/openstack/project-config/+/907963 regarding the three errors from yesterday | 08:26 |
| elodilles | tkajinam: yes, according to TC resolution, we are planning to EOL train and ussuri, and transition victoria, wallaby and xena to Unmaintained | 08:31 |
| tkajinam | ok | 08:32 |
| tkajinam | I'm aware of that rule. my question is more about if we need to "maintain" it still | 08:32 |
| tkajinam | probably it's no longer "we" but "you", but anyway... | 08:33 |
| tkajinam | just curious to know the requirement to think of what we can do for it | 08:33 |
| tkajinam | s/we/I/g , probably | 08:34 |
| elodilles | tkajinam: the responsibility will transfer to "unmaintained-core" | 08:34 |
| tkajinam | yeah | 08:34 |
| elodilles | no need to maintain it | 08:34 |
| elodilles | frickler: oh, thanks for the fix! +1'd it | 08:36 |
| frickler | elodilles: about deleting branches, in theory I should be able to run the script now, too, right? offering to share the load a bit | 08:42 |
| opendevreview | Riccardo Pittau proposed openstack/releases master: Allow eol of bugfix branches https://review.opendev.org/c/openstack/releases/+/900810 | 09:24 |
| frickler | I think we should skip waiting for PTL approval for the projects marked as inactive, like senlin etc. | 09:45 |
| frickler | did a test run of the branch deletion script, couple of open reviews, but only cloudkitty and kayobe with extra merged patches | 09:48 |
| elodilles | sorry, i had to be afk | 09:58 |
| elodilles | frickler: yepp, you can run it. which one did you run? the EOL one or the EOM one? o:) | 09:58 |
| opendevreview | Dr. Jens Harbott proposed openstack/releases master: [ironic] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906560 | 09:58 |
| elodilles | i can quickly run the abandon script as well | 09:59 |
| elodilles | forgot it yesterday... | 09:59 |
| frickler | elodilles: eom from your wip review | 09:59 |
| frickler | elodilles: o.k. so once you abandoned the reviews I will have a go at deleting the branches | 10:00 |
| elodilles | (it's more like DNM, as i simply reused the EOL one o:) we could probably create some automation there... i always wanted but never dared :)) | 10:01 |
| frickler | elodilles: well the first idea I had was to add a --dry-run option to be able to get the complete output without interaction. maybe once that looks fine, having a --yes-do-it option could be fine, too | 10:03 |
| elodilles | ++ | 10:04 |
| frickler | probably easier to convince oneself to add some automation in the eom case, where currently at least one can be sure that only a single series will be affected | 10:05 |
| opendevreview | Takashi Kajinami proposed openstack/releases master: Create new puppet-ceph release https://review.opendev.org/c/openstack/releases/+/907978 | 10:14 |
| opendevreview | Takashi Kajinami proposed openstack/releases master: Create new puppet-ceph release https://review.opendev.org/c/openstack/releases/+/907978 | 10:15 |
| elodilles | I've run the abandon script but it seems i need to rerun it... had some 'connection reset by peer' error during the run :S ( https://paste.opendev.org/show/bUb97yiFiBeyTpuX2n7L/ ) | 10:16 |
| elodilles | hmmm, it keeps resetting the connection after some time... | 10:21 |
| elodilles | anyway, after some attmepts (and reducing the list with the ready repos) now every open yoga patch is abandoned for the repos that are in Unmaintained already. | 10:27 |
| elodilles | 15 patches : https://review.opendev.org/q/branch:stable/yoga+is:abandoned | 10:27 |
| elodilles | frickler: you can run the EOM branch cleanup script if you want | 10:28 |
| elodilles | unfortunately, it will take time if the 'git clone' will be slow, as it was last time when i ran it :/ | 10:30 |
| elodilles | i need to be afk again soon, but after that i can run the script if you got tired :) | 10:31 |
| elodilles | btw, here we also have some patch to review: https://review.opendev.org/q/topic:train-eol+is:open+label:PTL-Approved%252B1 && https://review.opendev.org/q/topic:ussuri-eol+is:open+label:PTL-Approved%252B1 | 10:34 |
| elodilles | (and when i get back i'll PTL-Approved+1 the inactive projects' patches, too) | 10:34 |
| frickler | elodilles: ok, will start this now, thx for the update. and yes, I saw the train patches, but wanted to proceed with yoga as far as possible first | 10:35 |
| elodilles | frickler: yepp, that is now the top prio, the train and ussuri ones are not urgent | 10:35 |
| opendevreview | Takashi Kajinami proposed openstack/releases master: Create new puppet-ceph release https://review.opendev.org/c/openstack/releases/+/907978 | 10:35 |
| frickler | oh, that branch deletion uses the gerrit http api, not ssh. so I guess I'll finally need to set up some login for that | 10:42 |
| opendevreview | Takashi Kajinami proposed openstack/releases master: Create new puppet-ceph release https://review.opendev.org/c/openstack/releases/+/907978 | 10:55 |
| frickler | phew, done | 11:04 |
| frickler | elodilles: tonyb: would it make sense to have an #openstack-unmaintained channel to point people to? already getting the first reports in #-kolla | 12:01 |
| opendevreview | Merged openstack/releases master: [OpenStackAnsible] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/904819 | 12:20 |
| opendevreview | Merged openstack/releases master: [swift] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903217 | 12:25 |
| opendevreview | Merged openstack/releases master: [barbican] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903219 | 12:28 |
| opendevreview | Merged openstack/releases master: [octavia] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903188 | 12:29 |
| opendevreview | Merged openstack/releases master: [solum] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903196 | 12:29 |
| opendevreview | Merged openstack/releases master: [freezer] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903208 | 12:29 |
| opendevreview | Merged openstack/releases master: [ironic] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903199 | 12:29 |
| opendevreview | Merged openstack/releases master: [tripleo] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903195 | 12:30 |
| opendevreview | Merged openstack/releases master: [horizon] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903222 | 12:30 |
| opendevreview | Merged openstack/releases master: [sahara] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903215 | 12:30 |
| opendevreview | Merged openstack/releases master: [OpenStackAnsible] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/904820 | 12:37 |
| opendevreview | Merged openstack/releases master: [trove] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903292 | 12:37 |
| opendevreview | Merged openstack/releases master: [swift] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903308 | 12:47 |
| opendevreview | Merged openstack/releases master: [nova] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903278 | 12:47 |
| opendevreview | Merged openstack/releases master: [octavia] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903280 | 12:47 |
| opendevreview | Merged openstack/releases master: [cloudkitty] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903283 | 12:47 |
| opendevreview | Merged openstack/releases master: [monasca] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903316 | 12:47 |
| elodilles | frickler: yes, that sounds good (at 1st i thought #openstack-stable is enough, but yes, maybe it's better to have #openstack-unmaintained so that we emphasize that it's NOT stable :)) | 12:51 |
| opendevreview | Merged openstack/releases master: [congress] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903319 | 12:52 |
| opendevreview | Merged openstack/releases master: [senlin] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903320 | 12:52 |
| opendevreview | Merged openstack/releases master: [qinling] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903317 | 12:52 |
| opendevreview | Merged openstack/releases master: [winstackers] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903296 | 12:52 |
| opendevreview | Merged openstack/releases master: [karbor] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903318 | 12:53 |
| opendevreview | Merged openstack/releases master: [searchlight] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903289 | 12:53 |
| opendevreview | Merged openstack/releases master: [tricircle] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903288 | 12:53 |
| opendevreview | Merged openstack/releases master: [Packaging-rpm] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903279 | 12:53 |
| opendevreview | Merged openstack/releases master: [sahara] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903305 | 13:20 |
| opendevreview | Merged openstack/releases master: [senlin] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906590 | 13:31 |
| opendevreview | Merged openstack/releases master: [monasca] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906589 | 13:41 |
| opendevreview | Merged openstack/releases master: [solum] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906559 | 13:41 |
| opendevreview | Merged openstack/releases master: [winstackers] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906566 | 13:49 |
| opendevreview | Gregory Thiemonge proposed openstack/releases master: octavia release 10.1.1 for yoga https://review.opendev.org/c/openstack/releases/+/908027 | 14:01 |
| opendevreview | Gregory Thiemonge proposed openstack/releases master: [octavia] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906550 | 14:02 |
| gthiemonge | frickler: I think we're good to go ^, thanks for the heads up | 14:03 |
| tkajinam | gthiemonge, this is not a strict blocker but note that you may need https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/907959 to keep octava-tempest-plugin CI surviving | 14:07 |
| tkajinam | it has yoga job which is killed by transitioning stable/yoga to umnaintained/yoga | 14:07 |
| elodilles | frickler: when you get there this needs a 2nd core review o:) -> https://review.opendev.org/c/openstack/releases/+/906549 | 14:13 |
| frickler | gthiemonge: thx, will review once the CI has finished | 14:13 |
| frickler | elodilles: ack | 14:14 |
| gthiemonge | tkajinam: thanks, we will prioritize it | 14:20 |
| opendevreview | Merged openstack/releases master: [oslo] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906551 | 14:24 |
| opendevreview | Merged openstack/releases master: [sahara] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906578 | 14:24 |
| opendevreview | Merged openstack/releases master: [freezer] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906570 | 14:24 |
| opendevreview | Merged openstack/releases master: [nova] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906549 | 14:24 |
| opendevreview | Merged openstack/releases master: [solum] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903287 | 14:49 |
| opendevreview | Merged openstack/releases master: [freezer] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903298 | 14:51 |
| opendevreview | Merged openstack/releases master: [ironic] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906560 | 14:52 |
| elodilles | (more than 60 patches abandoned today, so far, on stable/yoga) | 15:16 |
| opendevreview | Merged openstack/releases master: [keystone] Transition Train to End of Life https://review.opendev.org/c/openstack/releases/+/903211 | 15:34 |
| opendevreview | Merged openstack/releases master: [keystone] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903301 | 15:42 |
| opendevreview | Merged openstack/releases master: [keystone] Transition Yoga to Unmaintained https://review.opendev.org/c/openstack/releases/+/906574 | 15:42 |
| opendevreview | Merged openstack/releases master: [OpenStackSDK] Transition Ussuri to End of Life https://review.opendev.org/c/openstack/releases/+/903303 | 15:42 |
| frickler | elodilles: I think that that should be about it for today, so I'd run the branch deletion script once again? both eom and eol I guess | 15:59 |
| elodilles | frickler: ACK, thanks \o/ | 16:00 |
| elodilles | i have to commute home soon, so i could only run the cleanup scripts later, or tomorrow :S | 16:01 |
| elodilles | (office day :S) | 16:01 |
| frickler | np, safe travels | 16:04 |
| elodilles | thx | 16:05 |
| tkajinam | oops. sorry I have accidentally created a wrong tag (show). Can someone from the release team delete this ? | 17:12 |
| tkajinam | https://review.opendev.org/admin/repos/openstack/puppet-ceph,tags | 17:12 |
| tkajinam | sorry for my dumb mistake... | 17:17 |
| frickler | wow, finally done with the first iteration. seems I severly underestimated what "just a few patches" can end up in terms of repos | 17:59 |
| frickler | also a lot still not deleted due to open reviews, will need to do a second run after abandoning those. but I guess I'll move that to tomorrow and let zuul catch up with the tag queue for now | 18:00 |
| frickler | guess it hasn't been helping to go through reconfigurations all the time | 18:01 |
| frickler | tkajinam: I can take a look tomorrow, getting late here, but also not sure about the policy for that | 18:01 |
| tkajinam | frickler, thanks ! | 18:02 |
| clarkb | frickler: tkajinam typically we don't delete tags beacuse tags are not updated when moved | 18:04 |
| clarkb | that means if you tag foo-eol and then delete it and retag foo-eol you may have downstream git repos that never update to the new foo-eol version | 18:04 |
| clarkb | in this case I think the issue is you don't want the tag at all which is probably safer to delete. Just keep in mind downstreams may not pick up the deletion | 18:04 |
| tkajinam | clarkb, yes... | 18:07 |
| frickler | oh, the tag is really called "show" as a duplicate of "jewel-eol", presumable from running "git tag show jewel-eol" or similar? yeah, I guess that should be fine to delete in either case | 18:07 |
| tkajinam | yeah that's the exact mistake I did | 18:07 |
| tkajinam | I meant to run git show jewel-eol apparently | 18:07 |
| tkajinam | and didn't notice that then pushed all tags according to an old guide X-( https://wiki.openstack.org/wiki/GerritJenkinsGith/ja#Tagging_a_Release | 18:08 |
| tkajinam | a fortunate thing at this moment is that these tags are not yet synced to github it seems | 18:08 |
| frickler | actually it seems my release powers made a "delete" button show up on that gerrit page, so I just used it | 18:10 |
| clarkb | yup that is the way it works. The permission for delete against /refs/* gives you the button | 18:10 |
| clarkb | you need force push perms to do deletes via git push (which I don't think the release team has) | 18:10 |
| clarkb | alternative is to use the api to hit the button effectively | 18:10 |
| tkajinam | hmm seems the tag is still present in https://opendev.org/openstack/puppet-ceph | 18:11 |
| tkajinam | likely need some manual cleanup ? | 18:12 |
| clarkb | no this is the issue I told you about | 18:13 |
| clarkb | tag deletions don't propagate | 18:13 |
| tkajinam | ok | 18:13 |
| clarkb | I don't know what that cleanup looks like on the gitea side. Maybe we can login as admin there and click a button or something | 18:13 |
| tkajinam | seems there is an interface though I don't have access to it apparently https://github.com/go-gitea/gitea/pull/12096 | 18:18 |
| tkajinam | I've blocked the open puppet-ceph changes for now to avoid triggering sync from opendev to github, but would be nice if I can get some help tomorrow. sorry for all the mess I caused ... | 18:19 |
| clarkb | tkajinam: I don't think you need to block anything at this point if the tag is removed from gerrit | 18:19 |
| tkajinam | clarkb, ack. ok | 18:19 |
| clarkb | unless we use the zuul repo state which is also "corrupted" | 18:20 |
| clarkb | fungi: ^ may know | 18:20 |
| fungi | would need to look at the role in zuul-jobs | 18:22 |
| fungi | i think it does a git mirror from the local copy but don't know for certain | 18:23 |
| clarkb | and possibly in the zuul executor/merger state (whcih we can safely delete and let it repopulate iirc) | 18:23 |
| clarkb | oh and maybe also the git caches for dib | 18:23 |
| clarkb | this is why we really don't like deleting tags :) | 18:23 |
| fungi | well, it's why we generally don't delete tags | 18:24 |
| fungi | and tell everyone they're effectively undeleteable | 18:24 |
| clarkb | ya this might end up being a little whacka mole but as long as we start at hte source and move downstream it should progressively get better over time | 18:49 |
| clarkb | definitely not something we want ot do regularly | 18:49 |
| frickler | elodilles: if you're still around, I think the eom deletion script is well tested now and I'd say let's merge it as is and rework later | 18:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!