-
Notifications
You must be signed in to change notification settings - Fork 543
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
Refactoring: simplified dashboards resources and networking panels utilities #3531
Conversation
…ilities 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.
LGTM
…ilities (#3531) Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com>
The backport to
To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new branch
git switch --create backport-3531-to-release-2.5 origin/release-2.5
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x b0a390ba2d830c83c28f9dcf4d0740f77ed625cf
# Push it to GitHub
git push --set-upstream origin backport-3531-to-release-2.5
git switch main
# Remove the local backport branch
git branch -D backport-3531-to-release-2.5 Then, create a pull request where the |
I think the reason why the backport via grafanabot failed is because this PR isn't merged into I'll remove the |
…ilities (grafana#3531) Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com>
…anels utilities (grafana#3531)" This reverts commit 62370a6.
…ilities (grafana#3531) (grafana#3578) Signed-off-by: Marco Pracucci <marco@pracucci.com> Signed-off-by: Marco Pracucci <marco@pracucci.com> (cherry picked from commit b0a390b) Co-authored-by: Marco Pracucci <marco@pracucci.com>
What this PR does
This is a dashboards refactoring PR, with no diff in the compiled dashboards. I'm introducing some utillities to build resources and networking panels based on the component name (all such functions are suffixed with
ByComponent
for clarity).Which issue(s) this PR fixes or relates to
Part of #3361
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]