14:27:07 <cverna> #startmeeting Fedora Tagger - Future development 14:27:07 <zodbot> Meeting started Fri Apr 20 14:27:07 2018 UTC. The chair is cverna. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:27:07 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic. 14:27:07 <zodbot> The meeting name has been set to 'fedora_tagger_-_future_development' 14:27:14 <elormes[m]> alick: will you like to chair today's meeting? 14:27:22 <cverna> ok let's start it 14:27:32 <cverna> I don't mind chairing 14:27:39 <cverna> #chair elormes[m] alickm 14:27:39 <zodbot> Current chairs: alickm cverna elormes[m] 14:28:00 <cverna> Did you guys had an agenda ? 14:28:03 <elormes[m]> cverna: great 14:28:56 <alickm> * share thoughts on future plan of tagger 14:29:27 <cverna> #topic Future plan for Fedora tagger 14:29:35 <alickm> * discuss possible spitting of work 14:29:45 <cverna> #undo 14:29:45 <zodbot> Removing item from minutes: <MeetBot.items.Topic object at 0x18826550> 14:29:52 <cverna> #topic Future plans for Fedora tagger 14:29:56 <x3mboy> .hello2 14:29:57 <zodbot> x3mboy: x3mboy 'Eduard Lucena' <eduardlucena@gmail.com> 14:30:44 <cverna> OK so maybe everyone can do a quick intro 14:31:06 <cverna> stating what he/she would like to help with 14:32:31 <cverna> alickm: you want to start ? 14:33:29 <alickm> I'd like to help with long term maintenance of tagger on Fedora infra 14:33:49 <alickm> Mainly sys admin side 14:34:57 <x3mboy> ! 14:35:34 <elormes[m]> I'll work mainly on the migration 14:35:38 <x3mboy> iirc the main issue reported is about the integration with the databases, right? 14:36:04 <elormes[m]> And also help with maintenance 14:36:40 <cverna> x3mboy: yes this is https://github.com/fedora-infra/fedora-tagger/issues/200 14:36:59 <cverna> we need to remove the dependencies to pkgdb 14:37:23 <x3mboy> Ok, it makes sense 14:37:24 <cverna> so it sounds like we have a good team between elormes[m] and alickm :) 14:37:41 <cverna> do you have more specific questions ? 14:38:25 <x3mboy> Yes, migration means what specifically? 14:39:56 <x3mboy> Moving it from pkgdb to pagure? 14:40:09 <x3mboy> Rigth? 14:40:28 <cverna> x3mboy: yes replacing the call to pkgdb to query pagure instead 14:41:04 <x3mboy> So, beyond those 2 tasks, that are HUGE! what other help is needed? 14:41:40 <cverna> actually pingou pointed out what code need to be changed https://github.com/fedora-infra/fedora-tagger/issues/200#issuecomment-377497214 14:41:51 <cverna> elormes[m]: did you look at that ? 14:42:05 <cverna> if yes do have questions ? 14:42:15 <cverna> x3mboy: I think that's all 14:43:24 <cverna> pingou: \o/ 14:43:33 <pingou> o/ 14:43:42 <bt0__> o/ 14:44:06 <alick_laptop> In that links we have 3 points to work on migration part 14:44:20 <alick_laptop> Two are about pkgdb and one is about yum. 14:44:43 <cverna> for those who just joined we are looking at https://github.com/fedora-infra/fedora-tagger/issues/200#issuecomment-377497214 14:44:44 <elormes[m]> Are there any differences between packages on pkgdb and those on pagure? Would it affect the db schema? 14:48:31 <alick_laptop> Maybe someone can work on porting the yum part? 14:48:34 <cverna> after reading the pingou's comment, I don't think it need to talk to pagure, it needs to interface with PDC and mdapi for the first 2 points and the last point should just be a link to src.fedoraproject.org 14:49:22 <cverna> let's do this one topic at the time. 14:49:42 <pingou> +1, there is no need to talk to pagure 14:49:46 <cverna> #topic Remove pkgdb from https://github.com/fedora-infra/fedora-tagger/blob/develop/fedoratagger/lib/retired.py 14:49:54 <cverna> #link https://github.com/fedora-infra/fedora-tagger/blob/develop/fedoratagger/lib/retired.py 14:50:45 <cverna> So we need to replace the PKGDB_URL to the equivalent in PDC 14:51:27 <cverna> the PDC apis are available here --> https://pdc.fedoraproject.org/rest_api/v1/ 14:51:48 <pingou> and potentially in a few months to something else, since PDC is being deprecated 14:52:04 <pingou> but that's for later :) 14:52:27 <cverna> or we could use src.fp.o json file ? 14:52:55 <cverna> https://src.fedoraproject.org/extras/pagure_poc.json 14:53:24 <cverna> that gives the full list of packages, correct ? 14:54:04 <pingou> no that includes the orphaned, not the retired packages 14:54:14 <pingou> though in this case maybe it doesn't matter :s 14:55:23 <cverna> ok so let's use PDC, pingou would you know which API to use ? 14:56:04 <cverna> elormes[m]: alick_laptop : do not hesitate to ask questions, if needed :) 14:56:27 <pingou> hm 14:57:22 <cverna> I think it should be https://pdc.fedoraproject.org/rest_api/v1/global-components/ 14:57:48 <cverna> at least it is what fedora-packages is using :) 15:01:23 <pingou> for the list of all packages or the list of retired ones? 15:02:14 <cverna> the list of all packages 15:02:19 <pingou> for the retired ones I think https://pdc.fedoraproject.org/rest_api/v1/component-branches/?active=false is good 15:02:43 <pingou> https://github.com/fedora-infra/fedora-tagger/blob/develop/fedoratagger/lib/retired.py this one is about retired packages 15:02:54 <pingou> for all packages, I think the pagure_poc.json is fine 15:04:36 <cverna> yeah I think in packages there is weird code to check if it still active or not. I think https://pdc.fedoraproject.org/rest_api/v1/component-branches/?active=false is better 15:05:14 <cverna> #info investigate https://pdc.fedoraproject.org/rest_api/v1/component-branches/?active=false in order to get the list of retired packages 15:05:47 <cverna> are we good to move to the yum part ? 15:06:12 <cverna> or do you have any questions about retired.py ? 15:08:20 <cverna> #topic use mdapi instead of yum in https://github.com/fedora-infra/fedora-tagger/blob/develop/fedoratagger/lib/update.py 15:09:22 <pingou> alick_laptop: elormes[m]: any comments? 15:09:29 <pingou> are you still there? 15:10:22 <alick_laptop> I was thinking maybe x3mboy might be interested in yum -> mdapi 15:10:32 <x3mboy> Sure! 15:10:45 <pingou> so the koji part can likely remain the same 15:11:12 <pingou> the yum part, so update_summaries, can likely be simplified by simply querying mdapi 15:11:16 <pingou> https://apps.fedoraproject.org/mdapi/ 15:11:25 <cverna> #link https://apps.fedoraproject.org/mdapi/ 15:13:33 <x3mboy> Ok, I will take notes about it and work on it this weekend 15:13:47 <cverna> ok moving on then :) 15:14:01 <x3mboy> Right now I'm doing some marketing stuff, so, I don't have the chance to check it rn 15:14:15 <cverna> #topic replace pkgdb link in https://github.com/fedora-infra/fedora-tagger/blob/develop/fedoratagger/frontend/app.py 15:15:17 <alick_laptop> Is this just a two-line change? 15:15:30 <cverna> I think so 15:15:34 <pingou> likely 15:16:30 <cverna> so I think we have covered the code change needed 15:17:21 <cverna> elormes[m] or alick_laptop do you want to look at the retired.py script and the pkgdb links ? 15:17:33 <cverna> and x3mboy will do the mdapi 15:18:11 <elormes> cverna: ok, I'll take the retired.py script 15:18:33 <alick_laptop> I can do the two-line change lol 15:18:40 <cverna> alick_laptop: thanks :) 15:18:58 <cverna> after the application needs to be packaged 15:19:25 <cverna> pingou: is it currently running on a rhel box ? 15:19:46 <elormes> I've not really looked at the tests but would that also need to be changed? 15:20:13 <pingou> cverna: likely 15:20:38 <cverna> elormes: I guess, if the tests are running and passing currently they should still do after the change 15:20:59 <cverna> #topic packaging the application 15:21:21 <cverna> someone needs to try to build the rpm using --> https://github.com/fedora-infra/fedora-tagger/blob/develop/fedora-tagger.spec 15:21:25 * alick_laptop need to be on the go 15:22:01 <cverna> thanks alick_laptop I think we will finish soon 15:22:45 <cverna> then once we have an rpm we should be good to deploy the application :) 15:23:41 <cverna> any other questions ? or should we end the meeting ? 15:24:35 <cverna> #info x3mboy to port the yum code to mdapi 15:24:56 <cverna> #info elormes to look at using pdc in the retired.py script 15:25:45 <cverna> #info alick_laptop to update the pkgdb links in the flask app 15:25:55 <cverna> ok I think we are good then 15:25:57 <cverna> thanks 15:26:25 <cverna> #endmeeting