>>@Alexandre: Does the assumption is correct, that you simply want to allow
>>only the one source ip, here 10.0.0.1?
If I remember, I'm using netX ip interface from ct.conf + ipfilter ips address
from ct.fw.
Can you send both files configuration ?
- Mail original -
De: "Christian Ebn
It seems that the ARP filtering with ebtables introduced a bug:
https://bugzilla.proxmox.com/show_bug.cgi?id=2193
After some digging, it turned out that the problem is that ebtables masks the
provided arp-ip-src address.
So while the provided rule looks like this:
-A veth100i0-OUT-ARP -p ARP --ar
in general: please reorder patch 1 and 2, as the API entry depends on
the QemuMigrateExternal to exist.
Am 4/29/19 um 12:01 PM schrieb Alexandre Derumier:
> Signed-off-by: Alexandre Derumier
> ---
> PVE/Makefile | 1 +
> PVE/QemuMigrateExternal.pm | 340
> +++
Am 4/30/19 um 4:07 PM schrieb Alexandre DERUMIER:
>> -my $conf = $self->{vmconf};
>> return if $self->{phase2errors};
>>
>> -my $tunnel = $self->{tunnel};
>> -
>
>>> above does probably not belongs in this patch, I'd guess.
>
> As it was the last sub in the phase, they were not us
> -my $conf = $self->{vmconf};
> return if $self->{phase2errors};
>
> -my $tunnel = $self->{tunnel};
> -
>>above does probably not belongs in this patch, I'd guess.
As it was the last sub in the phase, they were not used anymore.
but I can make an extra patch if you want.
-
Am 4/29/19 um 12:01 PM schrieb Alexandre Derumier:
> Signed-off-by: Alexandre Derumier
> ---
> PVE/QemuMigrate.pm | 36 +++-
> 1 file changed, 19 insertions(+), 17 deletions(-)
>
> diff --git a/PVE/QemuMigrate.pm b/PVE/QemuMigrate.pm
> index 4b521fd..6dd97e3 10064
Am 4/30/19 um 3:03 PM schrieb Dominik Csapak:
> do not poll the syslog api call multiple times simultaneously,
> since it can be very costly, depending on the journal size
>
> with this, there is always only one per client
>
> Signed-off-by: Dominik Csapak
> ---
> panel/LogView.js | 7 ++-
>
initially just wanted to review but to finaly move this forward: applied
comments still inline, can you please answer the question about the
second file info call? because I just threw a few followup on top, and
it would be great to verify that I did not missed anything ;-)
Am 4/30/19 um 2:20 PM s
maybe it is worth to introduce a volume_exists() helper?
> On 30 April 2019 at 14:20 Mira Limbeck wrote:
>
>
> use file_size_info to check for existence of cloudinit disk instead of
> '-e'. this should solve the problem with rbd where the path returned by
> PVE::Storage::path is not checkable w
do not poll the syslog api call multiple times simultaneously,
since it can be very costly, depending on the journal size
with this, there is always only one per client
Signed-off-by: Dominik Csapak
---
panel/LogView.js | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/p
use file_size_info to check for existence of cloudinit disk instead of
'-e'. this should solve the problem with rbd where the path returned by
PVE::Storage::path is not checkable with '-e'. Any size > 0 is
interpreted as the image existing.
Signed-off-by: Mira Limbeck
---
v2:
- switched to file_
Am 4/29/19 um 4:18 PM schrieb Alexandre Derumier:
> Signed-off-by: Alexandre Derumier
> ---
> src/PVE/Firewall.pm | 8 ++--
> 1 file changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/src/PVE/Firewall.pm b/src/PVE/Firewall.pm
> index 1607b74..806fd4b 100644
> --- a/src/PVE/Firewall.pm
12 matches
Mail list logo
