Skip to content

feat(argocd): ajoute le registre harbor du projet aux sources ArgoCD - #2705

Open
KepoParis wants to merge 1 commit into
mainfrom
feat/argocd-registry-oci-source
Open

feat(argocd): ajoute le registre harbor du projet aux sources ArgoCD#2705
KepoParis wants to merge 1 commit into
mainfrom
feat/argocd-registry-oci-source

Conversation

@KepoParis

Copy link
Copy Markdown
Contributor

Issues liées

Issues numéro: #1608 (reste ouverte, des changements front et probablement back sont encore nécessaires)


Quel est le comportement actuel ?

sourceRepositories ne contient que l'URL GitLab du projet et les dépôts
supplémentaires du plugin. Les charts poussés dans le registre Harbor du projet
ne sont pas déclarés comme source autorisée.

Quel est le nouveau comportement ?

sourceRepositories contient aussi oci://<host harbor>/<slug projet>/**.
L'URL est reconstruite depuis new URL(...).host, ce qui supprime le protocole
et tout slash final, donc pas de double slash.

L'entrée est omise si le registre est désactivé. ArgoCD lit USE_HARBOR et
HARBOR_URL dans sa propre config, avec la même règle que le ConditionalModule
qui enregistre RegistryModule (désactivé uniquement sur un false explicite).
Passer par harborConfigFactory aurait cassé le démarrage d'ArgoCD sans Harbor,
ses identifiants étant obligatoires.

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Tests ajoutés: USE_HARBOR=false avec HARBOR_URL défini, USE_HARBOR absent,
HARBOR_URL absent, et un cas service sans registre.

@KepoParis
KepoParis requested a review from a team as a code owner September 9, 2026 09:16
@KepoParis KepoParis self-assigned this Sep 9, 2026
@KepoParis KepoParis added the enhancement New feature or request label Sep 9, 2026
@KepoParis KepoParis added this to the 9.26.0 milestone Sep 9, 2026
@github-actions github-actions Bot added the built label Sep 9, 2026
@cloud-pi-native-sonarqube

Copy link
Copy Markdown

@shikanime shikanime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

plutot avoir une config generique sharedSourceRepositories ou un truc comme ca ? que d'avoir une dependance transverse vers Harbor

@KepoParis
KepoParis force-pushed the feat/argocd-registry-oci-source branch from c6b1db7 to 74dac96 Compare September 10, 2026 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

built enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants