OCPVE-349: Add PV and PVC controllers to publish events

[suleymanakbas91]

This PR adds a PV and a PVC controller to the LVM Operator.

• The PV controller watches for PVs created by LVMS and publishes an event if their spec.claimRef gets deleted:

  

• The PVC controller watches for the PVCs created by LVMS and publishes an event if the PVC requests a storage greater than available capacity on any node:

  

[openshift-ci]

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: suleymanakbas91

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 [suleymanakbas91]

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

[openshift-ci-robot]

@suleymanakbas91: This pull request references OCPVE-349 which is a valid jira issue.

In response to this:

This PR adds a PV and a PVC controller to the LVM Operator.

• The PV controller watches for PVs created by LVMS and publishes an event if their spec.claimRef gets deleted:

• The PVC controller watches for the PVCs created by LVMS and publishes an event if the PVC requests a storage greater than capacities of all the nodes:

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.

[openshift-ci-robot]

@suleymanakbas91: This pull request references OCPVE-349 which is a valid jira issue.

In response to this:

This PR adds a PV and a PVC controller to the LVM Operator.

• The PV controller watches for PVs created by LVMS and publishes an event if their spec.claimRef gets deleted:

• The PVC controller watches for the PVCs created by LVMS and publishes an event if the PVC requests a storage greater than capacity on any node:

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.

[openshift-ci-robot]

@suleymanakbas91: This pull request references OCPVE-349 which is a valid jira issue.

In response to this:

This PR adds a PV and a PVC controller to the LVM Operator.

• The PV controller watches for PVs created by LVMS and publishes an event if their spec.claimRef gets deleted:

• The PVC controller watches for the PVCs created by LVMS and publishes an event if the PVC requests a storage greater than available capacity on any node:

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.

[suleymanakbas91]

/hold for more testing

[codecov-commenter]

Codecov Report

Merging #356 (3deee74) into main (00c61db) will not change coverage.
The diff coverage is 0.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #356   +/-   ##
=======================================
  Coverage   14.52%   14.52%           
=======================================
  Files          23       23           
  Lines        1921     1921           
=======================================
  Hits          279      279           
  Misses       1618     1618           
  Partials       24       24           

Impacted Files	Coverage Δ
controllers/lvmcluster_controller.go	0.00% <0.00%> (ø)
controllers/utils.go	40.00% <0.00%> (ø)
pkg/vgmanager/vgmanager_controller.go	0.00% <0.00%> (ø)

[openshift-ci]

@suleymanakbas91: all tests passed!

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.

[brandisher]

Couple of questions/nits but lgtm. Keeping the hold on so @jeff-roche can take a look as well.

/lgtm

[suleymanakbas91]

@jeff-roche was only waiting for the claimRef test before giving his approval, so I think it should be ok to unhold it now.

/unhold

[openshift-ci-robot]

@suleymanakbas91: This pull request references OCPVE-349 which is a valid jira issue.

In response to this:

This PR adds a PV and a PVC controller to the LVM Operator.

• The PV controller watches for PVs created by LVMS and publishes an event if their spec.claimRef gets deleted:

• The PVC controller watches for the PVCs created by LVMS and publishes an event if the PVC requests a storage greater than available capacity on any node:

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.