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.

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

bug: yaml方式配置，还需要配置etcd #716

Closed
xsi640 opened this issue Oct 21, 2019 · 4 comments · Fixed by #737
Closed

bug: yaml方式配置，还需要配置etcd #716

xsi640 opened this issue Oct 21, 2019 · 4 comments · Fixed by #737
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@xsi640
Copy link

xsi640 commented Oct 21, 2019

启动报错。

/usr/local/openresty/luajit/bin/luajit: /usr/bin/apisix:515: attempt to index local 'etcd_conf' (a nil value)
stack traceback:
	/usr/bin/apisix:515: in function </usr/bin/apisix:507>
	/usr/bin/apisix:585: in main chunk
	[C]: at 0x55af6913e2c0
@membphis membphis added the bug Something isn't working label Oct 21, 2019
@membphis
Copy link
Member

welcome PR

@membphis membphis added this to the 0.9 milestone Oct 21, 2019
@moonming moonming added good first issue Good for newcomers help wanted Extra attention is needed labels Oct 22, 2019
@soulsoul
Copy link
Member

the etcd is data center, someone want to just use apisix by a configuration file ,avoiding the dependent of etcd. is right ? want to just configure the service / upstream in a ymal file ?

@moonming
Copy link
Member

the etcd is data center, someone want to just use apisix by a configuration file ,avoiding the dependent of etcd. is right ? want to just configure the service / upstream in a ymal file ?

APISIX already support that.

@moonming
Copy link
Member

the bug is here: https://github.com/iresty/apisix/blob/master/bin/apisix#L521
If user not use etcd as data center, we should not to read configures of etcd at all.

@membphis membphis mentioned this issue Oct 24, 2019
Merged
@moonming moonming removed this from the 0.9 milestone Oct 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bugfix: skipped to init etcd if use local file as config center.
4 participants
@soulsoul @membphis @xsi640 @moonming