-
-
Notifications
You must be signed in to change notification settings - Fork 255
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
Update reproducible comparing tests on linux (not docker image) #3687
Comments
Trying in adopitum jenkins with some necessary updates.
That is in linux_repro_build_compare.sh if it's rhel that some extra tools are needed to install https://github.com/adoptium/temurin-build/blob/master/tooling/reproducible/linux_repro_build_compare.sh#L27-L37. However jenkins doesn't have the permission. |
For non-rhel for now jq is need to install, which also has the permission issue. Can we ask the infra to install |
SCM_REF build issue #3710 |
Upstream build is triggered by personal temrurin-build ( could be the old repo openjdk-build) branch and sha, which will cause build failed -
@andrew-m-leonard should personal branch be supported? |
This happened if However I've tried the release version with main repo got the similar issue. - Using release jdk22 linux-x64 https://ci.adoptium.net/job/build-scripts/job/release-openjdk22-pipeline/5/artifact/target/linux/x64/temurin/OpenJDK22U-sbom_x64_linux_hotspot_22_36.json/*view*/ I got similar error
https://ci.adoptium.net/view/Test_grinder/job/Grinder/9302/consoleFull |
e95140b is the commit of branch https://github.com/adoptium/temurin-build/commits/v2024.03.01 , which is not in the master. Hence can't be found. Suppose reproducible comparing should support the release? @andrew-m-leonard |
Should it matter where the commitSHA is from ? |
Shouldn't be. But, according to the test the sha can't be found. |
I suspect this needs changing to not assume a git "branch" ? https://github.com/sophia-guo/openjdk-build/blob/33e187ebdab191756b3f6584ef3a24862e529d4c/test/system/reproducibleCompare/build.xml#L61 |
The build.xml only pre stages all shell scripts. https://github.com/sophia-guo/openjdk-build/blob/33e187ebdab191756b3f6584ef3a24862e529d4c/test/system/reproducibleCompare/build.xml#L61. The issue can be easily reproduced with master branch by following command
export $(TEST_RESROOT) to local temurin-build/tooling/reproducible. Download the jdkfile and sbom file to same directory. Update linux_repro_build_compare.sh to support local files. |
yeah, I am wondering if the SCM_REF for these examples in the SBOM should be empty ? |
The files are release files
With linux it should be easily reproduced by
|
@sophia-guo The jq command needs fixing, the git checkout command it is running is:
because jq is returning:
|
I see. Seems SBOM file content changed. For this specific sbomfile there are 4 components ( jdk, jre, static-lib, debug). All properties of those 4 components are duplicate except filename itself. |
The diff for reproducing release jdk22 is as following
That is for reproducing one the release file doesn't have those information, which is minor but not sure if it is expected. @andrew-m-leonard |
I thought that should be clean'd by this
|
Linux_repro_build_compare.sh doesn't use the repro_common.sh |
Ah yes, so we should re-use that clean method to make these compare if possible? |
We can do in separate issue or pr later. |
Part of #3683
fatal: Not a valid object name jdk-21.0.3+6_adopt-266-ge01a50b2cac
#3710The text was updated successfully, but these errors were encountered: