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

Example Add the WWN information for netApp luns #267

Merged
merged 1 commit into from
Oct 11, 2021
Merged

Example Add the WWN information for netApp luns #267

merged 1 commit into from
Oct 11, 2021

Conversation

guankc
Copy link

@guankc guankc commented Oct 11, 2021

Example Add the WWN information for netApp luns

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

Release note:

@codecov
Copy link

codecov bot commented Oct 11, 2021

Codecov Report

Merging #267 (7c231ad) into netapp_lun (d4c3afd) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           netapp_lun     #267      +/-   ##
==============================================
- Coverage       70.74%   70.73%   -0.01%     
==============================================
  Files             161      161              
  Lines           15565    15564       -1     
  Branches         1934     1934              
==============================================
- Hits            11011    11009       -2     
  Misses           3906     3906              
- Partials          648      649       +1     
Impacted Files Coverage Δ
delfin/drivers/netapp/dataontap/netapp_handler.py 66.23% <ø> (ø)
delfin/drivers/fake_storage/__init__.py 94.85% <0.00%> (-0.29%) ⬇️
delfin/common/constants.py 100.00% <0.00%> (ø)
...ager/scheduler/schedulers/telemetry/job_handler.py 70.37% <0.00%> (ø)

@guankc guankc merged commit 83b6067 into gh-ca:netapp_lun Oct 11, 2021
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.

1 participant