#36 c9s package updates
Merged by dcavalca. Opened by oidoming.
c9s-notifications  into  main

Download 36.patch

This solves https://pagure.io/centos-sig-hyperscale/sig/issue/121

For c9s packages we are going to listen on koji builds through AMQP (fedora messaging). The source for c9s packages is GitLab but the payload for GitLab topics (e.g. for commits) doesn’t provide useful information for this (like package version, release, etc.) and koji build topic does.

Note: Only last 4 commits are the changes for this specific PR. First merge https://pagure.io/centos-sig-hyperscale/package-updates/pull-request/29

Since we are using an in-memory DB we only need one db connection so I've made it global

Created this decorator to avoid having duplicate issues for same package. Now all the package updater functions (when creating new tickets) will have this decorator.

Now the package_updater functions are cleaner thanks to the @duplicateValidator :)

Changed the project setup when checking for package updates for first time running. Now it will call MQTT.package_updater or AMQP.package_updater depending on the package centos version

Here is the logic for c9s package updates (from koji package builds)

rebased onto 0b80328e4051b0253946c13a92c06c9f36345784

rebased onto ff420ad4e98c1e7bba8e342bfa788a3ae6616c42

Merge conflicts solved, ready to merge

This is just one commit now that's dropping one line and fixing comments -- it doesn't seem related to c9s package updates at all. Is that expected?

rebased onto 0b80328e4051b0253946c13a92c06c9f36345784

rebased onto 351e4bccb54e0bc56057f1bb229a0d3040be8afe

I solved the conflict the wrong way apparently, the previous c9s commits disappeared from the PR but the commits were still in the c9s-notifications branch (not tracked by the PR).

Now is ready, the c9s commits are back :D

This one needs another rebase

rebased onto da77667a20aa56d1cf7ebf33eb97993b1fa9a492

:thumbsup:

typo: s/rul/url/

Do we need to worry about point releases here? i.e. stuff coming out with el9.1 in the disttag

1 new commit added

  • Fix typos

Do we need to worry about point releases here? i.e. stuff coming out with el9.1 in the disttag

I don't think so, right now this is used to tag issues with 'package_name.centos_version ' to distinguish pagure open issues from same package but different centos version like rpm.c8s and rpm.c9s so we only care about the 'el9' (e.g. 'el8_6' https://pagure.io/centos-sig-hyperscale/package-updates/issue/32)

Pull-Request has been merged by dcavalca

Metadata