Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

feat: Customizable command-line option "node-meta" to the agent #728

Merged
merged 1 commit into from
Jul 1, 2021

Conversation

carnei-ro
Copy link
Contributor

Allow receive map to customize the Metadata section of the Node (consul agent).

@lkysow
Copy link
Member

lkysow commented Dec 9, 2020

Thanks for the PR! Can you add unit tests please (see https://github.com/hashicorp/consul-helm/blob/master/CONTRIBUTING.md#writing-unit-tests).

Specifically I think we need cases for:

  1. By default -node-meta=pod-name:${HOSTNAME} is set
  2. You can add additional nodeMeta to the map.

Copy link
Contributor

@thisisnotashwin thisisnotashwin left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @carnei-ro

@thisisnotashwin thisisnotashwin merged commit 7b539d6 into hashicorp:master Jul 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants