[ACE-1975] Use az aks command invoke to deploy through Azure ARM API, which bypasses the private AKS API endpoint#521
Draft
osh0501 wants to merge 22 commits into
Draft
[ACE-1975] Use az aks command invoke to deploy through Azure ARM API, which bypasses the private AKS API endpoint#521osh0501 wants to merge 22 commits into
osh0501 wants to merge 22 commits into
Conversation
…ation step to confirm ./helm/ is accessible in pod working directory
…there with --file .
…ssing dependency (unzip)
…ack, read rg and cluster name from repo secrets
…lling upgrades can run old and new pods concurrently
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Replaced kubelogin with
az aks command invoke— Self-hosted runners behind corporate TLS proxy cannot validate AKS API certificates directly.az aks command invokeuses the Azure ARM API (authenticated via OIDC) to create temporary pods inside the cluster that run helm commands, avoiding proxy interference entirelyMerged parallel jobs into a single sequential job — Previously deploy-orchestrator and deploy-catalog ran as independent parallel jobs. Combined into one
deploy-servicesjob with conditional steps to prevent namespace CPU quota conflicts (2 CPU limit). Retained conditional deployment flexibility — Can still deploy orchestrator only, catalog only, or both via deploy_orchestrator/deploy_catalog input flags.Uploaded only
helm/directory to temporary pod — Usingcd helm && --file. uploads only the chart directory, avoiding "Request Entity Too Large" errors from uploading the entire repo checkoutRemoved kubelogin, aks-set-context, and use-kubelogin actions — No longer needed since
az aks command invokehandles cluster authentication through ARM APIFixed
--wait/--atomicredundancy —--atomicalready implies--wait; refactored to elif to avoid specifying both flags simultaneouslyInfrastructure dependency (separate repo: DI-azure-infra-k8s): Requires adding Calico network policy for
aks-commandnamespace (stg + prd) to allow the temporary pod to reach the K8s API server through the zero-trust networking tier.Type of change
Checklist