Commit 0211fd6e04cdc402da20818df54299c6ded3d3cb introduced regression where newly defined networks were not made persistent. This patch makes the network persistent on each successful definition. --- src/network/bridge_driver.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c index 660c38d..975b7f6 100644 --- a/src/network/bridge_driver.c +++ b/src/network/bridge_driver.c @@ -3124,6 +3124,9 @@ static virNetworkPtr networkDefine(virConnectPtr conn, const char *xml) { goto cleanup; } + /* define makes the network persistent - always */ + network->persistent = 1; + /* def was asigned */ freeDef = false; -- 1.8.1

If addition of rules in networkAddIptablesRules() failed the real error was masked by error reported when trying to clean up the remaining rules. With this patch the original error message is saved and set back after the removal is complete. --- src/network/bridge_driver.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c index 975b7f6..f1be954 100644 --- a/src/network/bridge_driver.c +++ b/src/network/bridge_driver.c @@ -2088,6 +2088,7 @@ networkAddIptablesRules(struct network_driver *driver, { int ii; virNetworkIpDefPtr ipdef; + virErrorPtr orig_error; /* Add "once per network" rules */ if (networkAddGeneralIptablesRules(driver, network) < 0) @@ -2104,6 +2105,9 @@ networkAddIptablesRules(struct network_driver *driver, return 0; err: + /* store the previous error message before attempting removal of rules */ + orig_error = virSaveLastError(); + /* The final failed call to networkAddIpSpecificIptablesRules will * have removed any rules it created, but we need to remove those * added for previous IP addresses. @@ -2113,6 +2117,9 @@ err: networkRemoveIpSpecificIptablesRules(driver, network, ipdef); } networkRemoveGeneralIptablesRules(driver, network); + + /* return the original error */ + virSetError(orig_error); return -1; } -- 1.8.1