15:00:11 <jednorozec> #startmeeting RELENG (2021-09-14) 15:00:11 <zodbot> Meeting started Tue Sep 14 15:00:11 2021 UTC. 15:00:11 <zodbot> This meeting is logged and archived in a public location. 15:00:11 <zodbot> The chair is jednorozec. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:00:11 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic. 15:00:11 <zodbot> The meeting name has been set to 'releng_(2021-09-14)' 15:00:11 <jednorozec> #meetingname releng 15:00:11 <zodbot> The meeting name has been set to 'releng' 15:00:19 <jednorozec> #chair nirik sharkcz pbrobinson pingou mboddu dustymabe ksinny jednorozec 15:00:19 <zodbot> Current chairs: dustymabe jednorozec ksinny mboddu nirik pbrobinson pingou sharkcz 15:00:19 <jednorozec> #topic init process 15:00:51 <nirik> morning 15:00:56 <pmoura> morning 15:03:32 <mboddu> Morning 15:04:23 <sharkcz> hi 15:05:26 <jednorozec> so 15:05:40 <jednorozec> hello all 15:06:05 <jednorozec> I have one ticket that is interesting. 15:06:14 <jednorozec> .releng 10298 15:06:15 <zodbot> jednorozec: Issue #10298: Suspicious koji user - releng - Pagure.io - https://pagure.io/releng/issue/10298 15:06:36 <jednorozec> So my question is how do we drop users from koji? 15:06:45 <jednorozec> Just directly delete the user from DB? 15:07:26 <nirik> you can only disable normally, but yeah, db... but that could have complications 15:07:30 <mboddu> disable-user 15:07:35 <jednorozec> ok 15:07:47 <nirik> yeah, we could delete in this case I guess. 15:08:07 <jednorozec> But there will be some relations in the DB I assume. 15:08:21 <nirik> possibly 15:09:03 <jednorozec> ok I will disable the user 15:09:06 <mboddu> I suggest disabling the user for now and file a ticket with koji to add `remove-user` functionality 15:09:22 <jednorozec> that is good idea 15:09:41 <nirik> well, normally you wouldn't want to ever remove a user... 15:09:42 <mboddu> Or add `--remove` to disable-user option 15:09:45 <nirik> if they have builds, etc... 15:10:22 <jednorozec> yeah that would leave garbage behind 15:10:30 <mboddu> Then add the check before calling remove user 15:10:56 <mboddu> But anyway, disable-user is good for now 15:11:33 <jednorozec> .releng 10177 15:11:34 <zodbot> jednorozec: Issue #10177: epel-release-latest-8.noarch.rpm 403 forbidden - releng - Pagure.io - https://pagure.io/releng/issue/10177 15:11:55 <nirik> this is the script not updating that somehow again. ;( 15:12:03 <nirik> I thought we had fixed it. 15:12:42 <mboddu> Noooooooooooooooooooooooooooooooooooooooooooooooooo 15:12:48 <mboddu> Not this again 15:13:50 <nirik> I knew mboddu would be happy. ;) 15:14:31 <mboddu> haha, I wonder why its not working 15:14:51 <jednorozec> DO we know why it happened the first time? 15:15:23 <nirik> FileExistsError: [Errno 17] File exists: '8/Everything/x86_64/Packages/e/epel-release-8-13.el8.noarch.rpm' -> '/pub/epel/epel-release-latest-8.noarch.rpm' 15:16:00 <nirik> https://paste.centos.org/view/37ca2121 15:17:03 <nirik> not removing the old one first perhaps? 15:17:14 <jednorozec> yup 15:18:31 <mboddu> But https://pagure.io/fedora-infra/ansible/blob/main/f/roles/bodhi2/backend/files/new-updates-sync#_465 15:19:12 <jednorozec> but that never gets there 15:19:12 <nirik> yeah, weird. 15:19:29 <nirik> oh right, dest exists. 15:19:33 <jednorozec> If path exists is followed in not exists 15:19:44 <jednorozec> so it never gets there 15:21:13 <mboddu> jednorozec: That checks if path exists and if exists checks if the symlink exists 15:21:25 <mboddu> If the symlink is not available, then remove it 15:21:38 <jednorozec> mboddu, well it is the exact same condition 15:21:43 <jednorozec> but negated 15:22:02 <jednorozec> if it is not there what do you remove? 15:22:33 <jednorozec> and if the (dest) exists then if not os.path.exists(os.path.join(EPELDEST,os.readlink(dest))): is always false 15:23:52 <jednorozec> the link is there and the not negates it so its false therefor it never removes 15:23:58 <mboddu> The second one checks if the file exists whatever the symlink pointing to 15:24:19 <mboddu> But I guess removing the `if os.path.exists(dest):` should solve the issue 15:25:24 <nirik> well, we only want to change it if the epel-release changed. 15:25:29 <jednorozec> well rewriting the if to use positive logic would make it much more readable 15:26:02 <mboddu> jednorozec: Create a PR and we can review it 15:26:06 <nirik> someone want to make a pr and... 15:26:09 <nirik> yeah... ;) 15:26:47 <nirik> perhaps pmoura would like to try his hand at it... but I guess there's a lot of context here we all know. 15:27:20 <pmoura> yeah cool, Icould take a look 15:27:27 <mboddu> Thanks pmoura 15:27:37 <jednorozec> pmoura, if you need any help 15:27:40 <jednorozec> just ping 15:27:51 <pmoura> thanks :) 15:27:55 <nirik> pmoura: basically it's supposed to make/update the link only when there's a new thing to link to. 15:28:26 <mboddu> jednorozec, pmoura : It should update when the epel-release package is changed as well as a new epel-release is added (epel-release-9...) 15:29:03 <jednorozec> oh the new release if the complexity that I did not undrestood at first. 15:29:24 <jednorozec> *is 15:29:55 <jednorozec> .releng 10299 15:29:56 <zodbot> jednorozec: Issue #10299: dist-git hook rejecting a push because of EOL should report a reason - releng - Pagure.io - https://pagure.io/releng/issue/10299 15:30:16 <jednorozec> So adding meaningfull messages to git hooks is a good idea 15:30:23 <nirik> +1 15:30:30 <mboddu> Yeah, +1 15:30:30 <jednorozec> They live in the ansible repo? 15:31:05 <nirik> they should yes. 15:31:34 <jednorozec> This is a simple enough task that I would like one of our interns to take 15:32:06 <jednorozec> I have a meeting with them tmrw 15:32:53 <nirik> sounds good. As a side note... the https commit hook for src.fp.o is still really really really noisy 15:34:13 <jednorozec> .releng 10264 15:34:14 <zodbot> jednorozec: Issue #10264: New Repo for "rpms/pypy3.7" incompletely created - releng - Pagure.io - https://pagure.io/releng/issue/10264 15:34:17 <jednorozec> I have an update here 15:34:54 <jednorozec> I was able to use orphan branch to create new branch with 0 commits in it 15:35:12 <jednorozec> but when creating alias on It I get another exception from pagure 15:35:21 <jednorozec> so still WIP 15:35:31 <mboddu> Oh nice 15:35:40 <mboddu> Does it need any changes in pagure? 15:35:54 <jednorozec> I hope it does not. 15:36:03 <mboddu> Well, good luck :) 15:36:25 <jednorozec> I will give it chance until the freeze ends. 15:36:38 <jednorozec> If I cannot fix it on our side, pagure it is... 15:36:49 <mboddu> Okay, let me know if you need any help 15:40:06 <jednorozec> That is all from my side. 15:40:19 <jednorozec> #topic Open Floor 15:40:28 <pmoura> I have an update on 10270 15:40:39 <pmoura> https://pagure.io/releng/issue/10270 15:41:00 <jednorozec> pmoura, does zodbot recognize you? If not create account with it 15:41:08 <jednorozec> then you can use magic like 15:41:16 <jednorozec> .releng 10270 15:41:17 <zodbot> jednorozec: Issue #10270: Refresh Grokmirror manifest - releng - Pagure.io - https://pagure.io/releng/issue/10270 15:41:48 <pmoura> haha cool, how can I make it recongnize me? 15:42:38 <pmoura> should I create another account besides the one on fedora? 15:42:59 <jednorozec> just open query wit hit and write help 15:43:08 <jednorozec> there is something like register or so 15:43:20 <pmoura> ah ok, thanks 15:44:13 <jednorozec> anyhow I interupted your update 15:44:14 <pmoura> about the issue.. I compared the checkout of all repos with the manifest and I found which repos are missing 15:44:56 <pmoura> I think.. but I'm not finding them anywhere... still digging... 15:46:44 <nirik> pmoura: I still owe you some information about packaging and updating grokmirror package... just haven't had time to get to it yet. 15:47:17 <pmoura> no worries 15:48:22 <jednorozec> pmoura, so the requester is asking in the issue. 15:48:46 <jednorozec> I think it might be a good idea to give them update, tht somebody is working on it. 15:48:59 <pmoura> ok, I'll update it 15:50:04 <jednorozec> So any other updates or blockers? 15:50:38 <mboddu> Lot of blockers 15:50:42 <mboddu> For f35 beta :D 15:50:47 <jednorozec> yeah 15:50:57 <jednorozec> But they are being worked on 15:51:59 <jednorozec> its all jokes to you now when you work mostly on stream mboddu :P 15:52:26 <mboddu> jednorozec: Fedora is still my first love :D 15:52:36 <jednorozec> <3 15:52:59 <jednorozec> So if there are no topics i will close the meeting and give you back few minutes of the day 15:53:20 <mboddu> Nothing from me 15:54:37 <jednorozec> #endmeeting