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

bug: using the latest image(3.0.0-centos) to upgrade helm release failed: container cannot connect to etcd #389

Closed
xinxinh2020 opened this issue Nov 8, 2022 · 7 comments · Fixed by #390
Assignees

Comments

@xinxinh2020
Copy link

Current Behavior

When i upgraded the image of helm release from 2.15.0-centos(or 2.15.0-alpine) to the latest one, the latest container failed to run due to failing to connect to etcd cluster. The error message in container logs like:

/usr/local/openresty/luajit/bin/luajit ./apisix/cli/apisix.lua init
/usr/local/openresty/luajit/bin/luajit ./apisix/cli/apisix.lua init_etcd
Warning! Request etcd endpoint 'http://127.0.0.1:2379/version' error, connection refused, retry time=1
Warning! Request etcd endpoint 'http://127.0.0.1:2379/version' error, connection refused, retry time=2
request etcd endpoint 'http://127.0.0.1:2379/version' error, connection refused
all etcd nodes are unavailable

I have recorded all the steps to reproduce this bug using asciinema, see: https://asciinema.org/a/4q10QLD1kJw3N5viGAR7JgYd3

Expected Behavior

Upgraded successfully.

Error Logs

/usr/local/openresty/luajit/bin/luajit ./apisix/cli/apisix.lua init
/usr/local/openresty/luajit/bin/luajit ./apisix/cli/apisix.lua init_etcd
Warning! Request etcd endpoint 'http://127.0.0.1:2379/version' error, connection refused, retry time=1
Warning! Request etcd endpoint 'http://127.0.0.1:2379/version' error, connection refused, retry time=2
request etcd endpoint 'http://127.0.0.1:2379/version' error, connection refused
all etcd nodes are unavailable

Steps to Reproduce

https://asciinema.org/a/4q10QLD1kJw3N5viGAR7JgYd3

Environment

  • APISIX version (run apisix version): 3.0.0
  • Operating system (run uname -a):
  • OpenResty / Nginx version (run openresty -V or nginx -V):
  • etcd version, if relevant (run curl http://127.0.0.1:9090/v1/server_info):
  • APISIX Dashboard version, if relevant:
  • Plugin runner version, for issues related to plugin runners:
  • LuaRocks version, for installation issues (run luarocks --version):
@spacewander spacewander transferred this issue from apache/apisix Nov 8, 2022
@spacewander
Copy link
Member

Could you provide the helm chart commit version which you modify from?

@spacewander
Copy link
Member

BTW, we are developing the feature to use 3.0 image (which is not backward compatible) via a separate flag. So soon we don't need to manually modify the helm chart (which is not recommended).

@xinxinh2020
Copy link
Author

Could you provide the helm chart commit version which you modify from?

I think it is the latest version in the https://charts.apiseven.com repo:

helm search repo apisix
NAME                            	CHART VERSION	APP VERSION	DESCRIPTION
apisix/apisix                   	0.11.1       	2.15.0     	A Helm chart for Apache APISIX

@xinxinh2020
Copy link
Author

BTW, we are developing the feature to use 3.0 image (which is not backward compatible) via a separate flag. So soon we don't need to manually modify the helm chart (which is not recommended).

Is there a latest helm chart for 3.0 image ?
If not now, how can i manually modify the helm chart to upgrade to 3.0 image ?

@xinxinh2020
Copy link
Author

BTW, we are developing the feature to use 3.0 image (which is not backward compatible) via a separate flag. So soon we don't need to manually modify the helm chart (which is not recommended).

Is there a latest helm chart for 3.0 image ? If not now, how can i manually modify the helm chart to upgrade to 3.0 image ?

I think i have figured out the problem: the configuration structure in 3.0.0 is changed - The etcd field is moved to deployment: #7860

I will try to manually modify the configmap for the config.yaml later

Hope the latest helm chart to be released soon and If there's anything I can do, I would be pleased to help.

@spacewander
Copy link
Member

BTW, we are developing the feature to use 3.0 image (which is not backward compatible) via a separate flag.

We plan to carry out this feature this week.

@tzssangglass
Copy link
Member

I will try to manually modify the configmap for the config.yaml later

wait for #390 merged, I'm working on this.

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 a pull request may close this issue.

3 participants