
Jim Meyering wrote:
Cole Robinson wrote:
We aren't setting the persistent bit when a network is defined, so 'destroy' makes them disappear (though they will reappear later since their persistent config is never removed).
Attached patch fixes this.
Hi Cole,
Looks like you're on the right track. My reflex was to test for it... What before/after behavior change should I see? I tried this:
cat <<EOF > net.xml <network> <name>N</name> <ip address="192.168.199.1" netmask="255.255.255.0"></ip> </network> EOF qemud/libvirtd & src/virsh 'net-define net.xml; net-destroy N; net-list'
Then net-dumpxml N shows it's still there, as it should be. Of course, net-undefine does get rid of it for good, but you mentioned "destroy" above. Did I miss something?
To answer my own question, Yes ;-) we need net-lists' --all option. Before the patch, running this src/virsh 'net-define net.xml; net-destroy N; net-list --all' lists no network. After your patch, it lists N. So, ACK.