Merge charm-hw-health:add-supported-storage into charm-hw-health:master

Proposed by Facundo Ciccioli
Status: Merged
Approved by: James Troup
Approved revision: bac43dab0ae083a10dd23965fb8bc599d31df557
Merged at revision: e14ff3e5dafcf61cc5785c58eb1f2ff5f04b33b3
Proposed branch: charm-hw-health:add-supported-storage
Merge into: charm-hw-health:master
Diff against target: 12 lines (+1/-0)
1 file modified
src/lib/hwhealth/discovery/supported_vendors.py (+1/-0)
Reviewer Review Type Date Requested Status
🤖 prod-jenkaas-bootstack (community) continuous-integration Needs Fixing
BootStack Reviewers Pending
BootStack Reviewers Pending
Review via email: mp+415042@code.launchpad.net

Commit message

Install perccli for yet another Broadcom/LSI storage device

To post a comment you must log in.
Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

A CI job is currently in progress. A follow up comment will be added when it completes.

Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

FAILED: Continuous integration, rev:bac43dab0ae083a10dd23965fb8bc599d31df557

No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want jenkins to rebuild you need to trigger it yourself):
https://code.launchpad.net/~llama-charmers/charm-hw-health/+git/charm-hw-health/+merge/415042/+edit-commit-message

https://jenkins.canonical.com/bootstack/job/lp-charm-hw-health-ci/53/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/bootstack/job/lp-charm-test-unit/257/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/917/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-charm-hw-health-ci/53//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision e14ff3e5dafcf61cc5785c58eb1f2ff5f04b33b3

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/lib/hwhealth/discovery/supported_vendors.py b/src/lib/hwhealth/discovery/supported_vendors.py
2index 1412ee9..1e832ed 100644
3--- a/src/lib/hwhealth/discovery/supported_vendors.py
4+++ b/src/lib/hwhealth/discovery/supported_vendors.py
5@@ -9,6 +9,7 @@ SUPPORTED_STORAGE = {
6 "Broadcom / LSI": {
7 "SAS3008 PCI-Express Fusion-MPT SAS-3": tools.Sas3Ircu,
8 "MegaRAID Tri-Mode SAS3508": tools.PercCLI,
9+ "MegaRAID 12GSAS/PCIe Secure SAS39xx": tools.PercCLI,
10 },
11 # 'Mellanox Technologies': {
12 # 'MT27710 Family [ConnectX-4 Lx]': lambda: 'mlxconfig',

Subscribers

No one subscribed via source and target branches