[Stgt-devel] [PATCH 2/7] Add option to delete all the targets

[FUJITA Tomonori]

On Tue, 29 Jul 2008 15:56:44 +0200
Tomasz Chmielewski <mangoo at wpkg.org> wrote:

> FUJITA Tomonori schrieb:
> 
> (...)
> 
> >> --delete <ALL|tid=|targetname>
> >>
> >> It would only touch the target if it is not in use.
> >> ALL would delete all targets which are not in use; the targets which have initiators connected would not have its state changed (some informational message would be displayed to the user).
> >>
> >> --forcedelete <ALL|tid=|targetname>
> >>
> >> It would delete the targets irrespectively of their use.
> > 
> > How about the followings?
> > 
> > (--force) --delete <ALL|tid=|targetname>
> > 
> > It would only touch the target if it is not in use. If an active
> > target (or targets due to ALL) is specified, the warnings shows up.
> > 
> > If --force option is enabled, it would delete the targets
> > irrespectively of their use.
> 
> I thought about it, but we have already:
> 
>   -f, --force                don't exit on tgtadm errors
> 
> And I couldn't come up with a better name than "force" ;)
> 
> Since tgt-admin still changes a lot, perhaps it would be better to rename the current "--force" into something like "--ignore-errors".

Sounds more sane. Any other comments?

If we can't change the command line now, we can't change the format too.