Skip to content

Use files get to load istio dashboard#779

Merged
awhdesmond merged 1 commit into
mainfrom
awhdesmond/istio-dashboard-helm-file-get
Jul 6, 2026
Merged

Use files get to load istio dashboard#779
awhdesmond merged 1 commit into
mainfrom
awhdesmond/istio-dashboard-helm-file-get

Conversation

@awhdesmond

@awhdesmond awhdesmond commented Jul 4, 2026

Copy link
Copy Markdown
Contributor
  1. update-istio.sh will download the latest dashboards and save them locally.
  2. These JSON files will be exported as a BAZEL filegroup.
  3. During template rendering, we use Helm Files.Get to load the JSON files and avoid helm templating issue
bazel build //src/app_charts/base:base-cloud && bazel-bin/src/app_charts/base/app_management_test.runfiles/+non_module_deps+kubernetes_helm/helm template bazel-bin/src/app_charts/base/base-cloud-0.0.1.tgz -x templates/istio-dashboards.yaml --set-string use_istio=true | grep "name: istio-dashboard" | awk '{print $2}'


INFO: Analyzed target //src/app_charts/base:base-cloud (1205 packages loaded, 27375 targets configured).
INFO: Found 1 target...
Target //src/app_charts/base:base-cloud up-to-date:
  bazel-bin/src/app_charts/base/base-cloud-0.0.1.tgz
INFO: Elapsed time: 1.905s, Critical Path: 0.07s
INFO: 1 process: 1080 action cache hit, 1 internal.
INFO: Build completed successfully, 1 total action
istio-dashboard-istio-mesh-dashboard
istio-dashboard-istio-performance-dashboard
istio-dashboard-istio-service-dashboard
istio-dashboard-istio-workload-dashboard
istio-dashboard-pilot-dashboard

@awhdesmond awhdesmond marked this pull request as ready for review July 6, 2026 01:20
@awhdesmond awhdesmond requested review from Tobias-Pe and ensonic July 6, 2026 01:20

@ensonic ensonic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@awhdesmond awhdesmond merged commit c0366cc into main Jul 6, 2026
7 checks passed
@awhdesmond awhdesmond deleted the awhdesmond/istio-dashboard-helm-file-get branch July 6, 2026 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants