Skip to content

fix: install netcat-openbsd instead of netcat in test image build#4476

Merged
hezhangjian merged 1 commit intoapache:masterfrom
hezhangjian:fix-netcat
Jul 29, 2024
Merged

fix: install netcat-openbsd instead of netcat in test image build#4476
hezhangjian merged 1 commit intoapache:masterfrom
hezhangjian:fix-netcat

Conversation

@hezhangjian
Copy link
Member

No description provided.

@lhotari
Copy link
Member

lhotari commented Jul 28, 2024

 The command '/bin/sh -c wget -nv https://archive.apache.org/dist/zookeeper/bookkeeper/bookkeeper-4.1.0/bookkeeper-server-4.1.0-bin.tar.gz{,.sha1,.md5,.asc}' returned a non-zero code: 8

It looks like the command doesn't get properly expanded. I wonder if using bash shell would address the problem.

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
@hezhangjian
Copy link
Member Author

@lhotari I change FROM eclipse-temurin:8-jdk to FROM eclipse-temurin:8-jdk-jammy, to keep style consistent with other ubuntu docker image.

@lhotari
Copy link
Member

lhotari commented Jul 29, 2024

@lhotari I change FROM eclipse-temurin:8-jdk to FROM eclipse-temurin:8-jdk-jammy, to keep style consistent with other ubuntu docker image.

Oh yes, makes sense. It seems that things broken when eclipse-temurin switched to use Ubuntu 24.04 (noble) base image. Switching to jammy (22.04) makes the existing commands to work.

@hezhangjian hezhangjian merged commit 5662524 into apache:master Jul 29, 2024
@hezhangjian hezhangjian deleted the fix-netcat branch July 29, 2024 06:33
hangc0276 pushed a commit that referenced this pull request Aug 5, 2024
)

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
(cherry picked from commit 5662524)
hangc0276 pushed a commit that referenced this pull request Aug 5, 2024
)

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
(cherry picked from commit 5662524)
hangc0276 pushed a commit that referenced this pull request Aug 5, 2024
)

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
(cherry picked from commit 5662524)
horizonzy pushed a commit to horizonzy/bookkeeper that referenced this pull request Aug 5, 2024
…ache#4476)

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
(cherry picked from commit 5662524)
hangc0276 pushed a commit that referenced this pull request Aug 5, 2024
#4470)

* Fix ReadOnlyLedgerHandle leak issue when checkAllLedgers.

* fix: install netcat-openbsd instead of netcat in test image build (#4476)

Signed-off-by: ZhangJian He <shoothzj@gmail.com>
(cherry picked from commit 5662524)

---------

Co-authored-by: ZhangJian He <shoothzj@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments