Skip to content

Make java-test-image usable for testing in k8s#10102

Merged
sijie merged 1 commit intoapache:masterfrom
lhotari:lh-make-java-test-image-usable-in-k8s
Apr 1, 2021
Merged

Make java-test-image usable for testing in k8s#10102
sijie merged 1 commit intoapache:masterfrom
lhotari:lh-make-java-test-image-usable-in-k8s

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Mar 31, 2021

Motivation

When developing and testing Pulsar, it is very slow to build the pulsar-all image.
The java-test-image can be built in less than 1.5 minutes on most laptops.
When attempting to use the apachepulsar/java-test-image:latest with a local k8s setup (microk8s), it was found out that startup fails since the Pulsar helm charts expect that nc and nslookup are installed.

Modifications

Make nc and nslookup available to the java-test-image by adding netcat, dnsutils and iputils-ping to the packages to be installed. These packages are installed to the pulsar image and were missing from the java-test-image.
This change makes java-test-image usable for k8s installations (for developing and testing Pulsar itself).

- nc and nslookup are required by the pulsar helm charts
lhotari added a commit to lhotari/pulsar-playground that referenced this pull request Mar 31, 2021
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lhotari
Copy link
Member Author

lhotari commented Mar 31, 2021

/pulsarbot run-failure-checks

3 similar comments
@lhotari
Copy link
Member Author

lhotari commented Mar 31, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 1, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 1, 2021

/pulsarbot run-failure-checks

@sijie sijie added this to the 2.8.0 milestone Apr 1, 2021
@sijie sijie merged commit 9fd7b18 into apache:master Apr 1, 2021
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.

3 participants

Comments