Skip to content

Commit

Permalink
Explicitly set allowAll for all variables includeAll
Browse files Browse the repository at this point in the history
A follow-up to jupyterhub#91,
as the openscapes community reported this is causing issues
not just for the $hub variable but everything else as well.

jupyterhub#90 is
the long term fix
  • Loading branch information
yuvipanda committed Jul 25, 2024
1 parent b1b62b8 commit db914b0
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions dashboards/user.jsonnet
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,10 @@ local templates = [
// Allow viewing dashboard for multiple users
includeAll=true,
multi=true
),
) + {
// Explicitly set '$user_pod' to be `.*` when 'All' is selected, as we always use `$user_pod` as a regex
allValue: '.*',
},
template.new(
// Queries should use the 'instance' label when querying metrics that
// come from collectors present on each node - such as node_exporter or
Expand All @@ -50,7 +53,10 @@ local templates = [
// Allow viewing dashboard for multiple nodes
includeAll=true,
multi=true
),
) + {
// Explicitly set '$instance' to be `.*` when 'All' is selected, as we always use `$instance` as a regex
allValue: '.*',
},
];


Expand Down

0 comments on commit db914b0

Please sign in to comment.