#17 dsremove
Closed: Fixed 6 years ago Opened 7 years ago by firstyear.

Make the inverse of dscreate: dsremove. It should act as remove-ds.pl, and work on instances that are installed by either.

To achieve this we need:

  • ds_paths must be able to work with an offline dse.ldif to get the correct paths
  • a working instance backup (we should backup on remove)
  • a working instance restore (to prove the backup works)
  • move the paths to a location, then delete from there (or backup then delete) .

Metadata Update from @firstyear:
- Issue assigned to firstyear

7 years ago

Metadata Update from @firstyear:
- Custom field Origin adjusted to None
- Custom field Review Status adjusted to review

6 years ago

@firstyear I like the idea I thought we had a wrapper dsctl (old dsadm) that allowed to create/delete instance. Am I missing something ?

dsctl does the remove :)

dscreate has to be seperate due to a limitation in the way python command line argument processing works, so dsctl can't be used for create.

@tbordaz @vashirov @mreynolds Would you mind taking a look at this? I need it for python 3 test cases :)

Question, you say "This is not what you want! Press ctrl-c now ... "

So it either exits, or it says to press ctrl-C. A bit confusing. Is the wording wrong? Should it be:

"If this is not what you want, then press ctrl-c now ... "?

Probably :) I'll update the wording for you. TBH that warning is only there because it's still "pre-release" and less polished, but we'll have to remove the dragon soon when we are happier with it,

Metadata Update from @mreynolds:
- Custom field Review Status adjusted to ack (was: review)

6 years ago

commit 77df9a54a99e3ae54a775de0e4d1b57193a07670
To ssh://git@pagure.io/389-ds-base.git
11974a0..77df9a5 master -> master

Metadata Update from @firstyear:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 years ago

Login to comment on this ticket.

Metadata