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
Import sar data files #172
Conversation
files/Postgresql_performance.json
Outdated
@@ -24,7 +24,8 @@ | |||
"editable": true, | |||
"fiscalYearStartMonth": 0, | |||
"graphTooltip": 1, | |||
"id": 38, | |||
"id": 3, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are the changes here actually related to the PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I should have made a note about that, but the current Postgres dashboard has a misconfig where there's a default selection for the database sizes panel, so nothing shows up by default. I snuck that fix in here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it makes sense to pull that into another PR. you are now mixing an rnhancement with a bugfix in the same PT and commit. so reverting the enhancement will introduce a regression. And the dashboard fiy wont show up in the changelog.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, I took it out of this PR.
No description provided.