Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce v1beta1 CRD for replication objects. #132

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

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 [email protected]

@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 <[email protected]>
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 <[email protected]>
@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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants