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

[build] evmtool Dockerfile - update apt-get packages #8321

Merged
merged 2 commits into from
Feb 18, 2025

Conversation

macfarla
Copy link
Contributor

PR description

Fixes this build failure
#7 4.118 E: Version '20190909*' for 'ca-certificates-java' was not found

could see the ca-certificates-java gets pulled in even without this explicit install.

#6 32.17 Setting up openjdk-21-jre-headless:arm64 (21.0.6+7-1~24.04.1) ...
#6 32.19 update-alternatives: using /usr/lib/jvm/java-21-openjdk-arm64/bin/java to provide /usr/bin/java (java) in auto mode
#6 32.19 update-alternatives: using /usr/lib/jvm/java-21-openjdk-arm64/bin/jpackage to provide /usr/bin/jpackage (jpackage) in auto mode
#6 32.19 update-alternatives: using /usr/lib/jvm/java-21-openjdk-arm64/bin/keytool to provide /usr/bin/keytool (keytool) in auto mode
#6 32.19 update-alternatives: using /usr/lib/jvm/java-21-openjdk-arm64/bin/rmiregistry to provide /usr/bin/rmiregistry (rmiregistry) in auto mode
#6 32.19 update-alternatives: using /usr/lib/jvm/java-21-openjdk-arm64/lib/jexec to provide /usr/bin/jexec (jexec) in auto mode
#6 32.20 Processing triggers for libc-bin (2.39-0ubuntu8.3) ...
#6 32.20 Processing triggers for ca-certificates (20240203) ...
#6 32.20 Updating certificates in /etc/ssl/certs...
#6 32.35 0 added, 0 removed; done.
#6 32.35 Running hooks in /etc/ca-certificates/update.d...
#6 32.35 done.

Had to explicitly install adduser

Fixed Issue(s)

Fixes #8320

Tested with

gw ethereum:evmtool:distDocker
docker run  hyperledger/besu-evmtool:25.2-develop-3cd9384   --code=5B600080808060045AFA50600056
INSUFFICIENT_GAS
{"stateRoot":"0xa898ef8d5fb80ff2620fe4a6a6ba2379bec749406802bcf303a516b4e1cc2567","output":"0x","gasUsed":"0x2540be400","pass":false,"fork":"Prague","timens":70001025323,"time":70001025}

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • spotless: ./gradlew spotlessApply
  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests

Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
@macfarla macfarla changed the title [build] evmtool Dockerfile - update ca-certificates-java version [build] evmtool Dockerfile - update apt-get packages Feb 18, 2025
@macfarla macfarla enabled auto-merge (squash) February 18, 2025 11:14
@macfarla macfarla merged commit 1cbdcdb into hyperledger:main Feb 18, 2025
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

evmtool docker image publish failing
2 participants