Skip to content

Commit

Permalink
Merge pull request #91 from yuvipanda/all-star
Browse files Browse the repository at this point in the history
Explicitly set `allValues` for hub variable
  • Loading branch information
consideRatio authored Oct 19, 2023
2 parents c0dbe1c + 3ec1b7e commit 1aa2075
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 3 deletions.
5 changes: 4 additions & 1 deletion dashboards/jupyterhub.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,10 @@ local templates = [
// Allow viewing dashboard for multiple combined hubs
includeAll=true,
multi=true
),
) + {
// Explicitly set '$hub' to be `.*` when 'All' is selected, as we always use `$hub` as a regex
allValue: '.*',
},
];


Expand Down
5 changes: 4 additions & 1 deletion dashboards/usage-report.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,10 @@ local templates = [
// Allow viewing dashboard for multiple combined hubs
includeAll=true,
multi=false
),
) + {
// Explicitly set '$hub' to be `.*` when 'All' is selected, as we always use `$hub` as a regex
allValue: '.*',
},
];

local memoryUsageUserPods = barGaugePanel.new(
Expand Down
5 changes: 4 additions & 1 deletion dashboards/user.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,10 @@ local templates = [
// Allow viewing dashboard for multiple combined hubs
includeAll=true,
multi=true
),
) + {
// Explicitly set '$hub' to be `.*` when 'All' is selected, as we always use `$hub` as a regex
allValue: '.*',
},
template.new(
'user_pod',
datasource='$PROMETHEUS_DS',
Expand Down

0 comments on commit 1aa2075

Please sign in to comment.