[pve-devel] [PATCH common] Network: stop using ifconfig

2017-02-06 Thread Wolfgang Bumiller
We shouldn't mix different tool sets on the one hand, and on the other hand net-tools is an optional package in stretch and there's no real need for us to depend on it. --- src/PVE/Network.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/Network.pm b/src/PVE/Netwo

[pve-devel] pve-firewall / current git master

2017-02-06 Thread Stefan Priebe - Profihost AG
Hi, after upgrading my test cluster to latest git versions from 4.3. I've no working firewall rules anymore. All chains contain an ACCEPT rule. But i'm not sure whether this was also the case with 4.3. But it breaks the rules. The chains is this one: # iptables -L tap137i0-IN -vnx Chain tap137i0-

Re: [pve-devel] [PATCH common] Network: stop using ifconfig

2017-02-06 Thread Thomas Lamprecht
Am 06.02.2017 um 11:22 schrieb Wolfgang Bumiller: We shouldn't mix different tool sets on the one hand, and on the other hand net-tools is an optional package in stretch and there's no real need for us to depend on it. The pvereport commands should then probably be also changed from ifconfig to

Re: [pve-devel] [PATCH common] Network: stop using ifconfig

2017-02-06 Thread Wolfgang Bumiller
On Mon, Feb 06, 2017 at 11:29:34AM +0100, Thomas Lamprecht wrote: > Am 06.02.2017 um 11:22 schrieb Wolfgang Bumiller: > >We shouldn't mix different tool sets on the one hand, and on > >the other hand net-tools is an optional package in stretch > >and there's no real need for us to depend on it. >

[pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Wolfgang Bumiller
applied and added another missing hunk for qemu-server On Tue, Jan 31, 2017 at 11:15:22AM +0100, Wolfgang Bumiller wrote: > --- > in order to be able to bootstrap the package (eg. for stretch) > (copypasted as-is from Firewall.pm) > > src/PVE/FirewallSimulator.pm | 19 +-- > 1 fi

[pve-devel] [PATCH manager] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Emmanuel Kasper
iproute2 is a direct depency from pve-qemu-kvm and should be always available use -statistics so we get the RX/TX traffic sum like ifconfig --- PVE/Report.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Report.pm b/PVE/Report.pm index 901fcf8..d32b968 100644 --- a/PVE/R

Re: [pve-devel] pve-firewall / current git master

2017-02-06 Thread Wolfgang Bumiller
First a general note (for everyone on the list actually): Please don't let your mail clients line-break command outputs, it steals way too much of my time reading this :-\. (And please prefer iptables-save style output over iptables -L..., iptables -L is just horrible. I'm so looking forward to whe

Re: [pve-devel] [PATCH manager] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Wolfgang Bumiller
On Mon, Feb 06, 2017 at 02:37:16PM +0100, Emmanuel Kasper wrote: > iproute2 is a direct depency from pve-qemu-kvm and should > be always available > > use -statistics so we get the RX/TX traffic sum like ifconfig > --- > PVE/Report.pm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > d

Re: [pve-devel] [PATCH manager] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Emmanuel Kasper
On 02/06/2017 03:02 PM, Wolfgang Bumiller wrote: > On Mon, Feb 06, 2017 at 02:37:16PM +0100, Emmanuel Kasper wrote: >> iproute2 is a direct depency from pve-qemu-kvm and should >> be always available >> >> use -statistics so we get the RX/TX traffic sum like ifconfig >> --- >> PVE/Report.pm | 2 +-

[pve-devel] [PATCH manager v2] Use ip from iproute2 package instead of net-tools

2017-02-06 Thread Emmanuel Kasper
iproute2 is a direct depency from pve-qemu-kvm and should be always available use -statistics so we get the RX/TX traffic sum like ifconfig use -details to get extra info like current vlan_protocol --- changes since v1: * add -details as suggested by w.bumiller PVE/Report.pm | 2 +- 1 file chang

Re: [pve-devel] pve-firewall / current git master

2017-02-06 Thread Stefan Priebe - Profihost AG
Hi, sorry don't know how to teach thunderbird to not break lines. But i could sent the mail again using pastebin. Just request. Sorry again. Am 06.02.2017 um 14:59 schrieb Wolfgang Bumiller: > First a general note (for everyone on the list actually): > Please don't let your mail clients line-brea

[pve-devel] [manager] *** Minor Enhancement ***

2017-02-06 Thread Daniel Gallego Sánchez
From: Daniel1108 *** send nodename when updating CT/VM status *** Daniel1108 (1): A minor enhancement for external metrics to Influxdb and Graphite. So far with the data sent was impossible to know the node that host each lxc/qemu. This small modification allow this. It's very

[pve-devel] [manager] *** Minor Enhancement ***

2017-02-06 Thread Daniel Gallego Sánchez
From: Daniel1108 *** send nodename when updating CT/VM status *** Daniel1108 (1): A minor enhancement for external metrics to Influxdb and Graphite. So far with the data sent was impossible to know the node that host each lxc/qemu. This small modification allow this. It's very

[pve-devel] [manager] *** Minor Enhancement ***

2017-02-06 Thread Daniel Gallego Sánchez
From: Daniel1108 *** send nodename when updating CT/VM status *** Daniel1108 (1): A minor enhancement for external metrics to Influxdb and Graphite. So far with the data sent was impossible to know the node that host each lxc/qemu. This small modification allow this. It's very

[pve-devel] [manager] ***feature external metric***

2017-02-06 Thread Daniel Gallego Sánchez
From: Daniel1108 Correct style Regards Signed-off-by: Daniel1108 --- PVE/Service/pvestatd.pm | 4 ++-- PVE/Status/Graphite.pm | 6 +++--- PVE/Status/InfluxDB.pm | 8 PVE/Status/Plugin.pm| 4 ++-- 4 files changed, 11 insertions(+), 11 deletions(-) diff --git a/PVE/Service/pvest

Re: [pve-devel] pve-firewall / current git master

2017-02-06 Thread Wolfgang Bumiller
On Mon, Feb 06, 2017 at 03:59:40PM +0100, Stefan Priebe - Profihost AG wrote: > Hi, > > sorry don't know how to teach thunderbird to not break lines. But i > could sent the mail again using pastebin. Just request. Sorry again. Not sure how I feel about that. Pastebins have a tendency to be erased

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Dietmar Maurer
is there really no other way to solve this issue? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Stefan Priebe - Profihost AG
An alternative might be http://perldoc.perl.org/autouse.html Stefan Excuse my typo sent from my mobile phone. > Am 06.02.2017 um 18:20 schrieb Dietmar Maurer : > > is there really no other way to solve this issue? > > ___ > pve-devel mailing list > p

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Dietmar Maurer
> An alternative might be > http://perldoc.perl.org/autouse.html Thank for that interesting link. But It would prefer a way to avoid that cyclic package dependency at all. On the other way, the current solution is not that bad ;-) ___ pve-devel mailing

Re: [pve-devel] LXC with 2 and more NICs

2017-02-06 Thread Emmanuel Kasper
On 02/02/2017 10:03 PM, Detlef Bracker wrote: > Dear, > > I thing so, thats a bug! > > A ping from outside to the LXC-containers to all NICs works fine! > > A ping from console via the NICs 2- is not possible! So, this can > been a big problem, when a daemon will send from the NICs 2- >

Re: [pve-devel] applied: [PATCH firewall] simulator: make lxc/qemu optional

2017-02-06 Thread Stefan Priebe - Profihost AG
To avoid cycling deps an alternative might be forward decleration. http://www.perlmonks.org/?node_id=1057957 Stefan Excuse my typo sent from my mobile phone. Am 06.02.2017 um 18:38 schrieb Dietmar Maurer : >> An alternative might be >> http://perldoc.perl.org/autouse.html > > Thank for that i

[pve-devel] [PATCH manager 1/3] Fix handling of failed API call verifyFeature()

2017-02-06 Thread Emmanuel Kasper
This call can for instance fails when a source VM has references to non existing storages. --- www/manager6/qemu/Clone.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/qemu/Clone.js index ce6e4f3..8a3b34a 100644 --- a/www/manager6/qem

[pve-devel] [PATCH manager 0/3] Add a Clone button to the VM toolbar

2017-02-06 Thread Emmanuel Kasper
Cloning a VM was up to now only accessible via right clicking the resource tree, a path that many novice PVE users did not notice. Adding this is as a general button will make the feature easier to find. Emmanuel Kasper (3): Fix handling of failed API call verifyFeature() Move the snapshot AP

[pve-devel] [PATCH manager 3/3] Add a clone button in the VM toolbar

2017-02-06 Thread Emmanuel Kasper
Up to now the Migrate function had a first class button in the toolbar, but the Clone function didn't. With this button the toolbar still fit correctly in a smaller 1280 pixel wide display. --- www/manager6/qemu/Config.js | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --gi

[pve-devel] [PATCH manager 2/3] Move the snapshot API test to a static function

2017-02-06 Thread Emmanuel Kasper
This will allow code reuse in a following patch. No functional change. --- www/manager6/qemu/Clone.js | 25 +++-- www/manager6/qemu/CmdMenu.js | 18 +- 2 files changed, 24 insertions(+), 19 deletions(-) diff --git a/www/manager6/qemu/Clone.js b/www/manager6/

Re: [pve-devel] LXC with 2 and more NICs

2017-02-06 Thread Detlef Bracker
Dear Emmanuel, I thing so, thats a bug! It can been make better, but I thing so the developer has more expirience about the networking and they must test it, is this a bug or not! So I have make it in Promox 3 for bridges in this following form. This work with all NICs via same bridge OR with NIC