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

Show virt-id in demo output if 'name' is empty. #84

Merged
merged 1 commit into from
Jul 29, 2020

Conversation

smoser
Copy link
Collaborator

@smoser smoser commented Jul 29, 2020

The 'name' field of a virtual drive may be empty. If that is the
case, show 'virtid-' in output of 'megaraid disk-summary'.

Before:

$ ./demo megaraid disk-summary
Path     | Name | Type | State |
/dev/sda |      | HDD  | Optl  |

After:

$ ./demo megaraid disk-summary
Path     | Name     | Type | State |
/dev/sda | virtid-0 | HDD  | Optl  |

@codecov
Copy link

codecov bot commented Jul 29, 2020

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   53.86%   53.86%           
=======================================
  Files          16       16           
  Lines        1888     1888           
=======================================
  Hits         1017     1017           
  Misses        772      772           
  Partials       99       99           

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 88e49bd...f967a01. Read the comment docs.

The 'name' field of a virtual drive may be empty.  If that is the
case, show 'virtid-<num>' in output of 'megaraid disk-summary'.

Before:

    $ ./demo megaraid disk-summary
    Path     | Name | Type | State |
    /dev/sda |      | HDD  | Optl  |

After:

    $ ./demo megaraid disk-summary
    Path     | Name     | Type | State |
    /dev/sda | virtid-0 | HDD  | Optl  |
@smoser smoser force-pushed the fix/megaraid-virt-empty-name branch from 1ed7444 to f967a01 Compare July 29, 2020 18:43
@tych0 tych0 merged commit d8696e2 into project-machine:master Jul 29, 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.

2 participants