You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "lhotari (via GitHub)" <gi...@apache.org> on 2024/06/07 14:34:55 UTC
[PR] Make zookeeper healthchecks compatible with Alpine's busybox nc [pulsar-helm-chart]
lhotari opened a new pull request, #504:
URL: https://github.com/apache/pulsar-helm-chart/pull/504
### Motivation
The apachepulsar/pulsar:3.3.0 image contains busybox nc which doesn't support `-q 1` command line option.
See #22872.
### Modifications
replace `-q 1` with the use of `sleep 1` in the input.
Solution from https://unix.stackexchange.com/a/274603.
### Verifying this change
- [ ] Make sure that the change passes the CI checks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Make zookeeper healthchecks compatible with Alpine's busybox nc [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #504:
URL: https://github.com/apache/pulsar-helm-chart/pull/504#issuecomment-2155169975
Some DNS config issue:
```
pulsar-ci-pulsar-init-pfdv8:/pulsar$ nslookup pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar.svc.cluster.local
Server: 10.96.0.10
Address: 10.96.0.10:53
Name: pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar.svc.cluster.local
Address: 10.244.1.17
pulsar-ci-pulsar-init-pfdv8:/pulsar$ nslookup pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar
Server: 10.96.0.10
Address: 10.96.0.10:53
** server can't find pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar: NXDOMAIN
** server can't find pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar: NXDOMAIN
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Make zookeeper healthchecks compatible with Alpine's busybox nc [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #504:
URL: https://github.com/apache/pulsar-helm-chart/pull/504#issuecomment-2155173197
found https://github.com/k3s-io/k3s/issues/6561#issuecomment-1443590459 , but not a proper resolution.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Make zookeeper healthchecks compatible with Alpine's busybox nc [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #504:
URL: https://github.com/apache/pulsar-helm-chart/pull/504#issuecomment-2155168715
There's some other issue with the Alpine image:
```
Server: 10.96.0.10
Address: 10.96.0.10:53
** server can't find pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar: NXDOMAIN
** server can't find pulsar-ci-zookeeper-0.pulsar-ci-zookeeper.pulsar: NXDOMAIN
```
/etc/resolv.conf is
```
search pulsar.svc.cluster.local svc.cluster.local cluster.local 1evhaya2a4vu1idbbq22av5vmh.bx.internal.cloudapp.net
nameserver 10.96.0.10
options ndots:5
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Make zookeeper healthchecks compatible with Alpine's busybox nc [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari commented on PR #504:
URL: https://github.com/apache/pulsar-helm-chart/pull/504#issuecomment-2155199759
Looks like busybox nslookup isn't compatible. https://github.com/docker-library/busybox/issues/48#issuecomment-2069324947
will need `apk add bind-tools` in Alpine image.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Make zookeeper healthchecks compatible with Alpine's busybox nc [pulsar-helm-chart]
Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari merged PR #504:
URL: https://github.com/apache/pulsar-helm-chart/pull/504
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org