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

Update README.md #26

Merged

Conversation

rs-vertica
Copy link
Collaborator

@rs-vertica rs-vertica commented Nov 2, 2023

Revised README to prepare for public consumption:

  • Copy edits
  • badge links
  • link to docs/grafana

*Before going public, links to revision and dashboard ID need to be updated with correct links when the URLs exist

rs-vertica and others added 2 commits November 2, 2023 15:13
Complete revision:
- Copy edits
- badge links
- link to docs/grafana
@spilchen spilchen requested a review from jizhuoyu November 3, 2023 12:06
@spilchen
Copy link
Collaborator

spilchen commented Nov 3, 2023

I like the new format of the readme. @roypaulin do you mind taking a look?

@spilchen spilchen requested review from roypaulin and removed request for jizhuoyu November 3, 2023 12:07
@roypaulin
Copy link
Collaborator

I also like the new format. I think we should have a min vertica version and then add a max later when we know it. Also Shouldn't we change the min version to the current vertica release?

@spilchen
Copy link
Collaborator

spilchen commented Nov 3, 2023

I think we should have a min vertica version and then add a max later when we know it.

Yeah, the plan is to add a max once we make a change to the metrics that break dashboards (rename and/or remove of a metric). It is left out for now since none have a max. The version you see now is the min.

Also Shouldn't we change the min version to the current vertica release?

Technically the dashboards should work with 23.4.0 since we have added any new metrics in this PI.

@spilchen spilchen merged commit d8fe293 into vertica:main Nov 3, 2023
1 check passed
@spilchen spilchen deleted the feature/VER-88542-K8s-grafana-README_rjs branch November 3, 2023 12:55
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