[pve-devel] applied-series: [PATCH firewall/cluster 0/9] add locking to firewall config changes

[Thomas Lamprecht]

On 4/29/20 10:52 AM, Fabian Grünbichler wrote:
> the second cluster patch is optional, but improves usability of
> non-worker API calls that do
> 
> cfs_lock_foo(..., sub {
> 
>   raise_foo
> 
> });
> 
> the last three firewall patches are unrelated bug fixes that I found
> while testing.
> 
> pve-cluster:
> 
> Fabian Grünbichler (2):
>   cfs_lock: add firewall lock helper
>   cfs_lock: re-raise exceptions
> 
>  data/PVE/Cluster.pm | 16 +++++++++++++++-
>  1 file changed, 15 insertions(+), 1 deletion(-)
> 
> pve-firewall:
> 
> Fabian Grünbichler (7):
>   configs: add locking helpers
>   api: add locking helpers
>   api: lock configs
>   clone_vmfw_conf: lock new config
>   api/ipsets: parse_cidr before checking for duplicates
>   configs: warn about duplicate ipset entries
>   rules: verify referenced security group exists
> 

applied series, thanks!

>  src/PVE/API2/Firewall/Aliases.pm | 104 ++++++++++------
>  src/PVE/API2/Firewall/Cluster.pm |  36 +++---
>  src/PVE/API2/Firewall/Groups.pm  |  52 ++++----
>  src/PVE/API2/Firewall/Host.pm    |  42 ++++---
>  src/PVE/API2/Firewall/IPSet.pm   | 204 +++++++++++++++++++++----------
>  src/PVE/API2/Firewall/Rules.pm   | 130 ++++++++++++++------
>  src/PVE/API2/Firewall/VM.pm      |  43 +++----
>  src/PVE/Firewall.pm              |  65 ++++++++--
>  8 files changed, 449 insertions(+), 227 deletions(-)
>