Skip to content

refactor(code): Exported rescan functionality#455

Merged
akhilerm merged 1 commit intoopenebs-archive:masterfrom
RealHarshThakur:rescan
Jul 11, 2020
Merged

refactor(code): Exported rescan functionality#455
akhilerm merged 1 commit intoopenebs-archive:masterfrom
RealHarshThakur:rescan

Conversation

@RealHarshThakur
Copy link
Copy Markdown
Contributor

@RealHarshThakur RealHarshThakur commented Jul 10, 2020

Signed-off-by: Harsh Thakur harshthakur9030@gmail.com

Pull Request template

Why is this PR required? What issue does it fix?:
This PR exposes the method which is used to sync etcd and ndm local state

What this PR does?:
Exposes rescan method which helps to sync etcd and ndm local state. It also ensures that the udev method being called is thread safe.

Does this PR require any upgrade changes?:
No

If the changes in this PR are manually verified, list down the scenarios covered::
Tested it on a GKE cluster.

Any additional information for your reviewer? :
This is required for the gRPC PR.

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 10, 2020

Codecov Report

Merging #455 into master will decrease coverage by 0.03%.
The diff coverage is 7.14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
- Coverage   43.48%   43.44%   -0.04%     
==========================================
  Files          61       61              
  Lines        2969     2974       +5     
==========================================
+ Hits         1291     1292       +1     
- Misses       1576     1579       +3     
- Partials      102      103       +1     
Impacted Files Coverage Δ
cmd/ndm_daemonset/probe/eventhandler.go 12.76% <0.00%> (+0.39%) ⬆️
cmd/ndm_daemonset/probe/udevprobe.go 59.47% <9.09%> (-3.91%) ⬇️

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 017b755...f802f77. Read the comment docs.

Comment thread cmd/ndm_daemonset/probe/eventhandler.go Outdated
kmova
kmova previously approved these changes Jul 11, 2020
akhilerm
akhilerm previously approved these changes Jul 11, 2020
Copy link
Copy Markdown
Contributor

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

lgtm

@RealHarshThakur RealHarshThakur changed the title refactor(code): Rescan functionality refactor(code): Exported rescan functionality Jul 11, 2020
@RealHarshThakur RealHarshThakur dismissed stale reviews from akhilerm and kmova via ce1f0b5 July 11, 2020 11:46
Comment thread changelogs/unreleased/455-harshthakur9030 Outdated
Signed-off-by: Harsh Thakur <harshthakur9030@gmail.com>
Copy link
Copy Markdown
Contributor

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

/lgtm

@akhilerm akhilerm merged commit 97afa0b into openebs-archive:master Jul 11, 2020
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