IZ591: Missing -s option for show_service

[afisch]

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

        Issue #:      591          Platform:     Sun         Reporter: afisch (afisch)
       Component:     hedeby          OS:        All
     Subcomponent:    cli          Version:      1.0u1          CC:    None defined
        Status:       NEW          Priority:     P4
      Resolution:                 Issue type:    TASK
                               Target milestone: 1.0u5next
      Assigned to:    adoerr (adoerr)
      QA Contact:     adoerr
          URL:
       * Summary:     Missing -s option for show_service
   Status whiteboard:
      Attachments:


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


   Opened: Mon Nov 3 10:23:00 -0700 2008 
------------------------


   Description:
   The command show_service does not allow to specify a service to be listed.

     % sdmadm -s sdmaf ss -s spare_pool
     Error: Unrecognized option: -s
     Usage: sdmadm [global_options] show_service|ss [-dupval] [-noheader] [-sort
     columns] [-coldel char] [-headerdel char] [-hlp]

   This is inconsistent with other show commands like show_resource that allows to
   specify a resource with the -r option.

   Evaluation:
   The problem is rated as p4 task. It is not a bug, but the missing option is not
   reasonable.


   Suggested fix / Work around:
   Fix: The missing option should be introduced to show_service
   Work around: Not needed / not a bug.


   Analysis:
   An option annotation (@CliCommandDescriptor) for the -s option has to go into
   com.sun.grid.grm.cli.cmd.monitoring.ShowServicesCliCommand. And the
   corresponding filter has to be added.
   The documentation of the command (man page, -usage print, command reference) has
   to be updated, too.
   It should be kept in mind to treat the table model different in case that no
   service could be found (Issue590).


   How to test:
   The new option should be tested with a TS test, that uses the -s option to show
   a single service, eg. the systems spare_pool.

   ETC:
   3 PD {
     1 PD writing the new option
     1 PD test
     1 PD documentation
   }
               ------- Additional comments from rhierlmeier Wed Nov 25 07:21:10 -0700 2009 -------
   Milestone changed