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

ha-considerations.md: keepalived image missing curl #2478

Closed
jdefaver opened this issue May 18, 2021 · 3 comments · Fixed by #2481
Closed

ha-considerations.md: keepalived image missing curl #2478

jdefaver opened this issue May 18, 2021 · 3 comments · Fixed by #2481
Assignees
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/documentation Categorizes issue or PR as related to documentation. priority/backlog Higher priority than priority/awaiting-more-evidence.

Comments

@jdefaver
Copy link

What happened?

I used the documentation in ha-considerations.md to deploy kubernetes with haproxy and keepalived on static pods. I determined keepalived was not working as expected by looking at /var/log/messages on the host (journal: /etc/keepalived/check_apiserver.sh exited with status 1).

By connecting to the keepalived pod, I realized curl was missing:

$ kubectl exec -it  -n kube-system keepalived-kubernetes-control-2 -- /bin/bash
$ cd /etc/keepalived/
$ ./check_apiserver.sh 
./check_apiserver.sh: line 11: curl: not found
*** Error GET https://localhost:443/

What you expected to happen?

Keepalived check script returning 0 when the endpoint is reachable.

How to reproduce it (as minimally and precisely as possible)?

Deploy as stated in the documentation.

@neolit123
Copy link
Member

is this by using osixia/keepalived:1.3.5-1?

i think we just need a newer image tag:
osixia/docker-keepalived#29

try osixia/keepalived:2.0.20

/kind documentation
/priority backlog
/help

@k8s-ci-robot
Copy link
Contributor

@neolit123:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

is this by using osixia/keepalived:1.3.5-1?

i think we just need a newer image tag:
osixia/docker-keepalived#29

try osixia/keepalived:2.0.20

/kind documentation
/priority backlog
/help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-ci-robot k8s-ci-robot added kind/documentation Categorizes issue or PR as related to documentation. priority/backlog Higher priority than priority/awaiting-more-evidence. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. labels May 18, 2021
@neolit123 neolit123 added this to the v1.22 milestone May 18, 2021
@SataQiu
Copy link
Member

SataQiu commented May 20, 2021

/assign

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/documentation Categorizes issue or PR as related to documentation. priority/backlog Higher priority than priority/awaiting-more-evidence.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants