Skip to content

feat: disable git-server on git migration#3177

Open
CasLubbers wants to merge 98 commits intomainfrom
APL-1788
Open

feat: disable git-server on git migration#3177
CasLubbers wants to merge 98 commits intomainfrom
APL-1788

Conversation

@CasLubbers
Copy link
Copy Markdown
Contributor

📌 Summary

🔍 Reviewer Notes

🧹 Checklist

  • Code is readable, maintainable, and robust.
  • Unit tests added/updated

@CasLubbers CasLubbers changed the title Apl 1788 feat: disable git-server on git migration Apr 23, 2026
@CasLubbers
Copy link
Copy Markdown
Contributor Author

Should be merged after: #3103
First rebase with main after that merge.

@CasLubbers CasLubbers marked this pull request as ready for review April 28, 2026 14:17
@svcAPLBot
Copy link
Copy Markdown
Contributor

Comparison of Helm chart templating output:

# raw/templates/resources.yaml

@@ spec.refreshInterval @@
# external-secrets.io/v1beta1/ExternalSecret/apl-gitea-operator/apl-gitea-operator-secret
! ± value change
- 1h
+ 1m

# apl-operator/templates/deployment.yaml

@@ spec.template.spec.containers.apl-operator.image @@
! ± value change
- linode/apl-core:main
+ docker.io/linode/apl-core:main

# raw/templates/resources.yaml

@@ spec.refreshInterval @@
! ± value change
- 1h
+ 1m

# raw/templates/resources.yaml

@@ spec.refreshInterval @@
# external-secrets.io/v1beta1/ExternalSecret/argocd/argocd-repo-creds-gitea
! ± value change
- 1h
+ 1m

@@ spec.refreshInterval @@
# external-secrets.io/v1beta1/ExternalSecret/argocd/argocd-repo-creds-gitea-internal
! ± value change
- 1h
+ 1m

# New file added: git-server-git-server
# New file added: git-server-git-server-artifacts
# raw/templates/resources.yaml

@@ spec.refreshInterval @@
# external-secrets.io/v1beta1/ExternalSecret/gitea-admin-secret
! ± value change
- 1h
+ 1m

# otomi-api/templates/configmap.yaml

@@ data.VERSIONS @@
! ± value change in multiline text (one insert, one deletion)
  
- {"api":"main","aplCharts":"main","console":"main","consoleLogin":"main","core":"main","specVersion":64,"tasks":"main","tools":"main"}
+ {"api":"APL-1786","aplCharts":"main","console":"APL-1787","consoleLogin":"main","core":"main","specVersion":65,"tasks":"main","tools":"main"}

# otomi-api/templates/core-config.yaml

@@ data.core.yaml @@
! ± value change in multiline text (one insert, no deletions)
  adminApps:
  - deps:
    - prometheus
    ingress:
  
  [618 lines unchanged)]
  
    - app: harbor
      name: harbor
    - disableIstioInjection: true
      name: apl-harbor-operator
+   - name: git-server
    - name: gitea
    - disableIstioInjection: true
      name: apl-gitea-operator
    - disableIstioInjection: true
  
  [103 lines unchanged)]
  
      svc: tekton-dashboard
      type: public
    name: tekton
    ownHost: true

# otomi-api/templates/deployment.yaml

@@ spec.template.spec.containers.otomi-api.image @@
! ± value change
- docker.io/linode/apl-api:main
+ docker.io/linode/apl-api:APL-1786

# raw/templates/resources.yaml

@@ spec.refreshInterval @@
! ± value change
- 1h
+ 1m

# otomi-console/templates/deployment.yaml

@@ spec.template.spec.containers.otomi-console.image @@
! ± value change
- docker.io/linode/apl-console:main
+ docker.io/linode/apl-console:APL-1787

# rabbitmq-cluster-operator/templates/messaging-topology-operator/validating-webhook-configuration.yaml

# values-repo.yaml

@@ versions.specVersion @@
! ± value change
- 64
+ 65

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