Skip to content

EAI_5630_fix "--aiwb-only" flag#702

Draft
woojae-siloai wants to merge 2 commits intomainfrom
EAI_5630_fix_aiwb_only
Draft

EAI_5630_fix "--aiwb-only" flag#702
woojae-siloai wants to merge 2 commits intomainfrom
EAI_5630_fix_aiwb_only

Conversation

@woojae-siloai
Copy link
Copy Markdown
Contributor

@woojae-siloai woojae-siloai commented Apr 30, 2026

Summary

Problem Fixed
AIWB standalone deployments failing due to missing domain configuration in --aiwb-only mode

Core Change
Preserve appDomain parameter in AIWB-only deployments

What Changed:

  • Restored appDomain parameter in scripts/init-gitea-job/templates/cf-init-gitea-cm.yaml
  • Added back essential configuration:
# APP-DOMAIN (preserve from root/values.yaml)
- name: appDomain
  value: {{ .Values.domain | quote }}
  • Maintained standalone mode with standAloneMode: "true"

Impact
Before:

  • AIWB-only deployments missing domain context
  • Authentication redirects failing due to undefined domain
  • Bootstrap script --aiwb-only flag not preserving domain configuration
    After:
  • AIWB standalone deployments inherit domain from root values.yaml
  • Proper domain configuration for authentication flows
  • Seamless AIWB-only bootstrap functionality restored

Technical Details

  • Target Use Case: ./scripts/bootstrap.sh --aiwb-only
  • Configuration Source: Domain value templated from .Values.domain
  • Deployment Mode: Standalone AIWB with preserved domain context
  • Authentication Flow: Enables proper Keycloak redirect URLs

@woojae-siloai woojae-siloai changed the title Eai 5630 fix aiwb only EAI_5630_fix "--aiwb-only" flag Apr 30, 2026
@woojae-siloai
Copy link
Copy Markdown
Contributor Author

woojae-siloai commented Apr 30, 2026

@Q-Dub
Hi. I had to delete pre-hooks when I tested on pd3. These blocked keycloak argo app.
Every external-secret in keycloak namespace comes from openbao, so I think without pre-hook will be ok.
What do you think?
image

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.

1 participant