operator-framework / operator-courier

Build, verify and push operators
Apache License 2.0
41 stars 53 forks source link

Include a randomized container folder in the tarred bundle #153

Closed ecordell closed 5 years ago

ecordell commented 5 years ago

This will help prevent collisions when untarring into the same directory.

Before:

├── 0.6
│   ├── etcdcluster.crd.yaml
│   └── etcdoperator.clusterserviceversion.yaml
├── 0.8
│   ├── etcdbackup.crd.yaml
│   ├── etcdcluster.crd.yaml
│   ├── etcdoperator.v0.9.0.clusterserviceversion.yaml
│   └── etcdrestore.crd.yaml
├── 0.9
│   ├── etcdbackup.crd.yaml
│   ├── etcdcluster.crd.yaml
│   ├── etcdoperator.v0.9.2.clusterserviceversion.yaml
│   └── etcdrestore.crd.yaml
└── etcd.package.yaml

After (example with two operators with overlapping versions)

tree downloaded                                                                                           1 ↵
downloaded
├── etcd-wmrnyuth
│   ├── 0.6
│   │   ├── etcdcluster.crd.yaml
│   │   └── etcdoperator.clusterserviceversion.yaml
│   ├── 0.8
│   │   ├── etcdbackup.crd.yaml
│   │   ├── etcdcluster.crd.yaml
│   │   ├── etcdoperator.v0.9.0.clusterserviceversion.yaml
│   │   └── etcdrestore.crd.yaml
│   ├── 0.9
│   │   ├── etcdbackup.crd.yaml
│   │   ├── etcdcluster.crd.yaml
│   │   ├── etcdoperator.v0.9.2.clusterserviceversion.yaml
│   │   └── etcdrestore.crd.yaml
│   └── etcd.package.yaml
└── notetcd-1rrw4cow
    ├── 0.6
    │   ├── etcdcluster.crd.yaml
    │   └── etcdoperator.clusterserviceversion.yaml
    ├── 0.8
    │   ├── etcdbackup.crd.yaml
    │   ├── etcdcluster.crd.yaml
    │   ├── etcdoperator.v0.9.0.clusterserviceversion.yaml
    │   └── etcdrestore.crd.yaml
    ├── 0.9
    │   ├── etcdbackup.crd.yaml
    │   ├── etcdoperator.v0.9.2.clusterserviceversion.yaml
    │   ├── etcdrestore.crd.yaml
    │   └── notcluster.crd.yaml
    └── notetcd.package.yaml

Tested with appregistry-server release-4.1 and master.