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

[cloud-gce] warn message is displayed when not using gce discovery #13614

Closed
dadoonet opened this issue Sep 16, 2015 · 1 comment
Closed

[cloud-gce] warn message is displayed when not using gce discovery #13614

dadoonet opened this issue Sep 16, 2015 · 1 comment
Labels
:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs good first issue low hanging fruit help wanted adoptme v2.0.0-rc1

Comments

@dadoonet
Copy link
Member

Similar to #13581 (in master branch - 3.0):

GCE plugin now tries to start immediately gce discovery even if we don't set discovery.type: gce.
Not really an issue because I guess that if someone installs the cloud-gce plugin, it's probably to have discovery set to gce.

But we could may be do something better here...

[2015-09-16 13:04:11,048][WARN ][org.elasticsearch.discovery.gce] [Kubik] cloud.gce.project_id is not set.
[2015-09-16 13:04:11,048][WARN ][org.elasticsearch.discovery.gce] [Kubik] cloud.gce.zone is not set.
[2015-09-16 13:04:12,230][WARN ][org.elasticsearch.discovery.gce] [Kubik] cloud.gce.project_id is not set.
[2015-09-16 13:04:12,231][WARN ][org.elasticsearch.discovery.gce] [Kubik] cloud.gce.zone is not set.
[2015-09-16 13:04:12,231][WARN ][org.elasticsearch.discovery.gce] [Kubik] cloud.gce.project_id is not set.
[2015-09-16 13:04:12,231][WARN ][org.elasticsearch.discovery.gce] [Kubik] cloud.gce.zone is not set.
[2015-09-16 13:04:12,234][INFO ][org.elasticsearch.node   ] [Kubik] initialized
[2015-09-16 13:04:12,243][INFO ][org.elasticsearch.node   ] [Kubik] starting ...
[2015-09-16 13:04:12,437][INFO ][org.elasticsearch.transport.netty] [Kubik] Bound profile [default] to address {127.0.0.1:9300}
[2015-09-16 13:04:12,438][INFO ][org.elasticsearch.transport] [Kubik] bound_address {127.0.0.1:9300}, publish_address {127.0.0.1:9300}
[2015-09-16 13:04:12,445][INFO ][org.elasticsearch.discovery] [Kubik] elasticsearch/R_PjKk5fTB2zCV-jPcyQ3w
[2015-09-16 13:04:12,449][WARN ][org.elasticsearch.cloud.gce] [Kubik] disabling GCE discovery. Can not get list of nodes
[2015-09-16 13:04:13,955][WARN ][org.elasticsearch.cloud.gce] [Kubik] disabling GCE discovery. Can not get list of nodes
[2015-09-16 13:04:15,466][WARN ][org.elasticsearch.cloud.gce] [Kubik] disabling GCE discovery. Can not get list of nodes
@dadoonet dadoonet changed the title [cloud-gce] info message is displayed when not using gce discovery [cloud-gce] warn message is displayed when not using gce discovery Sep 16, 2015
@clintongormley clintongormley added the good first issue low hanging fruit label Sep 16, 2015
@xuzha xuzha closed this as completed in 625d06c Sep 29, 2015
@xuzha
Copy link
Contributor

xuzha commented Sep 29, 2015

Added 2.0, 2.1 labels, see the conversations in #13809

xuzha pushed a commit that referenced this issue Sep 29, 2015
GCE plugin tries to start immediately gce discovery even if we don't
set discovery.type. This commmit adds check `discovery.type` and
other required parameters before loading gce plugin.

closes #13614
xuzha pushed a commit that referenced this issue Sep 29, 2015
GCE plugin tries to start immediately gce discovery even if we don't
set discovery.type. This commmit adds check `discovery.type` and
other required parameters before loading gce plugin.

closes #13614
@clintongormley clintongormley added :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs and removed :Plugin Cloud GCE labels Feb 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs good first issue low hanging fruit help wanted adoptme v2.0.0-rc1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants