Signed-off-by: Rhonda D'Vine
---
debian/control | 1 +
1 file changed, 1 insertion(+)
diff --git a/debian/control b/debian/control
index bb01640..cf9a753 100644
--- a/debian/control
+++ b/debian/control
@@ -4,6 +4,7 @@ Priority: optional
Maintainer: Proxmox Support Team
Build-Depends
unzip is needed in the Build-Depends. While adding it I noticed that
it makes sense to switch the Homepage URL directly to https.
Rhonda D'Vine (2):
Add unzip to Build-Depends
Use https for Homepage URL
debian/control | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
--
2.11.0
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index cf9a753..e3bc4c8 100644
--- a/debian/control
+++ b/debian/control
@@ -6,7 +6,7 @@ Build-Depends: debhelper (>= 9),
p
The lxc packages were missing the SOURCE file. This patch adds them.
Rhonda D'Vine (1):
Add debian/SOURCE to docs to install
debian/docs | 1 +
debian/lxc-pve-dev.docs | 1 +
2 files changed, 2 insertions(+)
create mode 100644 debian/lxc-pve-dev.docs
--
2.11.0
The lxc packages were missing the SOURCE file in the docs.
Signed-off-by: Rhonda D'Vine
---
debian/docs | 1 +
debian/lxc-pve-dev.docs | 1 +
2 files changed, 2 insertions(+)
create mode 100644 debian/lxc-pve-dev.docs
diff --git a/debian/docs b/debian/docs
index e845566..310840d
These were the missing Build-Depends that were needed in building this
package.
Signed-off-by: Rhonda D'Vine
---
debian/control | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index e0860c33..7e2a91e5 100644
--- a/debian/control
The CIPHERS aren't really state-of-the-art. This change was inspired by
https://forum.proxmox.com/threads/the-robot-attack-vulnerability-on-proxmox-5-2.47652/
The updated CIPHERS string was tested with https://www.ssllabs.com/ssltest/
Rhonda D'Vine (2):
Update default CIPHERS to a more
The default CIPHERS allowed for a fair amount of not really considered
secure anymore connections. This updated cipher list is taken from
mozilla: https://wiki.mozilla.org/Security/Server_Side_TLS
Signed-off-by: Rhonda D'Vine
---
PVE/Service/pveproxy.pm | 2 +-
1 file changed, 1 insertion
Hi,
On 10/11/2018 04:58 PM, Fabian Grünbichler wrote:
> On Thu, Oct 11, 2018 at 12:05:20PM +0200, Rhonda D'Vine wrote:
>> These were the missing Build-Depends that were needed in building this
>> package.
>>
>> Signed-off-by: Rhonda D'Vine
>> ---
> pa
Even though this is meant as an example, it makes sense to keep it in
sync with what the actual default is.
Signed-off-by: Rhonda D'Vine
---
pveproxy.adoc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/pveproxy.adoc b/pveproxy.adoc
index fbb7394..251e98e 100644
Hi,
applied, and together with mirror_frr available in our git. Please base
future patches on top of that.
Enjoy,
Rhonda
On 10/16/18 5:46 AM, Alexandre Derumier wrote:
> changelog v3:
> - bump to 6.0
> - remove cumulus_mode patch. (vxlan support is enabled by default now)
> - add a build
This is the bash completion helper function for completing the snapshot
name. This is used both in qemu-server and pve-container.
This patch is the base for the patches in qemu-server and pve-container.
Signed-off-by: Rhonda D'Vine
---
PVE/AbstractConfig.pm | 13 +
1 file changed
The config CLI command offer the config and showcmd functions. The
output of that may vary with respect to a given snapshot. This adds a
switch that shows the corresponding snapshot's config.
This patch needs the one in pve-guest-common applied first.
Signed-off-by: Rhonda D'Vine
---
debian
These were found while building the package within a clean chroot.
Signed-off-by: Rhonda D'Vine
---
debian/control | 5 +
1 file changed, 5 insertions(+)
diff --git a/debian/control b/debian/control
index 45d7855..dc1a2bd 100644
--- a/debian/control
+++ b/debian/control
@@ -6,9 +6,14
This is the list of the properties that should get returned in the
pretty print format, too, not just in yaml/json output. Also adding
missing Build-Depends.
Rhonda D'Vine (2):
Fix #1971: display firewall rule properties
Add missing Build-Depends
debian/control | 2 ++
src
This is the list of the properties that should get returned in the
pretty print format, too, not just in yaml/json output.
Signed-off-by: Rhonda D'Vine
---
src/PVE/API2/Firewall/Rules.pm | 48 +-
1 file changed, 47 insertions(+), 1 deletion(-)
diff --git
Found while building in a clean chroot.
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index e959ae3..a68a81c 100644
--- a/debian/control
+++ b/debian/control
@@ -6,6 +6,8 @@ Build-Depends: debhelper
This patch allows the pvesh command to display firewall rules, not just its
position. This is related to https://bugzilla.proxmox.com/show_bug.cgi?id=1971
Rhonda D'Vine (2):
Fix #1971: display firewall rule properties
Add missing Build-Depends
debian/control | 2 ++
src
---
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index e959ae3..a68a81c 100644
--- a/debian/control
+++ b/debian/control
@@ -6,6 +6,8 @@ Build-Depends: debhelper (>= 7.0.50~),
dh-systemd,
libglib2.0-dev,
---
src/PVE/API2/Firewall/Rules.pm | 48 +-
1 file changed, 47 insertions(+), 1 deletion(-)
diff --git a/src/PVE/API2/Firewall/Rules.pm b/src/PVE/API2/Firewall/Rules.pm
index 1aa6d27..52a0935 100644
--- a/src/PVE/API2/Firewall/Rules.pm
+++
in
a clean chroot.
This patch needs the one in pve-guest-common applied first.
Rhonda D'Vine (2):
Fix #1924: add snapshot parameter
Add missing Build-Depends
PVE/API2/Qemu.pm | 16
PVE/CLI/qm.pm | 11 +--
PVE/QemuServer.pm | 12 +++-
debian/control| 7
-by: Rhonda D'Vine
---
PVE/API2/Qemu.pm | 16
PVE/CLI/qm.pm | 11 +--
PVE/QemuServer.pm | 12 +++-
debian/control| 2 +-
4 files changed, 37 insertions(+), 4 deletions(-)
diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm
index 52f4a5f..eb514b9 100644
Hi.
On 11/12/18 12:49 PM, Dominik Csapak wrote:
> On 11/12/18 12:27 PM, Rhonda D'Vine wrote:
>> ---
>> src/PVE/API2/Firewall/Rules.pm | 48
>> +-
>> 1 file changed, 47 insertions(+), 1 deletion(-)
>>
>> diff --gi
The config and showcmd CLI commands offer the config and showcmd
functions. Both of that outputs may vary with respect to a given
snapshot. This adds a switch that shows the corresponding snapshot's
config and command line.
Signed-off-by: Rhonda D'Vine
Rhonda D'Vine (2):
pve-container: Fix
/rules adjusted.
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 +-
src/PVE/API2/LXC/Config.pm | 19 +++
2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index 15f42ad..42a07df 100644
--- a/debian/control
+++ b/debian
the
versioned dependency in debian/control adjusted.
Signed-off-by: Rhonda D'Vine
---
PVE/API2/Qemu.pm | 19 +++
PVE/CLI/qm.pm | 12 ++--
PVE/QemuServer.pm | 13 -
3 files changed, 41 insertions(+), 3 deletions(-)
diff --git a/PVE/API2/Qemu.pm b/PVE/API2
Also adjust the debhelper minimum version to what's noted down in
debian/compat.
Signed-off-by: Rhonda D'Vine
---
debian/control | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index c921588..a9a9705 100644
--- a/debian/control
+++ b/debian
Also adjust debhelper dependency according to debian/compat content.
Signed-off-by: Rhonda D'Vine
---
debian/control | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index 7b326e8..392d0ed 100644
--- a/debian/control
+++ b/debian/control
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index 74823cb..f401118 100644
--- a/debian/control
+++ b/debian/control
@@ -6,9 +6,11 @@ Build-Depends: debhelper (>= 10~),
dh-syst
This is the bash completion helper function for completing the snapshot
name. This is used both in qemu-server and pve-container.
Signed-off-by: Rhonda D'Vine
---
PVE/AbstractConfig.pm | 18 ++
1 file changed, 18 insertions(+)
diff --git a/PVE/AbstractConfig.pm b/PVE
/rules adjusted.
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 +-
src/PVE/API2/LXC/Config.pm | 19 +++
2 files changed, 20 insertions(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index 15f42ad..42a07df 100644
--- a/debian/control
+++ b/debian
the
versioned dependency in debian/control adjusted.
Signed-off-by: Rhonda D'Vine
---
PVE/API2/Qemu.pm | 19 +++
PVE/CLI/qm.pm | 12 ++--
PVE/QemuServer.pm | 13 -
3 files changed, 41 insertions(+), 3 deletions(-)
diff --git a/PVE/API2/Qemu.pm b/PVE/API2
Signed-off-by: Rhonda D'Vine
---
Makefile | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 485cf02..890d4fb 100644
--- a/Makefile
+++ b/Makefile
@@ -1,4 +1,6 @@
-RELEASE=4.0
+VERSION != dpkg-parsechangelog -Sversion | cut -d- -f1
+PACKAGE != dpkg
This splits out the copying of the BUILDSRC into its own target. It also
adds the missing git SOURCE file to the package, and doesn't rely on a
hardcoded ARCH in the Makefile.
Signed-off-by: Rhonda D'Vine
---
Makefile| 35 +--
debian/docs | 1 +
2 files
The package meta data gets pulled from the debian/changelog instead of
being hardcoded. I also got rid of a few more hardcoded things, like the
ARCH. The central point though is adding the make dsc target for enabling
autobuilding.
Rhonda D'Vine (2):
Fetch package + version from the changelog
Pull out the build dir creation into its own target, and add the "dsc"
target. Also get some static information from the packaging.
Signed-off-by: Rhonda D'Vine
---
Makefile | 31 +--
1 file changed, 21 insertions(+), 10 deletions(-)
diff --git a/
Hi.
On 1/7/19 2:25 PM, Thomas Lamprecht wrote:
> On 1/7/19 1:51 PM, Rhonda D'Vine wrote:
>> Pull out the build dir creation into its own target, and add the "dsc"
>> target. Also get some static information from the packaging.
>>
>> Signed-off-by: Rh
Thanks, applied, minus some minimal whitespace change in patch 4 and 5.
Enjoy,
Rhonda
P.S.: I submitted the issue I found in the frr.postinst file as pull
request directly to upstream: https://github.com/FRRouting/frr/pull/3565
On 12/31/18 10:29 AM, Alexandre Derumier wrote:
> Update frr
Also use https for the homepage while in there already.
Signed-off-by: Rhonda D'Vine
---
debian/control.in | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/debian/control.in b/debian/control.in
index ec12fb6..b44e4a5 100644
--- a/debian/control.in
+++ b/debian/control.in
This package was missing the SOURCE information from the binary package.
Signed-off-by: Rhonda D'Vine
---
Makefile| 2 ++
debian/docs | 1 +
2 files changed, 3 insertions(+)
create mode 100644 debian/docs
diff --git a/Makefile b/Makefile
index 7401d08..8269eae 100644
--- a/Makefile
+++ b
r packages.
Rhonda D'Vine (3):
Add "make dsc" target
Add SOURCE file
Get version information from changelog
Makefile| 29 +
debian/docs | 1 +
2 files changed, 22 insertions(+), 8 deletions(-)
create mode 100644 deb
This target is required to make the package build automatically.
Signed-off-by: Rhonda D'Vine
---
Makefile | 23 +--
1 file changed, 17 insertions(+), 6 deletions(-)
diff --git a/Makefile b/Makefile
index ec67f2d..7401d08 100644
--- a/Makefile
+++ b/Makefile
@@ -2,7 +2,9
The duplication of the version information both in the Makefile and in
debian/changelog is a potential error point, and an unneeded one.
Signed-off-by: Rhonda D'Vine
---
Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index 8269eae..c02ba14
layouts
are pulled from PVE::Tools::kvmkeymaps().
Rhonda D'Vine (1):
Fix #1527: Use 'iso-codes' package country naming
country.pl | 14 --
debian/control | 1 +
2 files changed, 5 insertions(+), 10 deletions(-)
--
2.11.0
___
pve
The iso 3166-1 information in the iso-codes package seems to be more
up to par than the one that was used from tzdata.
Signed-off-by: Rhonda D'Vine
---
country.pl | 14 --
debian/control | 1 +
2 files changed, 5 insertions(+), 10 deletions(-)
diff --git a/country.pl b
This is an analogous change to what was done in pve-docs last year.
The Build-Depends has been adjusted to list all required ones, there
were some missing. Also, the compat level is set to 9 so bump
debhelper Build-Depends to >= 9~.
Signed-off-by: Rhonda D'Vine
---
Makef
This is an analogous change to what was done in pve-docs last year.
The Build-Depends has been adjusted to list all required ones, there
were some missing. Also, the compat level is set to 9 so bump
debhelper Build-Depends to >= 9~.
Signed-off-by: Rhonda D'Vine
---
Makef
minor outlying islands:::
> um:United States Minor Outlying Islands:::
< va:Vatican City:it::
> va:Holy See (Vatican City State):it::
< vc:St Vincent:::
> vc:Saint Vincent and the Grenadines:::
< vg:Virgin Islands (UK):::
> vg:Virgin Islands, British:::
< vi:
This one was missing. Formating got adjusted too.
Signed-off-by: Rhonda D'Vine
---
debian/control | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index f991da7..b84ae1c 100644
--- a/debian/control
+++ b/debian/control
@@ -2,7 +2,10
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/debian/control b/debian/control
index b84ae1c..60bc8b0 100644
--- a/debian/control
+++ b/debian/control
@@ -7,7 +7,7 @@ Build-Depends: debhelper (>= 9),
librs
The iso 3166-1 information in the iso-codes package seems to be more
up to par than the one that was used from tzdata.
Signed-off-by: Rhonda D'Vine
---
country.pl | 18 --
debian/control | 1 +
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/country.pl b
The iso 3166-1 information in the iso-codes package seems to be more
up to par than the one that was used from tzdata. Also remove
obsolete comments.
Signed-off-by: Rhonda D'Vine
---
country.pl | 20 +---
debian/control | 1 +
2 files changed, 6 insertions(+), 15 deletions
The config CLI command offer the config and showcmd functions. The
output of that may vary with respect to a given snapshot. This adds a
switch that shows the corresponding snapshot's config.
Also add debian/SOURCE to debian/docs for installation into the package.
Signed-off-by: Rhonda D'Vine
.
Rhonda D'Vine (2):
pve-container: Fix #1924: add snapshot parameter
qemu-server: Fix #1924: add snapshot parameter
pve-container/debian/control | 2 +-
pve-container/debian/docs| 1 +
pve-container/src/PVE/API2/LXC/Config.pm | 19 +++
qemu-server
The config and showcmd CLI commands offer the config and showcmd
functions. Both of that outputs may vary with respect to a given
snapshot. This adds a switch that shows the corresponding snapshot's
config and command line.
Signed-off-by: Rhonda D'Vine
---
PVE/API2/Qemu.pm | 19
-by: Rhonda D'Vine
---
PVE/API2/Qemu/Agent.pm | 26 +++---
PVE/QMPClient.pm | 9 -
2 files changed, 23 insertions(+), 12 deletions(-)
diff --git a/PVE/API2/Qemu/Agent.pm b/PVE/API2/Qemu/Agent.pm
index 839146c..4e3dc01 100644
--- a/PVE/API2/Qemu/Agent.pm
+++ b/PVE
The guest cmd commands set different timeouts. Some of those might take
longer, so for debugging purposes it would be useful to allow overriding
the internal timeout setting.
Also adds missing Build-Depends
Rhonda D'Vine (2):
Fix #2062: New timeout switch for guest cmd
Add missing Build
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index 1fb7ceb..beb82ad 100644
--- a/debian/control
+++ b/debian/control
@@ -12,9 +12,11 @@ Build-Depends: debhelper (>= 7.0.50~),
libxml-lib
Signed-off-by: Rhonda D'Vine
---
src/PVE/OTP.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/PVE/OTP.pm b/src/PVE/OTP.pm
index c062639..019076b 100644
--- a/src/PVE/OTP.pm
+++ b/src/PVE/OTP.pm
@@ -55,7 +55,7 @@ sub yubico_verify_otp {
die "yubico: wron
The guest cmd commands set different timeouts. Some of those might take
longer, so for debugging purposes it would be useful to allow overriding
the internal timeout setting.
Signed-off-by: Rhonda D'Vine
---
PVE/API2/Qemu/Agent.pm | 9 -
1 file changed, 8 insertions(+), 1 deletion
.
Rhonda D'Vine (2):
Fix #1924: add snapshot parameter
Add debian/SOURCE to docs
debian/control | 2 +-
debian/docs| 1 +
src/PVE/API2/LXC/Config.pm | 19 +++
3 files changed, 21 insertions(+), 1 deletion(-)
create mode 100644 debian/docs
The pct CLI command offer the config function. The output of that may
vary with respect to a given snapshot. This adds a switch that shows the
corresponding snapshot's config.
The code needs a newer libpve-guest-common-perl, thus bumping the
dependency.
Signed-off-by: Rhonda D'Vine
---
debian
Signed-off-by: Rhonda D'Vine
---
debian/docs | 1 +
1 file changed, 1 insertion(+)
create mode 100644 debian/docs
diff --git a/debian/docs b/debian/docs
new file mode 100644
index 000..8696672
--- /dev/null
+++ b/debian/docs
@@ -0,0 +1 @@
+debian/SOURCE
--
2.11.0
-by: Rhonda D'Vine
---
PVE/API2/Qemu.pm | 19 +++
PVE/CLI/qm.pm | 12 ++--
PVE/QemuServer.pm | 13 -
debian/control| 2 +-
4 files changed, 42 insertions(+), 4 deletions(-)
diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm
index 148ea1a..7f737bf 100644
in the GUI too.
Signed-off-by: Rhonda D'Vine
---
PVE/CLI/qm.pm | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm
index 87b97ce..b5e838e 100755
--- a/PVE/CLI/qm.pm
+++ b/PVE/CLI/qm.pm
@@ -926,12 +926,13 @@ our $cmddef
It is useful to have a way to see the translation statistics. The
output could potentially get used to have an overview page about the
translations and invite people to help out through that.
Signed-off-by: Rhonda D'Vine
---
Makefile | 3 +++
1 file changed, 3 insertions(+)
diff --git
The look of the tree is based on the GUI variant, so that we have a
consistent output when run multiple times, too.
Signed-off-by: Rhonda D'Vine
---
PVE/CLI/qm.pm | 52
1 file changed, 48 insertions(+), 4 deletions(-)
diff --git a/PVE/CLI
Hi,
thanks for the suggestions, they helped a lot. :)
On 5/8/19 11:26 AM, Thomas Lamprecht wrote:
> On 5/3/19 11:48 AM, Rhonda D'Vine wrote:
>> @@ -922,12 +922,67 @@ our $cmddef = {
>> listsnapshot => [ "PVE::API2::Qemu", 'snapshot_list', ['vmi
The look of the tree is based on the GUI variant, so that we have a
consistent output when run multiple times, too.
Signed-off-by: Rhonda D'Vine
---
PVE/CLI/qm.pm | 63 +++
1 file changed, 59 insertions(+), 4 deletions(-)
diff --git a/PVE
The first commit was a quick approach to have the output sorted - the second
commit creates the tree view like it's in the web GUI, too.
Rhonda D'Vine (2):
fix #1999: sort listsnapshot output by snaptime
fix #1999: create a tree view for qm listsnapshot
PVE/CLI/qm.pm | 61
Signed-off-by: Rhonda D'Vine
---
PVE/CLI/qm.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/PVE/CLI/qm.pm b/PVE/CLI/qm.pm
index cea2549..fc06fa5 100755
--- a/PVE/CLI/qm.pm
+++ b/PVE/CLI/qm.pm
@@ -922,7 +922,7 @@ our $cmddef = {
listsnapshot => [ "PVE::AP
Hi,
sorry for fumbling with the subject - this is for qemu-server of course.
So long,
Rhonda
On 5/3/19 11:48 AM, Rhonda D'Vine wrote:
> The first commit was a quick approach to have the output sorted - the second
> commit creates the tree view like it's in the web GUI, too.
>
This two missing dependencies makes it possible to install the package
on a stock Debian system (without PVE)
Signed-off-by: Rhonda D'Vine
---
debian/control | 2 ++
1 file changed, 2 insertions(+)
diff --git a/debian/control b/debian/control
index 82a5402..002c046 100644
--- a/debian/control
73 matches
Mail list logo