Skip to content

feat(reconcile, controller): disable reconciliation on custom resources#307

Merged
kmova merged 3 commits intoopenebs-archive:masterfrom
akhilerm:disable-reconcilation
Dec 4, 2019
Merged

feat(reconcile, controller): disable reconciliation on custom resources#307
kmova merged 3 commits intoopenebs-archive:masterfrom
akhilerm:disable-reconcilation

Conversation

@akhilerm
Copy link
Copy Markdown
Contributor

@akhilerm akhilerm commented Sep 17, 2019

openebs.io/reconcile: "false" annotation can be set on BlockDevice, Disk and BlockDeviceClaim custom resources to stop reconciliation of those resources.

Disabling reconciliation is useful while handling upgrades. No updates will be done on the Disk, BDs, BDCs etc if the reconciliation is disabled. This means that both the NDM operator and the NDM daemon will not be updating the Disk and BlockDevice resource when the reconcile annotation is set.

Signed-off-by: Akhil Mohan akhil.mohan@mayadata.io

@akhilerm akhilerm force-pushed the disable-reconcilation branch from bd39415 to 59a0845 Compare September 18, 2019 12:23
@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 18, 2019

Codecov Report

Merging #307 into master will increase coverage by 0.29%.
The diff coverage is 41.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
+ Coverage   51.22%   51.51%   +0.29%     
==========================================
  Files          55       55              
  Lines        2612     2634      +22     
==========================================
+ Hits         1338     1357      +19     
+ Misses       1171     1170       -1     
- Partials      103      107       +4
Impacted Files Coverage Δ
cmd/ndm_daemonset/controller/controller.go 17.47% <ø> (ø) ⬆️
cmd/ndm_daemonset/controller/diskstore.go 88% <33.33%> (-2.84%) ⬇️
cmd/ndm_daemonset/controller/blockdevicestore.go 83.7% <33.33%> (-2.46%) ⬇️
pkg/controller/disk/disk_controller.go 25% <50%> (+5%) ⬆️
...g/controller/blockdevice/blockdevice_controller.go 25.97% <50%> (+1.31%) ⬆️
...er/blockdeviceclaim/blockdeviceclaim_controller.go 38.46% <50%> (+0.3%) ⬆️
cmd/ndm_daemonset/probe/udevprobe.go 65.64% <0%> (+3.81%) ⬆️
cmd/ndm_daemonset/probe/eventhandler.go 53.94% <0%> (+7.89%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ca3dd7...609f2a3. Read the comment docs.

@akhilerm akhilerm changed the title feat(reconcile, controller): disable reconcilation on custom resources feat(reconcile, controller): disable reconciliation on custom resources Sep 18, 2019
@kmova kmova changed the title feat(reconcile, controller): disable reconciliation on custom resources WIP - feat(reconcile, controller): disable reconciliation on custom resources Oct 1, 2019
@kmova kmova added this to the 0.4.5 milestone Nov 7, 2019
`openebs.io/reconcile: "false"` annotation can be set on BlockDevice,
Disk and BlockDeviceClaim custom resources to stop reconcilation of
those resources.

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
add functionality to disable reconcilation from daemon also.
When the annotation is specified, the daemon also will not
update the details into the disk and blockdevice resource.

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@akhilerm akhilerm force-pushed the disable-reconcilation branch from a18a2ed to 609f2a3 Compare December 3, 2019 07:38
@akhilerm akhilerm requested a review from kmova December 3, 2019 07:55
@akhilerm akhilerm changed the title WIP - feat(reconcile, controller): disable reconciliation on custom resources feat(reconcile, controller): disable reconciliation on custom resources Dec 4, 2019
@kmova kmova merged commit 2f0d907 into openebs-archive:master Dec 4, 2019
akhilerm added a commit to akhilerm/node-disk-manager that referenced this pull request Dec 4, 2019
…es (openebs-archive#307)

`openebs.io/reconcile: "false"` annotation can be set on BlockDevice,
Disk and BlockDeviceClaim custom resources to stop reconcilation of
those resources.

add functionality to disable reconcilation from daemon also.
When the annotation is specified, the daemon also will not
update the details into the disk and blockdevice resource.

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
kmova pushed a commit that referenced this pull request Dec 4, 2019
…es (#307)

`openebs.io/reconcile: "false"` annotation can be set on BlockDevice,
Disk and BlockDeviceClaim custom resources to stop reconcilation of
those resources.

add functionality to disable reconcilation from daemon also.
When the annotation is specified, the daemon also will not
update the details into the disk and blockdevice resource.

Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@akhilerm akhilerm deleted the disable-reconcilation branch March 28, 2020 06:17
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.

4 participants