Re: [PATCH master] Add master_netmask to Cluster.UpgradeConfig

2011-11-04 Thread Andrea Spadaccini
Hi Guido, >> And also suppress pylint R0902 error about an object instance having >> more than 20 attributes. > > LGTM > > Perhaps we avoid the 20 attr and store them in a subobject (which we > already have) about the master info? The Cluster object has already more than 30 attributes, pylint jus

Re: [PATCH master] Add master_netmask to Cluster.UpgradeConfig

2011-11-04 Thread Guido Trotter
On Fri, Nov 4, 2011 at 3:29 PM, Andrea Spadaccini wrote: > And also suppress pylint R0902 error about an object instance having > more than 20 attributes. > LGTM Perhaps we avoid the 20 attr and store them in a subobject (which we already have) about the master info? Thanks, Guido

[PATCH master] Add master_netmask to Cluster.UpgradeConfig

2011-11-04 Thread Andrea Spadaccini
And also suppress pylint R0902 error about an object instance having more than 20 attributes. Signed-off-by: Andrea Spadaccini --- lib/objects.py |9 - 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/lib/objects.py b/lib/objects.py index 2467968..0c56620 100644 --- a/l