Alan-Jowett / CoPilot-For-Consensus / 20650093156
78%

DEFAULT BRANCH: main
Ran
Jobs 0
Files 0
Run time –
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
20650093156

push

github

web-flow 
feat(azure): Make Grafana optional for Azure deployments (#681)

* feat(azure): make Grafana optional for Azure deployments

- Remove GRAFANA_BACKEND from Azure gateway container app env
- Update nginx.conf to dynamically include/exclude Grafana block based on env
- Add conditional logic in 30-substitute-env.sh to strip Grafana block when not set
- Make UI AdminLinks conditionally show Grafana link via REACT_APP_GRAFANA_ENABLED
- Add REACT_APP_GRAFANA_ENABLED=true to Docker Compose ui service
- Expose REACT_APP_GRAFANA_ENABLED in Vite config for client code
- Add Key Vault Secrets Officer role assignment for JWT identity (RBAC fix)
- Refactor deploy.ps1 to use az CLI (parity with deploy.sh)

Grafana is now Docker Compose-only; Azure deployments skip it entirely.
Gateway no longer fails to start due to missing grafana DNS resolution.

Fixes: Gateway NGINX startup failure on Azure
Related: #<issue-number>
Signed-off-by: GitHub Copilot <copilot@github.com>

* fix(ui): add ts-ignore for process.env in AdminLinks

TypeScript compiler couldn't find process.env at build time.
Added @ts-ignore since Vite defines process.env via define config.

Signed-off-by: GitHub Copilot <copilot@github.com>

* fix(dockerfile): add REACT_APP_GRAFANA_ENABLED build arg

Pass REACT_APP_GRAFANA_ENABLED arg through Dockerfile so Vite
can define it during build. Docker Compose ui service sets it to true.

Signed-off-by: GitHub Copilot <copilot@github.com>

* fix: improve portability and correctness in deployment configs

- docker-compose.services.yml: Remove redundant runtime environment variable for REACT_APP_GRAFANA_ENABLED (build arg is sufficient for Vite)
- infra/nginx/nginx.conf: Add quotes around \ variable for safer envsubst substitution
- infra/nginx/30-substitute-env.sh: Replace non-portable sed -i with explicit temp file approach for macOS compatibility
- infra/azure/deploy.ps1: Fix stderr redirection to preserve JSON parsing; correct Azure CLI parameters syntax (no du... (continued)
Source Files on build 20650093156
Detailed source file information is not available for this build.