Michael Gregg wrote: > Jakub Hrozek wrote: > Hi, > > as discussed in https://bugzilla.redhat.com/show_bug.cgi?id=671019 to > delete a DNS RR one has to remove its record types one by one. > > This patch modifies the behaviour so that if the user runs dnsrecord-del > <zone> <record-name> with no other parameters, the whole record is > removed. > > Alternative solutions might be to expose the internal command that is > able to delete the record (although I think it is counterintuitive to > have one command to remove record types and one for the whole record) or > have a special flag (--del-all?) to remove the whole record. > > The patch also fixes the unit tests as they didn't reflect all the > recent changes.
> Going with this patch sounds good, but to make sure, I polled several people here, and they all seemed to think that having to add a --del-all or --del-record flag at the end would be better as it would be less prone to failure where admins would accidentally delete a entire record because they didn't specify anything after the "<zone> <record>" > So, maybe we do need a --del-all or --del-record operator. Agree. > Michael- > _______________________________________________ > Freeipa-devel mailing list > Freeipafirstname.lastname@example.org > https://www.redhat.com/mailman/listinfo/freeipa-devel -- Thank you, Dmitri Pal Sr. Engineering Manager IPA project, Red Hat Inc. ------------------------------- Looking to carve out IT costs? www.redhat.com/carveoutcosts/ _______________________________________________ Freeipa-devel mailing list Freeipaemail@example.com https://www.redhat.com/mailman/listinfo/freeipa-devel