Open mateusoliveira43 opened 2 days ago
To avoid duplication, we need to create OpenShift Templates for tests/e2e/sample-applications. This will also make it easier to create the applications for manual tests
tests/e2e/sample-applications
Today, developer must run two command to deploy an application, one to create application's PVCs (https://github.com/openshift/oadp-operator/blob/master/tests/e2e/sample-applications/mysql-persistent/pvc-twoVol/aws.yaml) and other for the other objects (https://github.com/openshift/oadp-operator/blob/master/tests/e2e/sample-applications/mysql-persistent/mysql-persistent-twovol-csi.yaml). And if developer want to use an different storageClassName, it need to manually edit the YAML files.
storageClassName
With OpenShift Template, it easy to create all object at once and easy modify its parameters. Example https://github.com/migtools/oadp-non-admin/blob/master/hack/samples/apps/mysql.yaml
Tried in https://github.com/openshift/oadp-operator/pull/1364, but approach would make developers running it harder (they would need to change replace strings manually).
To avoid duplication, we need to create OpenShift Templates for
tests/e2e/sample-applications
. This will also make it easier to create the applications for manual testsExample
Today, developer must run two command to deploy an application, one to create application's PVCs (https://github.com/openshift/oadp-operator/blob/master/tests/e2e/sample-applications/mysql-persistent/pvc-twoVol/aws.yaml) and other for the other objects (https://github.com/openshift/oadp-operator/blob/master/tests/e2e/sample-applications/mysql-persistent/mysql-persistent-twovol-csi.yaml). And if developer want to use an different
storageClassName
, it need to manually edit the YAML files.With OpenShift Template, it easy to create all object at once and easy modify its parameters. Example https://github.com/migtools/oadp-non-admin/blob/master/hack/samples/apps/mysql.yaml
Failed approach
Tried in https://github.com/openshift/oadp-operator/pull/1364, but approach would make developers running it harder (they would need to change replace strings manually).