Skip to content

Commit

Permalink
Omit duplicate CRD for workflows
Browse files Browse the repository at this point in the history
When `argoproj/argo-events` is installed alongside `argoproj/argo` via Kustomize,
there is a collision between the workflow CRD defined by the argo project
and the one which is duplicated in this project. An example of this
issue in action would be a kustomization file which looks like this:

```
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
bases:
  - github.com/argoproj/argo//manifests/cluster-install
  - github.com/argoproj/argo-events//hack/k8s/manifests/
```

A `kustomize build` with that kustomization.yml results in the following error with kustomize v3.2.0:

```
Error: accumulating resources: recursed merging from path 'github.com/argoproj/argo-events//hack/k8s/manifests/': may not add resource with an already registered id: apiextensions.k8s.io_v1beta1_CustomResourceDefinition|~X|workflows.argoproj.io
```

This is due to both `argoproj/argo` and `argoproj/argo-events`
defining a CRD for `workflows.argoproj.io`. Removing the reference to this CRD
from the kustomization.yml in this project will allow both projects to be deployed
alongside each other more easily with kustomize. The workflow CRD
perhaps should be removed from this project entirely.

This commit  addresses the issue reported in #367
  • Loading branch information
donoftime authored and Don Mayo committed Oct 3, 2019
1 parent 9605b8b commit 7ce6c7d
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion hack/k8s/manifests/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,3 @@ resources:
- ./sensor-controller-configmap.yaml
- ./sensor-controller-deployment.yaml
- ./sensor-crd.yaml
- ./workflow-crd.yaml

0 comments on commit 7ce6c7d

Please sign in to comment.