X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C8ACA4.751B7570@onstor-exch02.onstor.net>; Fri, 2 May 2008 15:32:51 -0700
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: please review - change 29064
Date: Fri, 2 May 2008 15:32:51 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E09624AE7@onstor-exch02.onstor.net>
In-Reply-To: <20080502133857.5484743a@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: please review - change 29064
Thread-Index: AcislIteWAZJ9YffRHCbY4csGYHIhQAD6czw
From: "Rendell Fong" <rendell.fong@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Ok, I'll have Ron take a look at this.
Perhaps he's has more insight into SNMP code.


-----Original Message-----
From: Andy Sharp=20
Sent: Friday, May 02, 2008 1:39 PM
To: Rendell Fong
Subject: Re: please review - change 29064

nfx-tree/code/sm-bsd-snmpd/agent/mibgroup/agile/onsVolume.c

     there could be huge glaring bugs in this code, and i wouldn't know.
     but i looked through it and nothing jumped out, not that it would.
     perhaps Ron might be able to do a better job for you?  i noticed
     one or two things that seemed like improvements over the way it
     used to be done.  i think.



On Fri, 2 May 2008 10:38:49 -0700 "Rendell Fong"
<rendell.fong@onstor.com> wrote:

> Change: 29064
>=20
> Client: rendellf-test
>=20
> User:   rendellf
>=20
> Description:
>     TED23524: SNMP - MIB OID for volumes change every reboot
>=20
>     Changed SNMP agent to sort the list of volumes by volume Id.=20
>     This should ensure that the specific OID assigned to each volume
> remains consistent across reboots.
>=20
> Files:
> =20
>
//depot/dev/nfx-tree/code/sm-bsd-snmpd/agent/mibgroup/agile/onsVolume.c
> # edit
>=20
>=20
> Tested on Cougar and Bobcat using snmp walk of volume info.
