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

consolidate package triggers into scheduler on the default branch #282

Merged
merged 3 commits into from
Sep 12, 2024

Conversation

aptalca
Copy link
Member

@aptalca aptalca commented Sep 4, 2024

Currently the package scheduler discovers the live branches and triggers the individual workflows in each branch. This is not efficient because a separate workflow runs for each branch, even though the package triggers are all identical (unlike the external triggers).
Also it is not accurate because the scheduler posts on discord all the live branches it discovered as being triggered, but the actual package trigger may skip the trigger due to the github organization secret for pausing package triggers.

This PR consolidates all package_trigger.yml function into the single package_trigger_scheduler.yml running on cron off of the default github branch.

It discovers live branches and then determines whether the trigger should run for each branch based on several factors. The discord message has been updated to list triggered and skipped branches as necessary.

This PR requires switching from individual github org secrets such as PAUSE_PACKAGE_TRIGGER_BASEIMAGE_KASMVNC_ARCH to a single editable github org variable SKIP_PACKAGE_TRIGGER, which contains the repos and branches in separate lines in the format of containername_repobranch all lowercase (ie. baseimage-kasmvnc_arch)
This is due to a limitation of our workflow discovering the branch names runtime, but not being able to procure the value of a github secret with the runtime discovered branch as part of the secret name/key. Afaik github workflows need to hardcode secret names in the workflow.

Tests performed:

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-a542184ee6cf225fa6b220ea14a79e07a921e332-pr-282/index.html
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-a542184ee6cf225fa6b220ea14a79e07a921e332-pr-282/shellcheck-result.xml

Tag Passed
amd64-1c444d00-pkg-1c444d00-dev-a542184ee6cf225fa6b220ea14a79e07a921e332-pr-282
arm64v8-1c444d00-pkg-1c444d00-dev-a542184ee6cf225fa6b220ea14a79e07a921e332-pr-282

@aptalca aptalca marked this pull request as ready for review September 4, 2024 18:43
@aptalca aptalca requested a review from a team September 4, 2024 18:44
@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-c68c0be75501d74f14738f5e0436de370221b0e5-pr-282/index.html
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-c68c0be75501d74f14738f5e0436de370221b0e5-pr-282/shellcheck-result.xml

Tag Passed
amd64-1c444d00-pkg-1c444d00-dev-c68c0be75501d74f14738f5e0436de370221b0e5-pr-282
arm64v8-1c444d00-pkg-1c444d00-dev-c68c0be75501d74f14738f5e0436de370221b0e5-pr-282

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-592396873a521302407c9953e60f7eb47cd87e9d-pr-282/index.html
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-592396873a521302407c9953e60f7eb47cd87e9d-pr-282/shellcheck-result.xml

Tag Passed
amd64-1c444d00-pkg-1c444d00-dev-592396873a521302407c9953e60f7eb47cd87e9d-pr-282
arm64v8-1c444d00-pkg-1c444d00-dev-592396873a521302407c9953e60f7eb47cd87e9d-pr-282

@LinuxServer-CI
Copy link
Contributor

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-b270a7212e8e98395baad291b41763f467131e12-pr-282/index.html
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/1c444d00-pkg-1c444d00-dev-b270a7212e8e98395baad291b41763f467131e12-pr-282/shellcheck-result.xml

Tag Passed
amd64-1c444d00-pkg-1c444d00-dev-b270a7212e8e98395baad291b41763f467131e12-pr-282
arm64v8-1c444d00-pkg-1c444d00-dev-b270a7212e8e98395baad291b41763f467131e12-pr-282

@aptalca aptalca merged commit f7063e4 into monthly Sep 12, 2024
4 checks passed
@aptalca aptalca deleted the monthly-package branch September 12, 2024 12:10
@aptalca aptalca mentioned this pull request Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants