So the snmp info will look like the following in the query parameters for
create/update
param[0].name=snmpcommunity&
param[0].value=public&
param[1].name=snmpport&
param[1].value=121
And in the list response the snmp info will look like the following
"params":{"snmpcommunity":"public","snmpport":161}
Is my understanding right?
Thanks,
Vijay V.
> -----Original Message-----
> From: Chiradeep Vittal
> Sent: Tuesday, August 21, 2012 2:45 AM
> To: Vijay Venkatachalam; CloudStack DeveloperList
> Cc: Pranav Saxena; David Nalley; Alena Prokharchyk; Deepak Garg
> Subject: Re: AutoScale comments.....(WAS:RE: where features are
> developed was: Review Request: Merge Kelven's VPC code for Vmware into
> asf vpc branch)
>
> You can look at the autoscale thread here:
> http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-
> dev/201207.mb
> ox/thread
>
>
>
> On 8/16/12 10:43 AM, "Vijay Venkatachalam"
> <vijay.venkatacha...@citrix.com> wrote:
>
> >
> >> -----Original Message-----
> >> From: Chiradeep Vittal
> >> Sent: Thursday, August 16, 2012 10:42 PM
> >> To: CloudStack DeveloperList
> >> Cc: Pranav Saxena; David Nalley; Vijay Venkatachalam; Alena
> >> Prokharchyk; Deepak Garg
> >> Subject: Re: AutoScale comments.....(WAS:RE: where features are
> >> developed was: Review Request: Merge Kelven's VPC code for Vmware
> >> into asf vpc branch)
> >>
> >> Can you address 3-6 ?
> >>
> >> >> 3. Any indexes at all on these new tables?
> >
> > Sure will introduce them as appropriate for each of the tables.
> > Will file a bug to track this as well.
> >
> >> >> 4. autoscale_vmprofiles (and java class) should not have reference
> >> >> to snmp. We discussed why before, and alternatives to this design
> >> >> on this mailing list.
> >
> > Sorry I could not bring up that email back, but it was something like
> > key-value pair.
> > Could you please give a brief explanation of what is the alternative?
> > Or if there is an example already in cloudstack can you point me to
> that?
> >
> >> >> 5. A lot of the changes to existing files are whitespace changes
> >> >> that are unrelated to the logic. Please avoid this.
> >
> > Sure will remove them and will configure my Eclipse IDE to avoid
> > this in future. Is there any other way?
> >
> >> >> 6. Why is a Counter not a ControlledEntity but Condition is?
> >
> >
> > Counter is at system level and is packaged by default/built-in or could
> > be added by root admin. But conditions are configured by tenants
> based
> > on their use-case. For ex. CPU > 80
> >
> >> >
> >> >> 8. The License header is not consistent (some with leading spaces,
> >> >> others
> >> >> without)
> >> >> 9. Not all files have the Apache license.
> >> >>
> > Will look into it and do the needful.
> >> >
> >> >Thanks,
> >> >Ram
> >> >
> >
