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

run jenkins-builder as local only, fix deprecation, hardcode for s6v3, restructure templating #243

Merged
merged 2 commits into from
Mar 6, 2024

Conversation

aptalca
Copy link
Member

@aptalca aptalca commented Feb 22, 2024

  • Run jenkins-builder as local only (1st step to removing the remote run steps that double the number of tasks, consistent behavior between local and ci, much easier to test and troubleshoot) 6a50b4f
    • Second step of removing the remote run steps would likely have to be done after all repos are updated otherwise jenkins build will fail running on an older Jenkinsfile. In such cases, a local update of templates pushed to github should fix it
  • Fix deprecation (s6v2 vs s6v3 detection was working on a local run, but failing on remote. This PR hardcodes it to s6v3) 6a50b4f
  • Increase verbosity of templating steps 0719f3e
  • Slightly restructure templating to prevent a potential race condition by ensuring each commit to main repo results in an immediate build exit as the commit will trigger a new build and we don't want simultaneous builds running 0719f3e

Prevent potential race condition by ensuring each commit to main repo results in an immediate build exit as the commit will trigger a new build and we don't want simultaneous builds running
@aptalca aptalca requested a review from a team February 22, 2024 15:21
@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/ea146545-pkg-ea146545-dev-0719f3e272b1694991f458d07a2db715296149df-pr-243/index.html
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/ea146545-pkg-ea146545-dev-0719f3e272b1694991f458d07a2db715296149df-pr-243/shellcheck-result.xml

Tag Passed
amd64-ea146545-pkg-ea146545-dev-0719f3e272b1694991f458d07a2db715296149df-pr-243
arm64v8-ea146545-pkg-ea146545-dev-0719f3e272b1694991f458d07a2db715296149df-pr-243

@aptalca aptalca mentioned this pull request Feb 26, 2024
4 tasks
@aptalca aptalca merged commit cc47dc8 into monthly Mar 6, 2024
6 checks passed
@aptalca aptalca deleted the monthly-local branch March 6, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants