UI: Fix OS Type displayed for a VM#11075
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11075 +/- ##
=========================================
Coverage 16.74% 16.74%
- Complexity 14061 14063 +2
=========================================
Files 5724 5724
Lines 507787 507785 -2
Branches 61733 61733
=========================================
+ Hits 85038 85039 +1
+ Misses 413266 413263 -3
Partials 9483 9483
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@Pearl1594 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
|
@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
|
shwstppr
left a comment
There was a problem hiding this comment.
LGTM - tested on QA server
|
@Pearl1594 can this be marked ready for review/merge? |
|
@sureshanaparti a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. |
|
UI build: ✔️ |
|
|
Verified with QA simulator env. |
* UI: Fix OS Type displayed * remove unnecessary method for os name change --------- Co-authored-by: Suresh Kumar Anaparti <sureshkumar.anaparti@gmail.com>
Description
This PR fixes the OS type displayed for a VM
Before fix:
After fix:
Types of changes
Feature/Enhancement Scale or Bug Severity
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?