containerized-data-importer icon indicating copy to clipboard operation
containerized-data-importer copied to clipboard

forklift: introduce forklift controller

Open bennyz opened this issue 1 year ago • 38 comments

What this PR does / why we need it: A controller to run the forklift populators, it will handle the population for the OvirtVolumePopulator and OpenstackVolumePopulator kinds

Example DVs:

apiVersion: cdi.kubevirt.io/v1beta1
kind: DataVolume
metadata:
  name: my-populator-dv
  annotations:
spec:
  storage:
    dataSourceRef:
        apiGroup: forklift.konveyor.io
        kind: OvirtVolumePopulator
        name: test
    resources:
      requests:
        storage: 1G
    accessMode:
      - ReadWriteOnce
    storageClassName: nfs-csi
apiVersion: forklift.konveyor.io/v1beta1
kind: OvirtVolumePopulator
metadata:
  name: test
  namespace: default
spec:
  engineUrl: https://ovirt.fqdn
  engineSecretName: ovirt-secret
  diskId: 134db723-7ba2-47c4-b19a-975383ff1491

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer: depends on https://github.com/kubevirt/containerized-data-importer/pull/2947 Release note:

Introduce a controller to handle forklift's volume populators (ovirt, openstack)

TODO:

  • [ ] Make sure all annotations are properly set
  • [ ] Handle pod cleanup
  • [ ] Complete tests

bennyz avatar Nov 13 '23 15:11 bennyz