avirajkhare00 commented on PR #3386:
URL: https://github.com/apache/iggy/pull/3386#issuecomment-4704113207

   - gated helm smoke on scripts/ci/setup-helm-smoke-cluster.sh and 
scripts/ci/test-helm.sh in components.yml so this matrix actually runs on 
future script edits
   - bumped Envoy Gateway to v1.8.1 + Gateway API v1.5.1 (inside the EG compat 
matrix on kind k8s v1.35)
   - pinned EnvoyProxy envoyService to ClusterIP (default LB never gets an 
address on kind, so the Gateway never reached Programmed - caught by the 
now-running smoke job)
   - split get_gateway_base_url into find_gateway_service + 
start_gateway_port_forward so the port-forward PID lands in the parent shell; 
added a single trap cleanup_smoke_state EXIT covering the PID and the temp 
values file
   - find_gateway_service retries 15x with stderr suppressed for the transient 
missing-Service window after Gateway Programmed
   - dropped the redundant kubectl rollout status calls (helm --wait already 
blocks)
   - added HELM_SMOKE_GATEWAY_PF_PORT env override and fixed the misleading "15 
s" comment (worst case ~45 s)
   - documented HELM_SMOKE_GATEWAY_NAMESPACE / HELM_SMOKE_GATEWAY_NAME coupling 
in both scripts


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to