Skip to content
This repository has been archived by the owner on Apr 25, 2022. It is now read-only.

CephBlockPool CR #16

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

CephBlockPool CR #16

wants to merge 3 commits into from

Conversation

hbinkim
Copy link
Contributor

@hbinkim hbinkim commented May 10, 2021

What's done

  • code generation
  • define cr

Todo in this PR

  • implement cr logic

Next PR

  • mirroring
    • mirror snapshot
    • MirroringStatus
  • stretched cluster
    • SubFailureDomain
    • ReplicasPerFailureDomain

@github-actions github-actions bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 10, 2021
@hbinkim hbinkim added the enhancement New feature or request label May 10, 2021
- FYI, kubebuilder generates the default path to import api package following the repo name of the PROJECT file. When you create an API for a new CR, look closely on the import module name under the controllers folder when you hit "package blah is not in GOROOT" error message.
- Kubebuilder generates the default path to import api package in controller go files following the repo name of the PROJECT file. Therefore renaming domain and repo is needed in order to reduce manual labor to fix import statement after generating controller codes for a new CR.
@github-actions github-actions bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels May 24, 2021
@hbinkim hbinkim force-pushed the blockpool branch 3 times, most recently from 01bd2af to 74e9aec Compare May 24, 2021 08:44
@github-actions github-actions bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels May 24, 2021
- update existing CRD CephCluster version v1beta1 to v1
- apply default CRD version to v1
@@ -1,7 +1,10 @@
domain: tmax.io
repo: hypersds-operator
domain: github.com
Copy link
Collaborator

@hyoung-90 hyoung-90 May 25, 2021

Choose a reason for hiding this comment

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

It seems to be necessary to check if there is no association with the apiVersion of cr. Specifically, it means group of crd.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants