Skip to content
This repository was archived by the owner on Dec 12, 2024. It is now read-only.

Conversation

@humblec
Copy link
Contributor

@humblec humblec commented Jun 7, 2022

This PR does number of things

  • Introduce v1beta1 CRD for replication and class object
  • controller gen version update to latest
  • makefile changes required to build and use available targets
  • bolierplate changes to reflect the current year
  • all dependencies are updated to latest version
  • bundle version change to v0.0.2
  • linter adjustment for avoiding error on reconciler object in controller

Fix # #122

Signed-off-by: Humble Chirammal hchiramm@redhat.com

humblec added 2 commits June 7, 2022 16:12
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec humblec changed the title add go build tag and also rebase to latest dependency versions. Introduce v1beta1 CRD for replicaion objects. Jun 9, 2022
@humblec humblec changed the title Introduce v1beta1 CRD for replicaion objects. Introduce v1beta1 CRD for replication objects. Jun 9, 2022
@humblec humblec force-pushed the update branch 9 times, most recently from 6ee46b2 to aa3adac Compare June 9, 2022 10:50
This commit add v1beta1 CRD version for the replication objects
and also update the CRD manifests.

Addition to that, bundle version has been updated to v0.0.2

boilerplate header has been updated to reflect current year.

Fix: csi-addons#122

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
limitations under the License.
*/

package v1beta1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure we are ready for beta yet!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Madhu-1 any thoughts from your end ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes as discussed we are planning to move volRep to csiaddons, we aren't ready to move to beta Yet.

This avoid below error:
  Error: hugeParam: r is heavy (80 bytes); consider passing it by pointer (gocritic)

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@mergify
Copy link

mergify bot commented Jun 23, 2022

This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants