Opened 50 years ago

Last modified 9 years ago

#897 new task

IZ591: Missing -s option for show_service

Reported by: afisch Owned by:
Priority: low Milestone:
Component: hedeby Version: 1.0u1
Severity: Keywords: Sun cli
Cc:

Description

[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

Change History (0)

Note: See TracTickets for help on using tickets.