IZ657: Modify component command allows to configure components with wrong configuration type

[afisch]

[Imported from gridengine issuezilla ​http://gridengine.sunsource.net/issues/show_bug.cgi?id=657]

        Issue #:      657          Platform:     Sun            Reporter: afisch (afisch)
       Component:     hedeby          OS:        All
     Subcomponent:    cli          Version:      1.0u3_Beta-1      CC:    None defined
        Status:       NEW          Priority:     P4
      Resolution:                 Issue type:    DEFECT
                               Target milestone: 1.0u5next
      Assigned to:    adoerr (adoerr)
      QA Contact:     adoerr
          URL:
       * Summary:     Modify component command allows to configure components with wrong configuration type
   Status whiteboard:
      Attachments:


     Issue 657 blocks:
   Votes for issue 657:     Vote for this issue


   Opened: Wed May 13 05:54:00 -0700 2009 
------------------------


   Description:
   The mod_component command allows to configure a component with any valid
   component config. A spare_pool can be reconfigured with a valid configuration
   for a geadapter.

   Evaluation:
   The issue is considered as a P4 bug because it describes a problem that is
   caused by using the mod_component command in an unintended way.

   Fix/Workaround:
   Fix: The modify component command should fail if a configuration of a wrong type
   is provided.
   Workaround: Reconfigure the component with a valid configuration.

   Analysis:
   The execute method of the class ModifyComponentConfigCliCommand has to be
   modified. We should use the GetConfigurationCommand to determine the components
   configurations class and compare it with the type of the configuration that is
   returned after editing or loading.


   How to test:
   A TS test should be written that tries to reconfigure the sparepool with a
   geadapter configuration and should check if it works.
   For the test the -f option can be used.

   ETC:
   1.5 PD
               ------- Additional comments from afisch Wed May 13 05:57:06 -0700 2009 -------
   rated as p4 bug
               ------- Additional comments from rhierlmeier Wed Nov 25 07:21:11 -0700 2009 -------
   Milestone changed