redhat-cop / rhel-edge-automation-arch

RHEL for Edge Automation Deployment Architecture
Apache License 2.0
44 stars 33 forks source link

Adding Quay chart with subscription #165

Closed jordigilh closed 2 years ago

jordigilh commented 2 years ago

Adds the subscription to the quay chart and also defines a template for the app-manager to trigger the quay chart deployment.

@sabre1041 I have not been able to test this in my environment due to dependencies in the quay deployment. If you can test it in your environment, please give it a try and let me know how it goes.

This is the app manifest generated pointing to my branch:

apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: quay
  namespace: openshift-gitops
spec:
  destination:
    namespace: openshift-operators
    server: https://kubernetes.default.svc
  project: default
  source:
    path: charts/quay
    repoURL: https://github.com/jordigilh/rhel-edge-automation-arch.git
    targetRevision: quay_chart
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
    syncOptions:
      - CreateNamespace=true

/Jordi

sabre1041 commented 2 years ago

@jordigilh my testing failed. I do not see a Subscription resource being generated.

jordigilh commented 2 years ago

@jordigilh my testing failed. I do not see a Subscription resource being generated.

ok thanks for testing it.

sabre1041 commented 2 years ago

@jordigilh You are missing the annotations on the QuayRegistry resource. The changes that you made were applied to the old Kustomize resources and not the one in the Helm chart