From 93ac2feab02ea9a43e251e8d8c3d3189368fb60c Mon Sep 17 00:00:00 2001
From: Apollon Oikonomopoulos <apollon@noc.grnet.gr>
Date: Tue, 18 Jan 2011 12:31:58 +0200
Subject: [PATCH] Fix disk count check in LUSetInstanceParams

LUSetInstanceParams checked instance.nics (and not instance.disks)
against constants.MAX_DISKS.

Signed-off-by: Apollon Oikonomopoulos <apollon@noc.grnet.gr>
Reviewed-by: Michael Hanselmann <hansmi@google.com>
---
 lib/cmdlib.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/cmdlib.py b/lib/cmdlib.py
index 8bd6d649c..f7bfc4536 100644
--- a/lib/cmdlib.py
+++ b/lib/cmdlib.py
@@ -9253,7 +9253,7 @@ class LUInstanceSetParams(LogicalUnit):
         _CheckInstanceDown(self, instance, "cannot remove disks")
 
       if (disk_op == constants.DDM_ADD and
-          len(instance.nics) >= constants.MAX_DISKS):
+          len(instance.disks) >= constants.MAX_DISKS):
         raise errors.OpPrereqError("Instance has too many disks (%d), cannot"
                                    " add more" % constants.MAX_DISKS,
                                    errors.ECODE_STATE)
-- 
GitLab