Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread dietmar
> >>Do you want to allocate IPs on VM creation time, or VM start time? > > I think at vm creation time, or nic hotplug/unplug. > the ipam api is called, return free ip address, and we write somewhere in vm > config the ip address. > (for nic hotplug/unplug, we need to handle ipam removal on confi

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread Alexandre DERUMIER
>>Do you want to allocate IPs on VM creation time, or VM start time? I think at vm creation time, or nic hotplug/unplug. the ipam api is called, return free ip address, and we write somewhere in vm config the ip address. (for nic hotplug/unplug, we need to handle ipam removal on config revert) L

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread dietmar
> >>You you also do not store the cidr there, and instead > >>store the some pool ID retured by IPAM? > > cidr should be the key/id of the subnet. Almost all ipam use the cidr as key. Ok, that makes sense now. Do you want to allocate IPs on VM creation time, or VM start time? __

Re: [pve-devel] [PATCH v2 common 1/2] JSONSchema: add format validator support and cleanup check_format

2020-06-23 Thread Fabian Grünbichler
LGTM, what do you think about the following follow-up: --8<- diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index b2ba9f7..d28143d 100644 --- a/src/PVE/JSONSchema.pm +++ b/src/PVE/JSONSchema.pm @@ -1878,9 +1878,12 @@ sub generate_typetext { sub print_property_string { my

[pve-devel] applied: [PATCH storage] Fix 2763: Revert "storage_migrate: check if target storage supports content type"

2020-06-23 Thread Fabian Grünbichler
On May 25, 2020 9:41 am, Fabian Ebner wrote: > This reverts commit 95015dbbf24b710011965805e689c03923fb830c. > > parse_volname always gives 'images' and not 'rootdir'. In most > cases the volume name alone does not contain the needed information, > e.g. vm-123-disk-0 can be both a VM volume or a c

Re: [pve-devel] [PATCH v2 manager] fix #2810: reset state properly when editing mount features of containers

2020-06-23 Thread Fabian Grünbichler
On June 22, 2020 3:55 pm, Oguz Bektas wrote: > initializing 'mounts' array in the panel scope causes edits on subsequent > containers to get the values (mount=nfs) from the previous container. fix > this by > initializing the 'mounts' array in 'onGetValues' and 'setValues' > separately. > > Signe

Re: [pve-devel] [PATCH qemu 2/2] Fix backup for not 64k-aligned storages

2020-06-23 Thread Roland Kammerer
On Mon, Jun 22, 2020 at 02:54:02PM +0200, Stefan Reiter wrote: > Zero out clusters after the end of the device, this makes restore handle > it correctly (even if it may try to write those zeros, it won't fail and > just ignore the out-of-bounds write to disk). > > For not even 4k-aligned disks, th

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread Alexandre DERUMIER
>>You you also do not store the cidr there, and instead >>store the some pool ID retured by IPAM? cidr should be the key/id of the subnet. Almost all ipam use the cidr as key. Alexandre Derumier Ingénieur système et stockage Manager Infrastructure Fixe : +33 3 59 82 20 10

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread dietmar
> No,no, I don't want to store ips in subnets.cfg, it's done in ipam. (or it > could be done in a localipam database if no ipam exist). You you also do not store the cidr there, and instead store the some pool ID retured by IPAM? ___ pve-devel mail

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread Alexandre DERUMIER
>>No. I mean you store all subnets and IPs in subnets.cfg, so why do we >>need/want IPAM plugins? oh ok sorry. No,no, I don't want to store ips in subnets.cfg, it's done in ipam. (or it could be done in a localipam database if no ipam exist). the "allocation-pools" option, was for: in this su

Re: [pve-devel] [PATCH manager] ui: fw: Close #2815: Add warning if fw is disabled

2020-06-23 Thread Dominik Csapak
Looks mosty ok, but i noticed two things: 1. We use this grid also on datacenter and node level, so we should adapt the message for those somehow 2. is inline On 6/23/20 11:31 AM, Dominic Jäger wrote: Currently people add firewall rules but forget to activate the firewall on guest level. Thi

Re: [pve-devel] [PATCH node_config 1/1] node_config: Allow leading underscore in ACME aliases

2020-06-23 Thread Fabian Grünbichler
LGTM, except for one minor nit. could you please send a CLA as described in our Developer Documentation to off...@proxmox.com ? https://pve.proxmox.com/wiki/Developer_Documentation#Software_License_and_Copyright Thanks in advance! On June 22, 2020 12:10 pm, Fabian Möller wrote: > --- > PVE/Nod

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread dietmar
> >>But you already mange subnets and allocation pools in subnets.cfg, so what > >>it the purpose > >>of those ipam plugins? I am unable to see the point? Keeping those > >>reservations in sync > >>looks hard to me. > > do you mean sync between dhcp && ipam ? No. I mean you store all subnets

[pve-devel] [PATCH manager] ui: fw: Close #2815: Add warning if fw is disabled

2020-06-23 Thread Dominic Jäger
Currently people add firewall rules but forget to activate the firewall on guest level. This commit adds a warning to the top bar of the firewall panel to make them aware of this if necessary. It seems a little cheaper but still sufficient to check only if some rule exists and not for every rule i

Re: [pve-devel] [PATCH pve-zsync 0/1] Allow pve-zsync jobs to share dest

2020-06-23 Thread Wolfgang Link
Hi Bruce, Once you've submitted the CLA, a member of the release team will take care of this patch. You will be notified when it is committed or when something is missing. Regards Wolfgang > On 06/22/2020 2:38 PM Bruce Wainer wrote: > > > Wolfgang, > Thanks for the confirmation. What is th

Re: [pve-devel] RFC: sdn: add ip management (IPAM -DHCP) ideas

2020-06-23 Thread Alexandre DERUMIER
>>But you already mange subnets and allocation pools in subnets.cfg, so what it >>the purpose >>of those ipam plugins? I am unable to see the point? Keeping those >>reservations in sync >>looks hard to me. do you mean sync between dhcp && ipam ? I'm more thinking about static dhcp reservation