All,
Updated webrev for this set of changes at:
http://cr.opensolaris.org/~johnfisc/list
These changes include feedback during a phone conversation from Ethan
as well as the installadm man page change to document the list
subcommand.
Thanks,
John
John Fischer wrote:
> Ethan, Clay and Sundar,
>
> When you get a chance can you review my webrev at:
>
> http://cr.opensolaris.org/~johnfisc/list
>
> These changes are for the installadm list subcommand fix and
> will address:
>
> 4330 - installadm list should show which server provide the install
> services.
> 4113 - nice to have an option to show what service the client is
> using
> 4298 - installadm list -n should give different output if there is
> no custom manifest
> 4597 - installadm list: expect usage statement when giving service
> name without "-n" flag.
> 4646 - list: showing added manifest for a non running service.
> 5300 - list: does not show informational message for non running
> service.
> 8496 - list: no verbiage indicating that a service does not exist
> when a non-existent service is given.
> 8529 - 'installadm list' command lists same service three times
> 6811 - list: should have similar output between list and list -n
> 8015 - list-manifests output is hard to read
> 9094 - installadm list: prints colons for empty MAC fields, and
> doesn't account for colons or periods in MAC field's width
> 4175 - install list error slips out
>
> The changed and added files are:
>
> usr/src/cmd/installadm/Makefile
> usr/src/pkgdefs/SUNWinstalladm-tools/prototype_com
> usr/src/cmd/installadm/installadm.c
> usr/src/cmd/installadm/installadm.h
> usr/src/cmd/installadm/list.py
>
> These changes also include a change to the Makefile and prototype_com
> file for the delete_service and delete_client to be consistent with
> the other python scripts within the /usr/lib/installadm directory.
>
> The remote service listing is being postponed due to a timing issue
> with multiple remote services that grows with the number of services
> within the domain.
>
> Thanks,
>
> John
> _______________________________________________
> caiman-discuss mailing list
> caiman-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
