Project

General

Profile

Bug #16455

Add lsiget tool

Added by Josh Paetzel over 2 years ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Nice to have
Assignee:
Warren Block
Category:
Middleware
Target version:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
Yes
Needs Merging:
Yes
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Associated revisions

Revision 12c9d5e8 (diff)
Added by Warren Block about 1 year ago

Create a new tools directory for additional support

Ticket: #16455

Revision 98ec9217 (diff)
Added by Warren Block about 1 year ago

Add the Broadcom lsiget script

Ticket: #16455

History

#1 Updated by Josh Paetzel over 2 years ago

  • Status changed from Unscreened to Screened

#2 Updated by Josh Paetzel about 2 years ago

  • Target version changed from 9.10.2 to 9.10.2-U1

#3 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 2 years ago

  • Target version changed from 9.10.2-U1 to 9.10.2-U2

#4 Updated by Josh Paetzel almost 2 years ago

  • Status changed from Screened to Unscreened
  • Assignee changed from Josh Paetzel to Kris Moore

#5 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 2 years ago

  • Assignee changed from Kris Moore to Warren Block

Warren,

You've been adding some things to the debug, care to take a crack at this one as well?

#6 Updated by Warren Block almost 2 years ago

  • Status changed from Unscreened to Screened

#7 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 2 years ago

  • Target version changed from 9.10.2-U2 to 9.10.3

Ping! Something you can still do?

#8 Updated by Warren Block almost 2 years ago

Yes, just need time. And answers to questions like "Where do we plan to import the utility from?"

#9 Updated by Warren Block almost 2 years ago

To answer my own question, from Broadcom. However, there is no license listed for this script. It includes a 15M compressed... let's say "sample", which is not needed but might be forced to be included by a license.

#10 Avatar?id=14398&size=24x24 Updated by Kris Moore almost 2 years ago

Let me know if you need me or somebody else to review that package, see if perhaps we can ignore that 15MB file ;)

#11 Updated by Warren Block almost 2 years ago

I know for a fact that 15M file is not required, because I tried it. The script seems to have been written by a raccoon foraging in a trash can.

#12 Updated by Dru Lavigne over 1 year ago

  • Target version changed from 9.10.3 to 9.10.3-U1

#13 Avatar?id=14398&size=24x24 Updated by Kris Moore over 1 year ago

  • Target version changed from 9.10.3-U1 to 11.0-U1

#14 Updated by Warren Block over 1 year ago

  • Status changed from Screened to Closed

Nobody has any information on Avago/Broadcom licensing, without which we cannot include this script. Closing.

#15 Updated by Vaibhav Chauhan over 1 year ago

  • Target version changed from 11.0-U1 to 11.0-U2

#16 Updated by Josh Paetzel over 1 year ago

Typically speaking LSI/Avago/Broadcom has been very willing to give FreeNAS permission to integrate and distribute their tools. They already did that for megacli and tw_cli. I see no reason why you couldn't just get the approval you need from them.

I guess it's easier just to close bugs than fix them. Flashbacks.

#17 Updated by Warren Block over 1 year ago

I checked with numerous departments over a period of months. This included a technical contact at Broadcom. Nobody had any information on existing or potential agreements with Avago or Broadcom, nor even who would be the right person or department to ask. If you have contacts at Broadcom, please share.

#18 Updated by Josh Paetzel over 1 year ago

Ask Brett who our account rep is. Phuoc Vu was our account rep for the 9years I was there. The NDAs and distribution agreements for the existing tools are on file as well. Valerie has them filed by vendor.

#19 Updated by Dru Lavigne over 1 year ago

  • Status changed from Closed to Investigation
  • Target version changed from 11.0-U2 to 11.1

#20 Updated by Dru Lavigne over 1 year ago

  • Status changed from Investigation to 46

Kris: is this a feature on the roadmap?

#21 Updated by Warren Block over 1 year ago

  • Status changed from 46 to 15

Resent request for license information to Cati.

#22 Updated by Warren Block over 1 year ago

  • Status changed from 15 to Investigation

#23 Avatar?id=14397&size=24x24 Updated by Brett Davis about 1 year ago

Broadcom/LSI/Avago has confirmed that no licensing is required to add these scripts. Please proceed.

#24 Updated by Warren Block about 1 year ago

  • Status changed from Investigation to Unscreened
  • Assignee changed from Warren Block to Bartosz Prokop

Transferring to Bartosz as discussed in today's conference call. The consensus was to add this tool to a new directory for such tools, rather than integrating it into freenas-debug.

#25 Updated by Bartosz Prokop about 1 year ago

  • Status changed from Unscreened to Screened

#26 Updated by Bartosz Prokop about 1 year ago

  • Assignee changed from Bartosz Prokop to Warren Block

Warren,

I've misunderstood the conversation. I thought it should be incorporated into freenas-debug(and I'm responsible for that part of the FN code).
Can you reassign this to someone who is skilled with our port/build infrastructure?

If the rest of the team is overloaded than please reassign this back to me.

Bartosz

#27 Updated by Warren Block about 1 year ago

  • Assignee changed from Warren Block to Kris Moore

#28 Avatar?id=14398&size=24x24 Updated by Kris Moore about 1 year ago

  • Assignee changed from Kris Moore to Warren Block

Warren, sending this back your way. Lets go ahead and create the tool dir, which should live here:

https://github.com/freenas/freenas/tree/master/src/freenas/usr/local

I'm thinking if you create a PR with a new "tools" dir, we can start filling it with sub-dirs like dtrace/ lsiget/ and so forth. Then doc where they are located for CLI users.

Make sense?

#29 Updated by Warren Block about 1 year ago

  • Subject changed from Add lsiget info to save-debug to Add lsiget tool
  • Status changed from Screened to Fix In Progress
  • Target version changed from 11.1 to 11.1-BETA1

#30 Updated by Warren Block about 1 year ago

  • Status changed from Fix In Progress to Needs Developer Review
  • Assignee changed from Warren Block to Kris Moore

#31 Updated by Warren Block about 1 year ago

  • Status changed from Needs Developer Review to Ready For Release
  • Assignee changed from Kris Moore to Warren Block

#32 Updated by Dru Lavigne about 1 year ago

  • Status changed from Ready For Release to Resolved

#33 Updated by Bonnie Follweiler about 1 year ago

The lsiget tool has been added

When I ran ./lsigetlunix.sh I got the following response:
OS_LSI is not set, either not a valid OS for this script or try running;
sudo bash ./lsigetlunix.sh -D -Q

I ran sudo bash ./lsigetlunix.sh -D -Q and got the following response:
Sorry, user root is not allowed to execute '/bin/bash ./lsigetlunix.sh -D -Q' as root on bonniemini.local.

So then I ran bash ./lsigetlunix.sh -D -Q and it worked

#34 Updated by Bonnie Follweiler about 1 year ago

  • Needs QA changed from Yes to No
  • QA Status Test Passes FreeNAS added
  • QA Status deleted (Not Tested)

Also available in: Atom PDF