Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate with pkgdb #13

Closed
jasontibbitts opened this issue Jun 25, 2015 · 16 comments
Closed

Integrate with pkgdb #13

jasontibbitts opened this issue Jun 25, 2015 · 16 comments
Assignees

Comments

@jasontibbitts
Copy link

So that the number of interfaces a maintainer must use to set up their packages stays reasonable, it would be really nice to integrate koschei with a knob in pkgdb which turns on the monitoring. I'm filing this ticket to start the conversation on the possibility of making that happen.

@ralphbean
Copy link
Contributor

/cc @ralphbean, @pypingou, @nirik

@pypingou
Copy link
Member

So we would have a switch on the package page to add support for koschei?

That would be unrelated to the Monitoring (for new updates) flag, right?

@mizdebsk
Copy link
Member

@pypingou Yes, that's how I understand that feature. Instead of adding package in Koschei you could add it directly from pkgdb2 by toggling boolean, just like you enable release monitoring.

@remicollet
Copy link

+1

@pypingou
Copy link
Member

Ok I have the changed made on the pkgdb side, this is how it would look:

pkgdb_koschei

Then the list of packages will be exposed via a similar API than:
https://admin.fedoraproject.org/pkgdb/api/#list_packages_monitored

And changes in the status will be announced via fedmsg

@ralphbean
Copy link
Contributor

@pypingou now that there are two headers on the right panel ("monitoring" and "koschei integration"), there should probably be a third header titled "other stuff". Right now it looks like "request new branch" has something to do with koschei.. which it does not.

@pypingou
Copy link
Member

Yes, I was thinking the same, just not sure what the right header would be Actions?

@mizdebsk
Copy link
Member

Changes that would need to be implemented at Koschei side:

  • koschei-watcher should watch for package.koschei.update messages and update database accordingly
  • koschei-polling should periodically sync list of monitored packages with pkgdb2's /api/koschei (or /api/koschei?format=json for JSON) - fallback method needed due to unreliable nature of fedmsg
  • packages blocked in Koji must remain marked as ignored
  • pkgdb2 integration needs to remain optional, so this feature must be possible to be disabled in config
  • with this feature enabled, it shouldn't be possible to add packages through web nor CLI interface

@mizdebsk mizdebsk self-assigned this Jul 1, 2015
mizdebsk added a commit that referenced this issue Sep 1, 2015
@mizdebsk
Copy link
Member

mizdebsk commented Sep 2, 2015

It's all implemented and can be tested in staging:

Changes in package DB should be immediately seen by Koschei.

@jasontibbitts
Copy link
Author

Awesome! Thanks. I imagine that pushing this to production will miss the freeze deadline but I look forward to seeing this more widely implemented.

I would, however, recommend a tooltip or something, since many folks won't have any idea as to what koschei actually is.

@pypingou
Copy link
Member

pypingou commented Sep 3, 2015

There is one fix needed for pkgdb before we're ready, I can do it today and cut the release.

When do we want to push to prod? (I'm ok with tomorrow or Monday)

@pypingou
Copy link
Member

pypingou commented Sep 3, 2015

pkgdb2 is updated and deployed in stg

@mizdebsk
Copy link
Member

mizdebsk commented Sep 8, 2015

When do we want to push to prod?

I would like to deploy current Koschei to production ASAP, but this is blocked by infra tickets #4878, #4879 and rel-eng ticket #6172.

@nirik
Copy link
Member

nirik commented Sep 8, 2015

Well, those are all stg issues... but yeah, I hope to get time to work on some of them over the next few weeks that we are in freeze.

@nirik
Copy link
Member

nirik commented Sep 14, 2015

I've fixed the infra tickets now. Do you need 6172? I can ask Dennis again later this week, but he is traveling now...

@mizdebsk
Copy link
Member

I've fixed the infra tickets now.

Thank you.

Do you need 6172? I can ask Dennis again later this week, but he is traveling now...

Yes. There were multiple improvements in the way Koschei communicates with Koji. Testing them properly requires either cooperation with staging Koji admins, or one of Koschei developers becoming Koji admin. Without that some features will have to go to production without prior testing in staging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants