search

redhat-cop / rhel-edge-automation-arch

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

adding machineset chart #166

Closed jordigilh closed 2 years ago

jordigilh commented 2 years ago

Adding manifests for machineset

@sabre1041 can you take a look? I've removed the annotation argocd.argoproj.io/sync-wave: "10" from the job, I don't think it's needed now, but let me know if you'd like to put it back as a helm hook weight.

sabre1041 commented 2 years ago

There are a number of enhancements that we can iterate upon. however, would it be possible to set namespace: {{ template "common.names.namespace" $ }} for all resources going into the rfe namespace?

jordigilh commented 2 years ago

There are a number of enhancements that we can iterate upon. however, would it be possible to set namespace: {{ template "common.names.namespace" $ }} for all resources going into the rfe namespace?

Done. Replaced all references of namespace: rfe to namespace: {{ template "common.names.namespace" $ }}