This is an archive of the discontinued LLVM Phabricator instance.

Add optional args to skip artifact upload and job triggers to clang jobs
ClosedPublic

Authored by justice_adams on Aug 3 2023, 11:19 AM.

Details

Summary

In this diff:

  • Update the clang jobs with both SKIP_UPLOAD and SKIP_TRIGGER variables which can be set on any jobs targeting the jenkinsfiles, and update the build script invocations accordingly
  • Update clang-stage1-RA jobs to run on both green-dragon-02 and green-dragon-08

This will allow us to setup new clang build jobs with different git branches, and the jenkinsfiles will respect the variables set in the job configuration.
See https://green.lab.llvm.org/green/job/clang-stage1-RA-release-17.x, which runs this patch

Diff Detail

Event Timeline

justice_adams created this revision.Aug 3 2023, 11:19 AM
Herald added a project: Restricted Project. · View Herald Transcript
justice_adams requested review of this revision.Aug 3 2023, 11:19 AM

We decided to run only the clang-stage1-RA job for the release branch, so the changes to the other jobs aren't really needed?

zorg/jenkins/jobs/jobs/clang-stage1-RA
1–2

This could now run on either of these nodes, but these nodes don't have the same configuration. green-dragon-08 is newer hardware/OS/Xcode whereas green-dragon-02 is older, which could cause issues.

zorg/jenkins/jobs/jobs/clang-stage2-Rthinlto
9 ↗(On Diff #546951)

We don't really need to skip the upload. It is possible somebody might want to download it and use it locally or debug something. It just gets uploaded to a local server.

Also, if we are starting to track multiple branches, then best would be to just move to multibranch pipelines.

azharudd requested changes to this revision.Aug 9 2023, 10:16 AM
This revision now requires changes to proceed.Aug 9 2023, 10:16 AM

Updated the diff to

  • limit changes to clang-stage1-RA
  • only skip-triggering new jobs
  • Added option to override node config in job config (defaults to green-dragon-02)
azharudd accepted this revision.Aug 14 2023, 10:52 AM

LGTM. Thanks!

This revision is now accepted and ready to land.Aug 14 2023, 10:52 AM
justice_adams added a comment.EditedAug 21 2023, 12:41 PM

@azharudd I don't believe I have commit access, could you land this for me?