Hi,

for me, it is most important to finish up. This project is ongoing for
quite some time meanwhile and I think it is time to deliver. MIB
moduels can be extended after publication as Proposed Standard. I am
OK to accept these modifications if we at the same time close the door
for new features and instead wrap up this MIB module.

/js

On Mon, Sep 29, 2014 at 04:27:13PM +0900, Keiichi SHIMA wrote:
> Hello Ian,
> 
> Thank you for contributing the text.  I personally think the text is fine.  
> Let's wait for a while if we receive any comments about the text from other 
> authors and WG people.
> 
> I believe this modification doesn't affect the current consensus we have 
> about the draft.  If the text is accepted by the WG, then we'll integrate it 
> to the draft and would like to move forward.
> 
> Regards,
> ---
> Keiichi SHIMA (島 慶一)
> WIDE project <[email protected]>
> Research Laboratory, IIJ Innovation Institute, Inc <[email protected]>
> 
> 
> 
> On 2014/09/29, at 15:33, Ian West <[email protected]> wrote:
> 
> > Attached is a diff against the text form of the draft, I believe I have 
> > addressed the changes that
> > might be required.
> > 
> > Also for convenience the actual text of the proposed additional counters is 
> > as follows.
> > 
> > vmStorageReadOctets OBJECT-TYPE
> >       SYNTAX       Counter64
> >       MAX-ACCESS   read-only
> >       STATUS       current
> >       DESCRIPTION
> >               "The total number of bytes read from this device.
> >               Discontinuities in the value of this counter can occur
> >               at re-initialization of the hypervisor, and
> >               administrative state (vmAdminState) changes of the
> >               virtual machine."
> >       ::= { vmStorageEntry 15 }
> > 
> > vmStorageWriteOctets OBJECT-TYPE
> >       SYNTAX       Counter64
> >       MAX-ACCESS   read-only
> >       STATUS       current
> >       DESCRIPTION
> >               "The total number of bytes written to this device.
> >               Discontinuities in the value of this counter can occur
> >               at re-initialization of the hypervisor, and
> >               administrative state (vmAdminState) changes of the
> >               virtual machine."
> >       ::= { vmStorageEntry 16 }
> > 
> > vmStorageReadLatency OBJECT-TYPE
> >       SYNTAX       Counter64
> >       MAX-ACCESS   read-only
> >       STATUS       current
> >       DESCRIPTION
> >               "The total number of microseconds read requests have been
> >               queued for this device.
> >               This would typically be implemented by storing the high 
> > precision
> >               system time stamp of when the request is received from the 
> > Virtual
> >               Machine with the request, the difference between this initial 
> > time
> >               stamp and the time at which the requested operation has 
> > completed
> >               should be converted to microseconds and accumulated.
> >               Discontinuities in the value of this counter can occur
> >               at re-initialization of the hypervisor, and
> >               administrative state (vmAdminState) changes of the
> >               virtual machine."
> >       ::= { vmStorageEntry 17 }
> > 
> > vmStorageWriteLatency OBJECT-TYPE
> >       SYNTAX       Counter64
> >       MAX-ACCESS   read-only
> >       STATUS       current
> >       DESCRIPTION
> >               "The total number of microseconds write requests have been
> >               queued for this device.
> >               This would typically be implemented by storing the high 
> > precision
> >               system time stamp of when the request is received from the 
> > Virtual
> >               Machine with the request, the difference between this initial 
> > time
> >               stamp and the time at which the requested operation has 
> > completed
> >               should be converted to microseconds and accumulated.
> >               Discontinuities in the value of this counter can occur
> >               at re-initialization of the hypervisor, and
> >               administrative state (vmAdminState) changes of the
> >               virtual machine."
> >       ::= { vmStorageEntry 18 }
> > 
> > Thankyou again for your consideration and feedback.
> > 
> > Regards,
> > Ian West.
> > 
> > -----Original Message-----
> > From: Juergen Schoenwaelder [mailto:[email protected]] 
> > Sent: Monday, 22 September 2014 15:25
> > To: Keiichi SHIMA
> > Cc: Ian West; [email protected]; ASAI Hirochika; SEKIYA Yuji; Tina Tsou; 
> > Cathy Zhou; ESAKI Hiroshi; Michael MacFaden
> > Subject: Re: [OPSAWG] Comments from niw.com.au operator on draft on
> > 
> > Hi,
> > 
> > it would help if people would propose concrete text of what the
> > description of the counters would be. What is most important is that
> > we define counters that are implementable in a meaningful way on a
> > large set of different hypervisors.
> > 
> > /js
> > 
> > On Sun, Sep 21, 2014 at 09:36:55PM +0900, Keiichi SHIMA wrote:
> >> Hello Ian,
> >> 
> >>> Your description of the accumulated milliseconds (or potentially 
> >>> microseconds to allow for things getting a lot faster) is exactly
> >>> what I had in mind. The delta of this figure divided by the delta iops, 
> >>> should give an approximate measure of the load the io
> >>> subsystem is under. 
> >> 
> >> Thank you for the clarification.  I got better understanding of your 
> >> proposal.  I tend to agree that the above usage may be useful.
> >> 
> >> I again don’t have any strong support/objection for adding the above new 
> >> entry to the VMM-MIB.  I’d like to hear if other authors and people in 
> >> this WG support this.
> >> 
> >> Regards,
> >> ---
> >> Keiichi SHIMA (島 慶一)
> >> WIDE project <[email protected]>
> >> Research Laboratory, IIJ Innovation Institute, Inc <[email protected]>
> >> 
> >> 
> >> 
> >> 2014/09/21 11:27、Ian West <[email protected]> のメール:
> >> 
> >>> Hello,
> >>> 
> >>> Thankyou for your clarification and feedback.
> >>> 
> >>> Your description of the accumulated milliseconds (or potentially 
> >>> microseconds to allow for things getting a lot faster) is exactly
> >>> what I had in mind. The delta of this figure divided by the delta iops, 
> >>> should give an approximate measure of the load the io
> >>> subsystem is under. 
> >>> 
> >>> Regards,
> >>> Ian West.
> >>> 
> >>> -----Original Message-----
> >>> From: Keiichi SHIMA [mailto:[email protected]] 
> >>> Sent: Saturday, 20 September 2014 18:03
> >>> To: [email protected]
> >>> Cc: SHIMA Keiichi; ASAI Hirochika; Jürgen Schönwälder; SEKIYA Yuji; Tina 
> >>> Tsou; Cathy Zhou; ESAKI Hiroshi; Ian West; Michael MacFaden
> >>> Subject: Re: [OPSAWG] Comments from niw.com.au operator on draft on
> >>> 
> >>> Hello,
> >>> 
> >>>> It would be really nice to have IO service times as well, ie something 
> >>>> like accumulated milliseconds of wait time for io servicing,
> >>>> this might be a harder to instrument. A counter64 of cumulative ms (us?) 
> >>>> awaiting delivery is a really handy read/write latency metric.
> >>> 
> >>> Let me clarify the above sentence.  Is the service time, you are talking 
> >>> about, the time between an I/O request arrives to the storage subsystem 
> >>> (in a hypervisor) and the time its response is returned to the calling 
> >>> VM?  And you want to add an entry to keep an accumulated value of those 
> >>> I/O request delay?
> >>> 
> >>> 
> >>>> The vmStorageEntry statistics while having read operation and write 
> >>>> operation counters don't allow for measuring the volume of read and 
> >>>> write traffic. It would be useful to include 64 bit counters for read 
> >>>> and write octets.
> >>> 
> >>> I don’t have any particular opinion on this topic.  If people in this WG 
> >>> support adding the above entries, then maybe it is possible to add them.
> >>> 
> >>> Regards,
> >>> ---
> >>> Keiichi SHIMA (島 慶一)
> >>> Research Laboratory, IIJ Innovation Institute, Inc <[email protected]>
> >>> WIDE project <[email protected]>
> >>> 
> >>> 
> >>> 
> >>> 2014/09/18 2:45、Michael MacFaden <[email protected]> のメール:
> >>> 
> >>>> Hi authors,
> >>>> 
> >>>> Regarding: https://datatracker.ietf.org/doc/draft-ietf-opsawg-vmm-mib/
> >>>> 
> >>>> Ian as allowed me to forward you his comments on the draft.
> >>>> 
> >>>> Cheers,
> >>>> Mike MacFaden
> >>>> 
> >>>> 
> >>>> From: Ian West <[email protected]>
> >>>> Subject: RE: Mail regarding draft-ietf-opsawg-vmm-mib
> >>>> Date: September 16, 2014 at 3:18:08 AM PDT
> >>>> To: 'Michael MacFaden' <[email protected]>
> >>>> 
> >>>> 
> >>>> Overall I think it looks great.
> >>>> 
> >>>> It would be really nice to have IO service times as well, ie something 
> >>>> like accumulated milliseconds of wait time for io servicing,
> >>>> this might be a harder to instrument. A counter64 of cumulative ms (us?) 
> >>>> awaiting delivery is a really handy read/write latency metric.
> >>>> 
> >>>> The cputime and memory figures look good.
> >>>> 
> >>>> I am thinking there would be a storage object for the swap space which 
> >>>> would give most of the paging stats if actual byte counts were added ?
> >>>> 
> >>>> Links back to the ifindex should also tie into vswitch I think ? My 
> >>>> current walks of vmware are nesting vswitch ports nicely so that’s
> >>>> looking good. Network stats seem good already, its primarily the per 
> >>>> vhost cpu usage and io stats that are weak from a real time
> >>>> perspective, this MIB looks like it should cover it really well.
> >>>> 
> >>>> Is this intended to be a vsphere (central management) resource, or a per 
> >>>> physical host (or both, which would be great) ?
> >>>> 
> >>>> If you were to query this from a vsphere central management host, I am 
> >>>> thinking you might want to add to the vmtable an identifier
> >>>> for the physical host that the vm is currently running on. This would 
> >>>> make the table useful either standalone or  across a bunch of
> >>>> physical hosts.
> >>>> 
> >>>> Ill think on it some more, this seems like a great step forward over 
> >>>> whats there now already in VMware’s implementation.
> >>>> 
> >>>> 
> >>>> The vmStorageEntry statistics while having read operation and write 
> >>>> operation counters don't allow for measuring the volume of read and 
> >>>> write traffic. It would be useful to include 64 bit counters for read 
> >>>> and write octets.
> >>>> 
> >>>> 
> >>>> _______________________________________________
> >>>> OPSAWG mailing list
> >>>> [email protected]
> >>>> https://www.ietf.org/mailman/listinfo/opsawg
> >>> 
> >>> 
> >> 
> > 
> > -- 
> > Juergen Schoenwaelder           Jacobs University Bremen gGmbH
> > Phone: +49 421 200 3587         Campus Ring 1, 28759 Bremen, Germany
> > Fax:   +49 421 200 3103         <http://www.jacobs-university.de/>
> > 
> > <draft-ietf-opsawg-vmm-mib-01.diff>
> 

-- 
Juergen Schoenwaelder           Jacobs University Bremen gGmbH
Phone: +49 421 200 3587         Campus Ring 1, 28759 Bremen, Germany
Fax:   +49 421 200 3103         <http://www.jacobs-university.de/>

_______________________________________________
OPSAWG mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/opsawg

Reply via email to