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