Skip to content

ci: change TEST_COMMIT to TEST_BRANCH#2842

Open
johnzupin wants to merge 1 commit intoLunarG:devfrom
johnzupin:zupin-test-commit-to-branch
Open

ci: change TEST_COMMIT to TEST_BRANCH#2842
johnzupin wants to merge 1 commit intoLunarG:devfrom
johnzupin:zupin-test-commit-to-branch

Conversation

@johnzupin
Copy link
Copy Markdown
Contributor

@johnzupin johnzupin commented Mar 31, 2026

Most CI scripts were using TEST_COMMIT but jenkinsfile.extended and jenkinsfile.manual define it as TEST_BRANCH. Values defined for a branch to test on VulkanTests were not being honored in jenkins.

cloneTests.bat:
cloneTests.sh:
runJob.bat:
runJob.sh:
runJobAndroid.sh:

  • Change TEST_COMMIT to TEST_BRANCH

Most CI scripts were using TEST_COMMIT but jenkinsfile.extended
and jenkinsfile.manual define it as TEST_BRANCH. Values were
not being honored in jenkins.

cloneTests.bat:
cloneTests.sh:
runJob.bat:
runJob.sh:
runJobAndroid.sh:
- Change TEST_COMMIT to TEST_BRANCH
@johnzupin johnzupin requested a review from beau-lunarg March 31, 2026 23:07
@johnzupin johnzupin requested a review from a team as a code owner March 31, 2026 23:07
@johnzupin johnzupin removed the request for review from a team March 31, 2026 23:09
@johnzupin johnzupin added the approved-to-run-ci Can run CI check on internal LunarG machines label Apr 1, 2026
@johnzupin
Copy link
Copy Markdown
Contributor Author

johnzupin commented Apr 1, 2026

http://jenkins:8080/job/gfxr-pipelines/job/gfxr-manual/141/parameters/ is a manual run has the change to gfxreconstruct and it checkouts the correct branch to VulkanTests.

Switched to a new branch 'zupin-wayland-authority-file'
branch 'zupin-wayland-authority-file' set up to track 'origin/zupin-wayland-authority-file'.
9ebd87a

http://jenkins:8080/job/gfxr-pipelines/job/gfxr-manual/144/parameters/ is a manual run without the change to gfxreconstruct and it fails to checkout the branch to VulkanTests, it just checkouts test.ref.

Note: switching to '2b3bb6b4be5554e50277495646fe0f230a06ff80'.

@johnzupin
Copy link
Copy Markdown
Contributor Author

Failure to internal CI, will look into it, but it is still displaying the correct behavior checking out what is in the test.ref file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-to-run-ci Can run CI check on internal LunarG machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant