-
Notifications
You must be signed in to change notification settings - Fork 512
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
Docs: move dashboards to hugo bundles #1514
Conversation
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
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.
Looks good to me with a nit about the find command.
I ran the local webserver and checked all the images look good.
Makefile
Outdated
@@ -468,7 +468,7 @@ mixin-serve: ## Runs Grafana (listening on port 3000) loading the mixin dashboar | |||
@./operations/mimir-mixin-tools/serve/run.sh | |||
|
|||
mixin-screenshots: ## Generates mixin dashboards screenshots. | |||
@rm -f docs/sources/images/dashboards/*.png | |||
@find docs/sources/operators-guide/visualizing-metrics/dashboards -name '*.png' | xargs rm |
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.
It's considered safer to use -delete
or -print0 | xargs -0 rm
to handle files with spaces and stuff
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 used -delete
Signed-off-by: Marco Pracucci <marco@pracucci.com>
* Moved dashboards to Hugo bundles Signed-off-by: Marco Pracucci <marco@pracucci.com> * Updated screenshot generator tool and screenshots Signed-off-by: Marco Pracucci <marco@pracucci.com> * Addressed review feedback Signed-off-by: Marco Pracucci <marco@pracucci.com>
What this PR does
This is the 1st PR to migrate docs to Hugo bundles. This PR addresses the dashboard images. I've also took the opportunity to update the screenshots, so we're ready for 2.0.0 final.
Which issue(s) this PR fixes or relates to
N/A
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]