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

[GCE] Update hostname to be unique #1737

Merged
merged 4 commits into from
Jul 30, 2015
Merged

Conversation

tmichelet
Copy link
Contributor

fixes #1736

  • update the hostname to host_metadata['instance']['hostname'] so it's unique
  • drop some tags that we don't want to collect
  • add "instance_name.project_id" as a host alias

These tags are either not useful or bad security practices, let's
exclude them
The current hostname we use is not unique, if people have several
projects with instances with the same name, their hostname will be
the same, their tags will be wrong and their metrics will be aggre-
gated.
This commit fixes that, and adds an option on the config to keep
the backward compatibility.
As the hostname isn't available via the GCE API, we need this host
alias list to link metrics and tags from the crawler to the agent's
@@ -43,6 +43,9 @@ api_key:
# for more information
# collect_instance_metadata: yes

# use unique hostname for GCE hosts. see #1736
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's create a wiki page and add the link

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ping me when this is done

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tmichelet
Copy link
Contributor Author

@remh link updated, good to merge?

tmichelet added a commit that referenced this pull request Jul 30, 2015
@tmichelet tmichelet merged commit bc62a7e into master Jul 30, 2015
@tmichelet tmichelet deleted the tristan/gce-hostname-fix branch July 30, 2015 17:22
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 this pull request may close these issues.

[GCE] Non unique hostname
2 participants