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

Add nomad env to envoy bootstrap #12959

Merged
merged 2 commits into from
Sep 22, 2022
Merged

Conversation

jorgemarey
Copy link
Contributor

@jorgemarey jorgemarey commented May 12, 2022

Just saw #12543 .We are doing things a bit different in a fork we have. This PR takes a different aproach and adds enviroment variables to the envoy boostrap hook. This along with hashicorp/consul#13049 allow us to configure things like the following in the consul configuration:

kind = "proxy-defaults"
name = "global"

Config {
      envoy_dogstatsd_url = "udp://172.26.64.1:9125"
      envoy_stats_tags = [
        "alloc_id=${NOMAD_ALLOC_ID}",
        "nomad_group=${NOMAD_GROUP_NAME}",
        "nomad_job=${NOMAD_JOB_NAME}",
        "nomad_namespace=${NOMAD_NAMESPACE}",
      ]
}

This can also be set on the nomad sidecar configuration.

@Amier3
Copy link
Contributor

Amier3 commented May 25, 2022

We might want to put a hold on this pending the changes in hashicorp/consul#13049

@tgross tgross added the stage/waiting-on-upstream This issue is waiting on an upstream PR review label Jun 6, 2022
@tgross tgross moved this from Needs Triage to In Progress in Nomad - Community Issues Triage Jun 6, 2022
@tgross tgross moved this from In Progress to Needs Roadmapping in Nomad - Community Issues Triage Jul 5, 2022
@shoenig
Copy link
Member

shoenig commented Sep 22, 2022

Unless I'm mistaken, closed by #14311

@shoenig shoenig closed this Sep 22, 2022
Nomad - Community Issues Triage automation moved this from Needs Roadmapping to Done Sep 22, 2022
@jorgemarey
Copy link
Contributor Author

Hi @shoenig. I guess that this specific case presented in the issue (adding the stats tags to envoy) is solved by #14311, but I don't think it's the only use case for passing the NOMAD_ environment variables to the envoy task. I think that providing those values to envoy could allow users to better modify their escape-hatchs if necesary. What do you think?

@shoenig
Copy link
Member

shoenig commented Sep 22, 2022

Sounds reasonable enough to me ~

@shoenig shoenig reopened this Sep 22, 2022
Nomad - Community Issues Triage automation moved this from Done to Needs Triage Sep 22, 2022
Copy link
Member

@shoenig shoenig left a comment

Choose a reason for hiding this comment

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

LGTM

@shoenig shoenig added this to the 1.4.0 milestone Sep 22, 2022
@shoenig shoenig removed the stage/waiting-on-upstream This issue is waiting on an upstream PR review label Sep 22, 2022
@shoenig shoenig merged commit 451ecf3 into hashicorp:main Sep 22, 2022
Nomad - Community Issues Triage automation moved this from Needs Triage to Done Sep 22, 2022
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants