waiverdb

#369 C3I: use of pipeline-as-a-service, reduce number of jobs
Merged by lholecek. Opened by mkovarik.
mkovarik/waiverdb templating-final  into  master

Download 369.patch

Integration with pipeline-as-a-service which replaces factor2-deployer. Number of jobs is reduced by merging triggers and executors which provide whole process in one jobs. Triggering of external jobs is reduced to minimum, only c3i project or pipeline-as-a-service are requested during pipeline execution. The testing itself is run inside main job which triggered the execution.

@mikeb replacement for https://pagure.io/waiverdb/pull-request/362, all jobs tested. I should create a meeting for factory2 team with demo of changes and pipeline-as-a-service.

1 new commit added

  • Use new SERVICES_TO_DEPLOY parameter

rebased onto b98d57774e4ff023713aab792e17b33ad79dfe32

1 new commit added

  • Removed duplicated function

@gnaponie @lholecek Any objections to merging this? Any questions about how to manage the jobs going forward?

+1 to merge. Thanks for the patch ...but also sorry, it's impossible for me to review -- endless lines of YAML and Groovy scare me away every time I try.

Hey, I would need more time to review this too. It's a huge change and I'm not really familiar with all these files. I can try to have a deeper look tomorrow.

Hey, I would need more time to review this too. It's a huge change and I'm not really familiar with all these files. I can try to have a deeper look tomorrow.

I would rather go through the implementation after the patch is merged (the big diff is much more difficult to read).

Would it be possible to have a short meeting with @mikeb and @mkovarik and go through the files in openshift/ directory and c3i library?

Agreed with @lholecek. @mkovarik told me we can have a small demo of the change. Maybe we can go through the changes together over bluejeans or something.

Let's merge this now.

Pull-Request has been merged by lholecek
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.