containerbuildsystem / koji-containerbuild

Container build support for Koji buildsystem

GNU Lesser General Public License v2.1

29 stars 32 forks source link

Add a priority queue allowlist check. URL pointing to allowlist is... #194

Closed ben-alkov closed 3 years ago

ben-alkov commented 3 years ago

...read from a config file located on the koji hub, '/etc/koji-hub/plugins/koji-containerbuild.conf', which looks like

[priority_allowlist]
priority_target_list_url = https://foo/bar.yaml

Allowlist looks like

targets:
- managed-services-containers
- ryan-is-special

Signed-off-by: Ben Alkov ben.alkov@redhat.com

CLOUDBLD-3573

Maintainers will complete the following section

[ ] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Pull request has a link to an osbs-docs PR for user documentation updates

packit-as-a-service[bot] commented 3 years ago

Congratulations! One of the builds has completed. :champagne:

:warning: Please note that our current plans include removal of these comments in the near future (at least 2 weeks after including this disclaimer), if you have serious concerns regarding their removal or would like to continue receiving them please reach out to us. :warning:

You can install the built RPMs by following these steps:

sudo yum install -y dnf-plugins-core on RHEL 8
sudo dnf install -y dnf-plugins-core on Fedora
dnf copr enable packit/containerbuildsystem-koji-containerbuild-194
And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

ben-alkov commented 3 years ago

@lcarva, @rcerven; WDYT so far? No ACKs, please, just feedback.

I realize that I need unit tests :)

ben-alkov commented 3 years ago

Closing as Dropped.