Skip to content
This repository was archived by the owner on Jul 28, 2026. It is now read-only.

[helm] Allow configuring dnsPolicy - #3447

Merged
rfratto merged 2 commits into
grafana-cold-storage:mainfrom
carlosjgp:helm-dns-policy
Apr 4, 2023
Merged

[helm] Allow configuring dnsPolicy#3447
rfratto merged 2 commits into
grafana-cold-storage:mainfrom
carlosjgp:helm-dns-policy

Conversation

@carlosjgp

Copy link
Copy Markdown
Contributor

PR Description

Allows configuring dnsPolicy especially important when enabling hostNetwork: true although not strictly necessary unless you have to resolve a DNS in the same cluster we are running the grafana agent pod

Related to this PR #728

Which issue(s) this PR fixes

N/A

Notes to the Reviewer

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

@rfratto rfratto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks!

@rfratto
rfratto merged commit 888d817 into grafana-cold-storage:main Apr 4, 2023
clayton-cornell pushed a commit that referenced this pull request Aug 14, 2023
* [helm] Allow configuring dnsPolicy

* [helm] Add more examples
clayton-cornell pushed a commit that referenced this pull request Aug 14, 2023
* [helm] Allow configuring dnsPolicy

* [helm] Add more examples
@github-actions github-actions Bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Mar 2, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants