On (06/03/09 09:19), Peter Memishian wrote: > > > > jdc-11 How will the overlap between 'ipmpstat' and 'ipadm show-ipmp' > > > be handled? > > > > Today ipmpstat has the unfortunate task of displaying all the ipmp state, > > due to lack of better alternatives. After the delivery of the ipmp > component > > of ipadm, ipmpstat will only display statistics. ipadm show-ipmp will > > display ipmp state (such as interface membership, state etc.) > > I strongly disagree with this proposed change. First, it makes little > sense since ipmpstat does not show *any* statistics today. Second, the > customers and support engineers that I have spoken with *love* the > one-stop shopping nature of ipmpstat for IPMP status. Third, you would > have to add five distinct show subcommands to fold in all of the > information in a way that would provide equivalent functionality which > would be a mess. > > The proper level for ipadm show-ipmp should be the kind of stuff ifconfig > shows today: basic group membership and health (roughly similar in content > to the 20k foot view shown by ipmpstat -g). That is, it should allow an > administrator/support personnel to do basic sanity checks of the IPMP > configuration. They should then use ipmpstat to obtain more detailed > information. Having ipadm provide other IPMP-specific stuff like IPMP > probe and target information would just weigh down ipadm and lead to a > confusing schism for customers trying to get a handle on IPMP-specific > information. So, please leave ipmpstat be and have show-ipmp provide > high-level IPMP configuration inforation analogous to "ipmpstat -g".
I can go either way (since this PSARC 2009/306 is not about fixing ipmpstat, but rather about improving over ifconfig). However, having ipmpstat print something other than statistics makes it different from the other newer *stat commands like dlstat. I realize that ipmpstat covers a lot of territory, but we don't have an "aggrstat" command that prints out various views of the link-aggregation state. Why can't the ipmpstat functionality that displays IPMP state be moved into ipadm, leaving ipmpstat to do statistics-ish things like "-p"? --Sowmini
