Skip to content

fix: align portal-tenants ApplicationSet repoURL to SSH#23

Merged
stxkxs merged 1 commit into
mainfrom
fix/portal-tenants-ssh-repourl
May 31, 2026
Merged

fix: align portal-tenants ApplicationSet repoURL to SSH#23
stxkxs merged 1 commit into
mainfrom
fix/portal-tenants-ssh-repourl

Conversation

@stxkxs
Copy link
Copy Markdown
Member

@stxkxs stxkxs commented May 31, 2026

The portal worker's git layer is SSH-only and pushes rendered tenant manifests to git@github.com:nanohype/tenants.git. This ApplicationSet used the HTTPS URL for both the directory generator and the template source, so ArgoCD and the portal disagreed on repo identity and an SSH credential wouldn't match.

Flips both repoURLs to the SSH form so producer (portal push) and consumer (ArgoCD pull) reference the same repo. Documents that ArgoCD needs a matching SSH repo credential (read-only deploy key on the private tenants repo) registered for this URL — the remaining consumer-half runtime step.

The portal worker's git layer is SSH-only — it pushes rendered tenant manifests
to the tenants GitOps repo over SSH (git@github.com:nanohype/tenants.git). This
ApplicationSet pointed at the HTTPS URL for both the directory generator and the
template source, so ArgoCD and the portal disagreed on the repo identity and a
registered SSH credential wouldn't match.

Flips both repoURLs to the SSH form so producer (portal push) and consumer
(ArgoCD pull) reference the same repo, and documents that ArgoCD needs a matching
SSH repo credential (a read-only deploy key on the private tenants repo)
registered for this URL.
@github-actions
Copy link
Copy Markdown

CI Results

Check Status
YAML Lint
Environment Kustomize Build
dev
staging
production

All validations passed.

@stxkxs stxkxs merged commit 74d611f into main May 31, 2026
5 checks passed
@stxkxs stxkxs deleted the fix/portal-tenants-ssh-repourl branch May 31, 2026 19:31
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