test: added test to validate the LVMO resources

[riya-singhal31]

This test validates all the resources created by LVMO

Signed-off-by: riya-singhal31 [email protected]

[nbalacha]

Suggested change

	Context("Validate all the resources", func() {
	Context("Validate LVMCluster reconciliation", func() {

[nbalacha]

Suggested change

	daemonSetName = "topolvm-node"
	topolvmNodeDaemonSetName = "topolvm-node"

[nbalacha]

Suggested change

	deploymentName = "topolvm-controller"
	topolvmCtrlDeploymentName = "topolvm-controller"

[nbalacha]

You might need a larger timeout here. Maybe 30 s?

[riya-singhal31]

It's working fine for 10 s. Should I still change it to 30?

[riya-singhal31]

/retest

[nbalacha]

Change the order here to match the order in the LVMCluster reconcile function:

1. CSI Driver
2. Topolvm controller
3. vg manager
4. LVMVolumeGroup
5. topolvm node
6. Storageclass

	// Validate CSI Driver
	err := ValidateCSIDriver()
	if err != nil {
		return err
	}

	//Validate TopoLVM Controller
	err = ValidateTopolvmController()
	if err != nil {
		return err
	}

	// Validate VG Manager Daemonset
	err = ValidateVGManager()
	if err != nil {
		return err
	}
	// Validate LVMVg
	err = ValidateLVMvg()
	if err != nil {
		return err
	}

	// Validate Topolvm node
	err = ValidateTopolvmNode()
	if err != nil {
		return err
	}
	// Validate Storage class
	err = ValidateStorageClass()
	if err != nil {
		return err
	}

[nbalacha]

Suggested change

	func ValidateTopolvmNode() error {
	ds := appsv1.DaemonSet{}
	Eventually(func() bool {
	err := DeployManagerObj.GetCrClient().Get(context.TODO(), types.NamespacedName{Name: topolvmNodeDaemonSetName, Namespace: InstallNamespace}, &ds)
	if err != nil {
	debug("TopoLVM Node: %v", err)
	return false
	}
	return true
	}, timeout, interval).Should(BeTrue())
	debug("TopoLVM node found\n")
	func ValidateTopolvmNode() error {
	ds := appsv1.DaemonSet{}
	Eventually(func() bool {
	debug("%s\n", "Starting function - topolvmnode")
	err := DeployManagerObj.GetCrClient().Get(context.TODO(), types.NamespacedName{Name: topolvmNodeDaemonSetName, Namespace: InstallNamespace}, &ds)
	if err != nil {
	debug("topolvmNode : %s\n", err.Error())
	return false
	}
	return ds.Status.DesiredNumberScheduled == ds.Status.NumberReady
	}, timeout, interval).Should(BeTrue())

[nbalacha]

/test lvm-operator-bundle-e2e-aws

[riya-singhal31]

/retest

[riya-singhal31]

/retest

[riya-singhal31]

/retest

[nbalacha]

Suggested change

	debug("depoloyment found\n")
	debug("topolvm-controller deployment found\n")

[riya-singhal31]

Updated

[openshift-ci]

@riya-singhal31: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/lvm-operator-bundle-e2e-aws	571bac0	link	false	/test lvm-operator-bundle-e2e-aws

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nbalacha, riya-singhal31

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [nbalacha]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment