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.

Sign up for GitHub

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

Jump to bottom

Automatically overwrite dashboard UIDs with a prescribed naming pattern #224

Merged
merged 6 commits into from
Jan 21, 2025
Merged

Automatically overwrite dashboard UIDs with a prescribed naming pattern #224

merged 6 commits into from
Jan 21, 2025
+197 −99

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented Dec 17, 2024 •
edited
Loading

Issue

Solution

In tandem with:

Context

Testing Instructions

Tested with the new lib :

graph LR
ubuntu --- hardware-observer --- grafana-agent ---|CMR| grafana
ubuntu2 --- hardware-observer
ubuntu --- grafana-agent
ubuntu2 --- grafana-agent
Loading 
default-base: [email protected]/stable
saas:
  graf:
    url: microk8s:admin/graf.graf
applications:
  ga:
    charm: local:grafana-agent-11
  ho:
    charm: local:hardware-observer-0
    resources:
      perccli-deb: 1
      sas2ircu-bin: 1
      sas3ircu-bin: 1
      storcli-deb: 1
  ub:
    charm: ubuntu
    channel: latest/stable
    revision: 25
    num_units: 2
  ub2:
    charm: ubuntu
    channel: latest/stable
    revision: 25
    num_units: 1
relations:
- - ho:general-info
  - ub:juju-info
- - ho:general-info
  - ub2:juju-info
- - ho:cos-agent
  - ga:cos-agent
- - ga:juju-info
  - ub:juju-info
- - ga:juju-info
  - ub2:juju-info
- - ga:grafana-dashboards-provider
  - graf:grafana-dashboard

Upgrade Notes

See upgrade notes for canonical/grafana-k8s-operator#363.

@sed-i sed-i mentioned this pull request Dec 17, 2024
Merged
This was referenced Dec 17, 2024
Merged
Merged
@sed-i sed-i force-pushed the feature/dashboard_uid branch from 937ae39 to 99c2c2f Compare January 19, 2025 04:37
@sed-i sed-i force-pushed the feature/dashboard_uid branch from 99c2c2f to 6fa0266 Compare January 19, 2025 04:41
@sed-i sed-i marked this pull request as ready for review January 19, 2025 05:43
@sed-i sed-i requested a review from a team as a code owner January 19, 2025 05:43
@sed-i sed-i changed the title Autogenerate dashboard_alt_uid on the provider side Automatically overwrite dashboard UIDs with a prescribed naming pattern Jan 20, 2025
@sed-i sed-i mentioned this pull request Jan 20, 2025
Closed
@sed-i sed-i mentioned this pull request Jan 21, 2025
Merged
@sed-i sed-i merged commit ebda655 into main Jan 21, 2025
12 checks passed
@sed-i sed-i deleted the feature/dashboard_uid branch January 21, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lucabello lucabello lucabello approved these changes

Assignees
No one assigned
Labels
Libraries: Out of sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sed-i @lucabello