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.

Sign up for GitHub

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

Jump to bottom

bugfix: can't specify nodePort #35

Merged

Jaesang merged 1 commit into main from fix-bug

Feb 22, 2021

Contributor

estherk0 commented Feb 22, 2021 •

edited

Loading

es 9200 port는 hotdata, master, http가 공통으로 갖는다.
http의 nodePort 30002로 세팅하고 싶지만 targetPort가 모두 9200으로 같아서 설정상 어떻게 해도 적용이 안되는 느낌.
따라서 type: NodePort로만 설정하면 다음과 같이 랜덤하게 nodePort가 잘 적용된다.

eck-elasticsearch-es-hotdata          ClusterIP   None            <none>        9200/TCP             30m
eck-elasticsearch-es-http             NodePort    10.233.5.88     <none>        9200:30928/TCP       16m
eck-elasticsearch-es-master           ClusterIP   None            <none>        9200/TCP             30m

random 하게 설정된 nodePort를 가져오도록 validate-lma job을 수정함


          bugfix: can't specify nodePort

5f2eb1b

estherk0 requested review from Jaesang, zugwan, intelliguy, robertchoi80 and seungkyua

February 22, 2021 02:14

estherk0 self-assigned this

zugwan approved these changes

View reviewed changes

Jaesang approved these changes

View reviewed changes

Jaesang merged commit dcf69ee into main

Jaesang deleted the fix-bug branch

February 22, 2021 02:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

zugwan zugwan approved these changes

Jaesang Jaesang approved these changes

intelliguy Awaiting requested review from intelliguy

robertchoi80 Awaiting requested review from robertchoi80

seungkyua Awaiting requested review from seungkyua

Labels

None yet