Skip to content

Need the latest changes to seaweed branch#607

Draft
pwistbac wants to merge 433 commits intoEAI-1452-seaweed-s3from
main
Draft

Need the latest changes to seaweed branch#607
pwistbac wants to merge 433 commits intoEAI-1452-seaweed-s3from
main

Conversation

@pwistbac
Copy link
Copy Markdown
Contributor

No description provided.

@pwistbac pwistbac requested a review from a team as a code owner February 24, 2026 09:27
Q-Dub and others added 29 commits March 17, 2026 15:53
fix(bootstrap.sh): argocd - cleanup /tmp files via mktemp -d
…-openbao

fix: remove not needed aiwb-cnpg-user-username in openbao init
feat: make import_databases.sh compatible with post helm infra split
fix: move airm and aiwb ignoredifferences for externalsecrets to new …
Q-Dub and others added 23 commits April 29, 2026 12:55
EAI-6059 Prepare Pluggable S3 instructions for changes in upstream
…of ociRegistry key which contains both Docker and ghcr
…idance

- Add banner linking to core repo Helm Installation Guide for full platform
- Add AIRM components (API, Agent, RabbitMQ, PostgreSQL) to Component Reference
- Document devuser@localhost email limitation when layering AIRM
- Add Validation section with E2E test links to core repo
- Add custom GPU metrics labels guidance (AIRM integration) to gpu_operator.md

Co-authored-by: Cursor <cursoragent@cursor.com>
…stall-doc

EAI-5646: Cross-reference core INSTALL.md and add AIRM integration guidance
The aiwb-cnpg-user and keycloak-cnpg-user Secrets in
secrets-override-hardcoded.yaml duplicated identical content already
created by secrets-aiwb-cnpg.yaml in PLUGGABLE_DB=false mode, and were
orphaned (referenced no chart) in PLUGGABLE_DB=true mode. Only the
default-user MinIO Secret remains, since its API_ACCESS_KEY value is
coupled to a hardcoded reference in silogen/openbao-config.
Replace the static secrets-aiwb-cnpg.yaml with inline `kubectl create secret`
calls in install_base.sh, populated from AIWB_DB_USER/PASSWORD,
KEYCLOAK_DB_USER/PASSWORD, and new AIWB_CNPG_SUPERUSER_* /
KEYCLOAK_CNPG_SUPERUSER_* env vars (default: placeholder). The CNPG cluster
now bootstraps with the same username/password the AIWB and Keycloak charts
read at startup, so the override step in secrets-override-hardcoded.yaml is
no longer needed for the *-cnpg-user secrets. Pass --set postgresql.username
unconditionally to the AIWB and Keycloak helm template calls so the chart
matches the bootstrapped CNPG user in both PLUGGABLE_DB modes.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
…e openbao-config

Two related changes:

1. Standalone helm install path (docs/manual_helm_install): replace static
   secrets-aiwb-minio.yaml + secrets-override-hardcoded.yaml with inline
   `kubectl create secret` calls in install_base.sh, populated from new
   MINIO_API_ACCESS_KEY / MINIO_API_SECRET_KEY / MINIO_CONSOLE_ACCESS_KEY /
   MINIO_CONSOLE_SECRET_KEY env vars (all default: placeholder). The same
   API_* pair feeds both `default-user` (Tenant bootstrap) and
   `minio-credentials` (AIWB / workbench) so the credentials match by
   construction. Renames MINIO_ACCESS_KEY / MINIO_SECRET_KEY →
   MINIO_API_ACCESS_KEY / MINIO_API_SECRET_KEY in the PLUGGABLE_S3=true
   branch and helper script (s3_minio_container.sh prints the new names).
   Both static YAMLs are now redundant and removed.

2. openbao-config chart: parametrize the previously hardcoded
   `api-default-user` and `default-user` access keys via new values
   `minio.apiAccessKey` / `minio.consoleAccessKey`. Defaults preserve current
   behaviour for the Argo install path; integrators can override via
   `--set minio.apiAccessKey=...`.

Co-Authored-By: Claude Opus 4 <noreply@anthropic.com>
Restructure both component docs to cover in-cluster (default) and external
(pluggable) modes with concrete kubectl/helm commands the reader can run
directly. Drop references to install_base.sh and other shell scripts so the
markdown stands on its own; the scripts are now an executable companion to
these docs rather than their source of truth.
feat: switch to oci images for aim-enigne and aim-engine-crds
config(cluster-auth): point to 0.5.9 (cache invalidation)
EAI-6099 Parametrize Minio & Postgresql usernames and OpenBAO domain in Helm
@pre pre marked this pull request as draft May 7, 2026 13:33
Q-Dub and others added 6 commits May 8, 2026 12:23
This commit selectively reverts parts of PR #692 (EAI-5893) that added
support for multiple helm value files.

Changes:
- Removed valuesFiles support from single-source app definitions in cluster-apps.yaml
- Modified bootstrap.sh render_actual_helm_manifests() to use valuesFile (singular) instead of valuesFiles
- Reverted SBOM validation script to pre-EAI-5893 version
- Removed sources/kaiwo/values.yaml

Kept unchanged:
- root/values.yaml (kaiwo OCI configuration - newer functionality depends on this)
- sbom/components.yaml (current kaiwo metadata is correct)
- sbom/generate-compare-components.sh (already updated for OCI support in commit 089da0c)
- OCI multi-source support in cluster-apps.yaml (valuesFiles retained for OCI functionality)
- bootstrap.sh render_actual_helm_manifests() function (modified to remove valuesFiles, kept the function)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
OCI-based apps like kaiwo use repoURL instead of path. The validation
script needs to check for app existence (not just non-null path) and
normalize empty/null values for proper comparison.

This fixes the CI failure where kaiwo showed as:
  cluster-configs='' vs components.yaml='null'
Replace the chart field with path for OCI repositories in ArgoCD Application specs, aligning with modern ArgoCD OCI support. Defaults to "." when path is not explicitly defined.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Partially revert EAI-5893: Remove valuesFiles from single-source apps
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.

9 participants