-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-18590. Publish SBOM artifacts #5281
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
Conversation
cc @steveloughran , @sunchao , @snmvaughan |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, pending CI.
- Confirmed 2.7.3 is the latest version of the plugin.
- Applied patch locally.
- Ran
mvn -B -T 8 clean install -DskipShade -DskipTests
. - Verified presence of cyclonedx.json and cyclonedx.xml files under ~/.m2/repository.
@dongjoon-hyun , thanks for driving this! I'm going to hold off committing until next week in case some of the others you cc'd want to comment.
Thank you, @cnauroth ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thank you, @sunchao ! |
💔 -1 overall
This message was automatically generated. |
Apache Yetus(Jenkins) error: unit job seems to pass although it has a
|
Signed-off-by: Chris Nauroth <cnauroth@apache.org> (cherry picked from commit 6f99558)
I merged this to trunk. I also cherry-picked to branch-3.3 and branch-3.2 after resolving a minor merge conflict. @dongjoon-hyun , thank you for the contribution! @sunchao , thank you for the code review. (Sorry, I forgot to mention you in the |
How does this work for shaded things the Hadoop build itself pulls in? |
Hi, @steveloughran . Basically, these are SBOM for jars. So, this PR aims to use
|
FYI, here is Apache ORC 1.8.2 RC1 vote artifact, @steveloughran . |
got it. will add to 3.3.5 too as this will be our first release with this |
Signed-off-by: Chris Nauroth <cnauroth@apache.org> (cherry picked from commit 6f99558)
Thank you, @steveloughran . |
ok, verified the artifacts get into the local mvn repo. i do like the xml version BTW, including all the signatures. makes it easier to spot tampering, doesn't it? also highlights how hadoop-common has way too many dependencies. we should see what we can do to prune back stuff, such as my #4996. every jar we cut saves many, many integration problems |
It's great. :)
We already did like the other official Apache artifacts. Please see the official Apache ORC 1.8.2. https://repo1.maven.org/maven2/org/apache/orc/orc-core/1.8.2/ |
Causes HADOOP-18641. cyclonedx maven plugin breaks on recent maven releases This reverts branch-3.3 commit b6fd322.
Causes HADOOP-18641. cyclonedx maven plugin breaks on recent maven releases This reverts commit 61f6628.
Causes HADOOP-18641. cyclonedx maven plugin breaks on recent maven releases This reverts commit 6f99558.
Causes HADOOP-18641. cyclonedx maven plugin breaks on recent maven releases This reverts commit 6f99558.
Description of PR
This PR aims to publish SBOM artifacts.
Here is an article to give some context.
Software Bill of Materials (SBOM) are additional artifacts containing the aggregate of all direct and transitive dependencies of a project. The US Government (based on NIST recommendations) currently accepts only the three most popular SBOM standards as valid, namely: CycloneDX, Software Identification (SWID) tag, Software Package Data Exchange® (SPDX).
This PR uses CycloneDX maven plugin, a lightweight software bill of materials (SBOM) standard designed for use in application security contexts and supply chain component analysis.
How was this patch tested?
Manually. For example,
hadoop-auth-3.4.0-SNAPSHOT.jar
will havehadoop-auth-3.4.0-SNAPSHOT-cyclonedx.xml
andhadoop-auth-3.4.0-SNAPSHOT-cyclonedx.json
SBOM files additionally.For code changes:
LICENSE
,LICENSE-binary
,NOTICE-binary
files?