Skip to content

EAI-5630 again aiwb only fix#767

Merged
woojae-siloai merged 4 commits into
mainfrom
EAI-5630-again-aiwb-only-fix
Jul 1, 2026
Merged

EAI-5630 again aiwb only fix#767
woojae-siloai merged 4 commits into
mainfrom
EAI-5630-again-aiwb-only-fix

Conversation

@woojae-siloai

@woojae-siloai woojae-siloai commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This PR adds "appDomain" helmparameter at cf-init-gitea-cm.yaml.

In short, this will work if there is a proper "2.0.0" version of aiwb-chart pointing envoy-gateway-system for its httproutes.

It is tested on a test machine with modification of

  • aiwb.repoVersion: "1.1.9" at root/values.yaml
  • at gitea's cluster-values as follows
aiwb:
  helmParameters:
    # KGATEWAY-NAMESPACE
    - name: kgateway.namespace
      value: "envoy-gateway-system"

aiwb.repoVersion: "2.0.0-rc.1" doesn't exist so "1.1.9" selected as a workaround to run aiwb. Plus "1.1.9" versions uses "kgateway.namespace: kgateway-system" so updating cluster-values was needed for testing.

The AIWB Helm chart defaults to appDomain: "PUBLIC-IP", but ArgoCD Application helmParameters (from cluster-values.yaml) have highest priority and override chart defaults. Without adding appDomain to helmParameters in gitea-init-job, the deployed AIWB uses the placeholder value, causing NEXTAUTH_URL to be https://aiwbui.PUBLIC-IP instead of the actual domain. This fix ensures gitea-init-job generates cluster-values.yaml with the correct appDomain override for AIWB_ONLY deployments.

@woojae-siloai woojae-siloai marked this pull request as ready for review July 1, 2026 10:56
@woojae-siloai woojae-siloai requested a review from a team as a code owner July 1, 2026 10:56
@woojae-siloai woojae-siloai changed the title Eai 5630 again aiwb only fix EAI-5630 again aiwb only fix Jul 1, 2026

@pre pre 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.

Nice! I was also hit by this yesterday.

@woojae-siloai woojae-siloai merged commit b9d8a41 into main Jul 1, 2026
14 checks passed
@woojae-siloai woojae-siloai deleted the EAI-5630-again-aiwb-only-fix branch July 1, 2026 11:07
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