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

Improve provisioning example in README.md #444

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

schroedermatthias
Copy link
Contributor

@schroedermatthias schroedermatthias commented Aug 27, 2024

The provided provisioning example were incomplete and erroneous.
I merged them (not obivous why log access need different setup) and documented them inline.
This example is proven to be working.

What this PR does / why we need it:

  • Improved documentation
  • Less frustration during setup
  • No reverse-engineering of typescript needed

Which issue(s) this PR fixes:

  • Bad / outdated provisioning example

The provided provisioning example were incomplete and erroneous. I merged them and documented them inline.
This example is proven to be working.

Removed the typographic quotes which are no valid yaml
@schroedermatthias schroedermatthias requested a review from a team as a code owner August 27, 2024 20:37
@schroedermatthias schroedermatthias requested review from idastambuk and kevinwcyu and removed request for a team August 27, 2024 20:37
@CLAassistant
Copy link

CLAassistant commented Aug 27, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@idastambuk idastambuk left a comment

Choose a reason for hiding this comment

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

Hi @schroedermatthias thanks for the contribution, this looks good. And thank you for the patience with us getting to this!

@idastambuk idastambuk merged commit 701ed80 into grafana:main Oct 18, 2024
2 checks passed
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.

3 participants