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

Loki: Make DNS resolver configurable in Gateway #5973

Merged
merged 2 commits into from
Apr 21, 2022
Merged

Loki: Make DNS resolver configurable in Gateway #5973

merged 2 commits into from
Apr 21, 2022

Conversation

irizzant
Copy link
Contributor

Signed-off-by: Ivan Rizzante i.rizzante@gmail.com

What this PR does / why we need it:
Currently it's not possible to configure the DNS resolver in Gateway, which is a problem if kube-dns Service does not exists.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Checklist

  • Documentation added
  • Tests updated
  • Is this an important fix or new feature? Add an entry in the CHANGELOG.md.
  • Changes that require user attention or interaction to upgrade are documented in docs/sources/upgrading/_index.md

Signed-off-by: Ivan Rizzante <i.rizzante@gmail.com>
@irizzant irizzant requested a review from a team as a code owner April 20, 2022 18:54
@irizzant
Copy link
Contributor Author

How am I supposed to make the lint stage pass?

@kavirajk
Copy link
Contributor

How am I supposed to make the lint stage pass?

@irizzant you can do make fmt-jsonnet.

@CLAassistant
Copy link

CLAassistant commented Apr 21, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@kavirajk kavirajk left a comment

Choose a reason for hiding this comment

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

LGTM!

@kavirajk kavirajk merged commit c700744 into grafana:main Apr 21, 2022
@irizzant irizzant deleted the dns-resolver branch April 21, 2022 06:57
@irizzant
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants