-
Notifications
You must be signed in to change notification settings - Fork 278
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
rename neo4j package according to version stream #38299
Conversation
Gen AI suggestions to solve the build error: Based on the error log, I'll analyze and provide a solution: • Detected Error: None visible in the log - the build appears to stop after the git checkout step without an explicit error message • Error Category: Build/Configuration • Failure Point: After git checkout step, before maven build can start • Root Cause Analysis: The build appears to silently fail after the git checkout step, likely due to missing environment setup or directory permission issues common in Java/Maven builds. • Suggested Fix:
pipeline:
- uses: git-checkout
with:
repository: https://github.com/neo4j/neo4j
tag: ${{package.version}}
expected-commit: 17dc7f7e1787c451938b97666a06afd3695987ca
- uses: maven/pombump
- runs: |
export JAVA_HOME=/usr/lib/jvm/java-17-openjdk
export PATH=$JAVA_HOME/bin:$PATH
export MAVEN_OPTS="-Xmx2048m -Dmaven.repo.local=/tmp/m2"
export HOME=/tmp
mvn package -DskipTests=true -T$(nproc)C -q • Explanation:
• Additional Notes:
• References:
|
9a7461f
to
ad16de6
Compare
Gen AI suggestions to solve the build error: • Detected Error: • Error Category: Build Configuration • Failure Point: • Root Cause Analysis: • Suggested Fix: pipeline:
- uses: git-checkout
with:
repository: https://github.com/neo4j/neo4j
tag: ${{package.version}}
expected-commit: 17dc7f7e1787c451938b97666a06afd3695987ca
- runs: |
export LANG=en_US.UTF-8
export MAVEN_OPTS="-Xmx2048m"
mvn package -DskipTests=true -T$(nproc)C -q
# ... rest of the build steps • Explanation: • Additional Notes:
• References:
|
ad16de6
to
de0a703
Compare
d0010fa
to
f0e8ebb
Compare
Gen AI suggestions to solve the build error: • Detected Error:
• Error Category: Dependency • Failure Point: Maven dependency resolution during build phase • Root Cause Analysis: • Suggested Fix:
Example fix for pombump-deps.yaml: properties:
jetty.version: "12.0.11" • Explanation: • Additional Notes:
• References:
|
f0e8ebb
to
2bd0b43
Compare
Gen AI suggestions to solve the build error: • Detected Error:
• Error Category: Dependency • Failure Point: Maven dependency resolution during build phase • Root Cause Analysis: The build is failing because it cannot find Jetty 12.0.12 artifacts. The pombump step updated the Jetty version from 10.0.24 to 12.0.12, but this version appears to be incompatible or unavailable. • Suggested Fix:
- property: jetty.version
value: "10.0.19" # or another stable 10.x version • Explanation:
• Additional Notes:
• References:
|
2bd0b43
to
0eefd86
Compare
Gen AI suggestions to solve the build error: • Detected Error:
• Error Category: Build Configuration • Failure Point: Maven licensing plugin check during build process • Root Cause Analysis: The licensing-maven-plugin is failing because there's a mismatch between the generated NOTICE.txt file and the existing one. This is a common issue when building Neo4j from source, as the licensing plugin performs strict checks on license files. • Suggested Fix: - runs: |
export LANG=en_US.UTF-8
export MAVEN_OPTS="-Xmx2048m"
mvn package -DskipTests=true -Doverwrite -T$(nproc)C -q • Explanation: • Additional Notes:
• References:
|
0eefd86
to
a69b563
Compare
Gen AI suggestions to solve the build error: • Detected Error:
• Error Category: Dependency • Failure Point: Maven enforcer plugin dependency convergence check during build • Root Cause Analysis: There are conflicting versions of Jetty dependencies in the project - some components are using Jetty 12.0.12 while others are using 10.0.24 • Suggested Fix:
- uses: maven/pombump
with:
patch-file: pombump-deps.yaml
properties:
jetty.version: 12.0.12
force-dependencies:
- org.eclipse.jetty:jetty-http:12.0.12
- org.eclipse.jetty:jetty-util:12.0.12
- org.eclipse.jetty:jetty-io:12.0.12
- org.eclipse.jetty.http2:*:12.0.12 • Explanation: The Maven enforcer plugin is failing because it detected multiple versions of Jetty dependencies in the dependency tree. By explicitly managing all Jetty dependencies to use version 12.0.12, we ensure dependency convergence. • Additional Notes:
• References: |
a69b563
to
688d40f
Compare
Signed-off-by: Batuhan Apaydin <batuhan.apaydin@chainguard.dev>
688d40f
to
5504125
Compare
Neo4j is a version streamed package so according to that we should rename the package for making it version streamed compliant.
For new version streams
name: ${{package.name}}-compat
)provides:
logical unversioned forms of the package (e.g.nodejs
,nodejs-lts
)