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

spark-3.5-scala-2.13/3.5.4-r0: cve remediation #38452

Closed

Conversation

Copy link
Contributor Author

octo-sts bot commented Dec 28, 2024

Gen AI suggestions to solve the build error:

Based on the build error, I'll provide a structured analysis and solution:

• Detected Error: "Could not resolve dependencies for project org.apache.spark:spark-core_2.13:jar:3.5.4: The following artifacts could not be resolved: org.eclipse.jetty:jetty-server:jar:9.4.56"

• Error Category: Dependency

• Failure Point: Maven dependency resolution for spark-core_2.13 module

• Root Cause Analysis:
The build is failing because it cannot find Jetty server version 9.4.56 in the Maven repository. This appears to be an incorrect version specification as Jetty 9.4.56 is not a valid release version.

• Suggested Fix:
Update the jetty dependencies in the pom.xml (we need to modify pombump-properties.yaml) to use the latest stable 9.4.x version:

# In pombump-properties.yaml
properties:
  jetty.version: 9.4.54.v20240208  # Update to latest stable 9.4.x version

• Explanation:

  • Jetty 9.4.56 doesn't exist in Maven Central
  • Version 9.4.54.v20240208 is the latest stable release in the 9.4.x series
  • This aligns with Spark's compatibility requirements
  • The fix ensures all Jetty components are on the same version

• Additional Notes:

  • Make sure both jetty-server and jetty-servlets use the same version
  • Consider adding version management in properties section
  • Test locally before pushing changes
  • This maintains compatibility with existing Spark 3.5.x dependencies

• References:

…isories filed for each of the affected components

Signed-off-by: Mark McCormick <[email protected]>
@mamccorm
Copy link
Member

I removed the attempted pombumps for jetty, as advisories were required for them (see last comment)

@octo-sts octo-sts bot added bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. manual/review-needed labels Jan 15, 2025
@egibs egibs added the staging-build If this label is set on a PR, it will be built by elastic-build in staging label Jan 15, 2025
@jamie-albert
Copy link
Member

#39819

@mamccorm
Copy link
Member

This should be resolved by the work done in this PR:

@mamccorm mamccorm closed this Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. GHSA-2jc4-r94c-rp7h GHSA-8qv5-68g4-248j GHSA-g8m5-722r-8whq GHSA-j26w-f9rq-mr2q GHSA-rcjc-c4pj-xxrp manual/review-needed maven/pombump P0 This label indicates our scanning found CRITICAL CVEs for these packages. request-cve-remediation spark-3.5-scala-2.13/3.5.4-r0 staging-build If this label is set on a PR, it will be built by elastic-build in staging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants