On Thu, Oct 18, 2012 at 1:01 PM,  <charles_r...@dell.com> wrote:
> On 10/18/2012 01:02 AM, Zdenek Styblik wrote:
>> On Mon, Oct 8, 2012 at 8:19 AM,  <charles_r...@dell.com> wrote:
>> [...]
>>>> A startup script that can do these:
>>>>
>>>> 1. Set OS Name, Version, System Hostname in BMC
>>>> 2. Get BMC URL/IP address from BMC
>>>>
>>>> Would benefit:
>>>> 1. users who would like to fetch hostname/OS Name/Version.
>>>> 2. launch web-interface to BMC where supported with "xdg-open $BMC_URL"
>>>> 3. Use the BMC_IP for other purposes like
>>>>     3a. snmp proxy from host to BMC when BMC agent present.
>>>>     3b. expose BMC IP/URL via other systems management interfaces (like 
>>>> wsman).
>>>>
>>>> ----------------------------------------------------------------------
>>>>
>>> Hello,
>>>
>>> I request feedback on this proposal that will help users who would like 
>>> certain pieces of OS/BMC
>>> information set during system start-up.
>>>
>>> Here is a start-up script that implements this proposal:
>>>     
>>> http://sourceforge.net/tracker/?func=detail&aid=3571446&group_id=95200&atid=610552
>>>
>>
>> Hi Charles,
>>
>> at first, I wanted to say I see no problem with committing this
>> script. And I still don't in general. However ...
>>
>> Is this script useful to somebody else than Dell customers running
>> RHEL-based distribution?
>
> Zdenek,
> There are four pieces of information that are exchanged here:
>  1. OS Name - this is a standard IPMI command
>  2. OS Hostname - this is a standard IPMI command
>  3. OS Version - this is a delloem command. The script has provision to 
> include other OEMs if they
> support this command.
>  5. BMC IP - this is a standard IPMI command
>  6. BMC URL - this is a delloem command. Although, on non-dell systems, this 
> would simply be
> "https://$BMC_IP:443";
>
> So, with the exception of #3, all of them should work on any system that 
> supports the respective
> commands.
>

Ok, I just wanted to know, making sure, of broader audience :)

> While I developed these for RHEL, I would like to have this working on 
> popular Linux distros. I will
> include a debian and suse version of the script soon (one this one is 
> accepted). There is a systemd
> unit coming up as well.
>

Great.

>>
[...]
>
> Thanks for reviewing this and for the feedback. Here is an updated patch that 
> incorporates your
> feedback and few other clean ups as well.
>

You're welcome. Can you please attach updated version to ticket at
SF.net or send it to mailing list as an attachment? I kind of failed
to get it from e-mail itself. Despite of that new version looks
alright.

Please note this ``printf "BMC_IPv4=%s\nBMC_URL=%s" '' should have
been, or should be, ``printf "BMC_IPv4=%s\nBMC_URL=%s\n" ''. It is
missing an '\n' at the very end. It isn't a big deal if read by
computer. But much nicer if read by a human later. I'm to blame.
Sorry.

+        KERNEL_VERSION=3D$(uname -r -m) <<< this line has spaces
instead of tabs(\t). I guess it is the last one in the script.

Thanks and have an easy Monday,
Z.

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_sfd2d_oct
_______________________________________________
Ipmitool-devel mailing list
Ipmitool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel

Reply via email to