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

Reorder User Data #124

Merged
merged 1 commit into from
Aug 2, 2018
Merged

Reorder User Data #124

merged 1 commit into from
Aug 2, 2018

Conversation

lawliet89
Copy link
Collaborator

Currently, Fluentd and Prometheus/whatever time series DB can
potentially be run on Nomad. If the Nomad cluster is unhealthy
or the jobs are not running properly, new nodes will never be
able to bootstrap properly. We rearrange the order of launching
so that these nodes can still bootstrap their respective services.

On the downside, this means that we might not even know that the nodes
are not sending their logs.

Perhaps #44 will help?

Currently, Fluentd and Prometheus/whatever time series DB can
potentially be run on Nomad. If the Nomad cluster is unhealthy
or the jobs are not running properly, new nodes will never be
able to bootstrap properly. We rearrange the order of launching
so that these nodes can still bootstrap their respective services.

On the downside, this means that we might not even know that the nodes
are not sending their logs.

Perhaps #44 will help?
@lawliet89 lawliet89 requested a review from guangie88 August 2, 2018 01:41
lawliet89 added a commit that referenced this pull request Aug 2, 2018
Copy link
Collaborator

@guangie88 guangie88 left a comment

Choose a reason for hiding this comment

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

Yup, this is objectively better. The main functionalities are more important.

@lawliet89 lawliet89 merged commit 3c7c74d into master Aug 2, 2018
@lawliet89 lawliet89 deleted the reorder-user-data branch August 2, 2018 03:22
lawliet89 added a commit that referenced this pull request Aug 3, 2018
* Configure Telemetry for Consul via a new script

* Configure Nomad telemetry in Nomad configuration

* Procstat log

Also rearrange startup sequence

* Add for Vault and re-order user_data sequence

* Simplify Telegraf configuration

* Move telegraf and td-agent to the back

Replicates #124

* Update README

* Fix incorrect paths

* Rework Telegraf configuration

* Differentiate between consul server and agents

* Format Terraform

* Update README

* Add note about consul-template

* Update path

* Rename variable to `telegraf_enabled`

* Add missing flag

* Render all known nodes for ES

* Remove telegraf conf from Packer template

* Fix script errors

* Fix typo
qbiqing pushed a commit that referenced this pull request Mar 10, 2020
* Add statsd and update to consul hcl config

* Update to use inline config

* Update AMI with working inline config

* Update submodule

* Update submodule

* Use back the correct module commit

* Update again

* Update vendor to the right stuff

* Add td-agent module as part of core

* Fix previous mistake in td-agent.conf for Vault

* Fix another bug with td-agent-vars for Vault

* Update AMI with td-agent core integrataion

* Update vendor

* Update to use vendor commit on master
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.

2 participants