The replacement is Pool (singular).
Signed-off-by: Aaron Lauterer
---
PVE/API2/Ceph.pm | 7 -
PVE/API2/Ceph/Makefile | 1 -
PVE/API2/Ceph/Pools.pm | 766 -
3 files changed, 774 deletions(-)
delete mode 100644 PVE/API2/Ceph/Pools.pm
diff --git a
Both are superseeded by ceph/cfg/raw and ceph/cfg/db
Signed-off-by: Aaron Lauterer
---
PVE/API2/Ceph.pm | 73
1 file changed, 73 deletions(-)
diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm
index 4893c957..1b765b75 100644
--- a/PVE/API2/Ceph.pm
ping?
On 4/11/23 15:40, Aaron Lauterer wrote:
The main goal of this series is to improve the handling of configured
default size & min_size values when creating a new Ceph Pool in the GUI.
A new Ceph API endpoint, 'cfg/value', is added. It allows us to fetch
values for config keys that are set
applied series, thanks
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
picked this into stable-7 as well
On Tue, Jun 13, 2023 at 02:06:32PM +0200, Leo Nunner wrote:
> in the cluster class, we save the cluster config into the 'fw_conf'
> variable, and not into 'cluster_conf', which in turns is set to 'undef'
> instead.
>
> Signed-off-by: Leo Nunner
> ---
> src/PVE/
this field has a fixed height, which is too tall for the create wizard,
and one cannot see the last entry when scrolling all the way down.
to fix this, make it 20px shorter
Signed-off-by: Dominik Csapak
---
alternatively we could also increase the height in the create wizard
the optimal solutio
forgot to write: we should apply this for stable-7 as well
On 6/15/23 10:08, Dominik Csapak wrote:
this field has a fixed height, which is too tall for the create wizard,
and one cannot see the last entry when scrolling all the way down.
to fix this, make it 20px shorter
Signed-off-by: Dominik
if a real json boolean is sent via the api, $value is a
JSON::PP::Boolean here instead of a string/scalar
so we should validate that too
the $value itself can be used normally in conditions like
if ($value) {
This worked for most api calls by accident before commit:
f398a3d ("proxy req
Changes since v1:
- fix some style nits
- add a new component to show IPs in the LXC summary page
container:
Leo Nunner (1):
api: network: get interfaces from containers
src/PVE/API2/LXC.pm | 50 +
src/PVE/LXC.pm | 26 ++
Adds an 'interfaces' endpoint in the API
(/nodes/{node}/lxc/{vmid}/interfaces'), which returns a list of
interface names, together with a MAC, IPv4 and IPv6 address. This list
may be expanded in the future. Note that this is only returned for
*running* containers, stopped containers simply return a
modelled after the QEMU Guest Agent UI. We only show the first
non-loopback IP on the summary page itself.
Signed-off-by: Leo Nunner
---
www/manager6/Makefile | 1 +
www/manager6/lxc/ContainerIPView.js | 194 ++
www/manager6/panel/GuestStatusView.js |
adds a call to /nodes/{node}/lxc/{vmid}/interfaces and merges the
returned data with the existing configuration. This will update the
IPv4 and IPv6 address, as well as the interface name (in case the
container changed it).
Signed-off-by: Leo Nunner
---
www/manager6/lxc/Network.js | 57 ++
On Thu, Jun 15, 2023 at 11:32:15AM +0200, Dominik Csapak wrote:
> if a real json boolean is sent via the api, $value is a
> JSON::PP::Boolean here instead of a string/scalar
>
> so we should validate that too
>
> the $value itself can be used normally in conditions like
>
> if ($value) {
> -
during some tests recently I wondered why a debug log-message was not
printed, despite running with PBS_QEMU_DEBUG.
This patch sets the loglevel for the cli logger to debug if the
variable is present and not-empty (see qemu_helper.rs for the other
usage).
Signed-off-by: Stoiko Ivanov
---
proxmo
The first two patches fix the setting of zfs_arc_min and zfs_arc_max via
the kernel commandline - afaics they should not have been honored until
now - verification was done in the debug-image by inspecting
`grep '^c_' /proc/spl/kstat/zfs/arcstats`
the first 2 patches can be squashed - it explicit
zfs_arc_min was raised to 32M (for linux) in zfs-commit
121b3cae742a0670d902a51bc61d49dc4a3e4445
while the current logic would still set the min_size to 32M (it's
max(32M, allmem/32), which results to 32M for memory sizes up to
1024M), setting it explicitly to the minimum makes it clear, and will
Currently the values set for zfs_arc_min and zfs_arc_max are ignored
by the kernel:
```
Unknown kernel command line parameters... will be passed to user space
```
module parameters provided on the commandline usually need to be
prefixed with the modulename (e.g. zfs.zfs_arc_min, see [0] for a bit
o
On 6/15/23 11:51, Wolfgang Bumiller wrote:
On Thu, Jun 15, 2023 at 11:32:15AM +0200, Dominik Csapak wrote:
if a real json boolean is sent via the api, $value is a
JSON::PP::Boolean here instead of a string/scalar
so we should validate that too
the $value itself can be used normally in conditio
Since commit 8e623a2930f7aee4b3309b1f297613a250ee4698, the inequality
check for content-dirs prints a warning if a content directory path
could not be resolved, i.e., if `abs_path` returns undef. Among other
things, `abs_path` returns undef if the path has an inner (= any but
last) component that d
The QAPI change for QEMU 8.0 dropped redundant has_foo parameters, but
in the blockdev_mirror_common() function (which is not part of the
QAPI itself but called from there) the argument pair was has_bitmap
and bitmap_name rather than has_bitmap and bitmap.
Reported-by: Aaron Lauterer
Signed-off-b
There's still some context changes not covered by earlier series. No
functional change intended.
Signed-off-by: Fiona Ebner
---
...VE-add-savevm-async-for-background-state-snapshots.patch | 4 ++--
.../pve/0020-PVE-Add-dummy-id-command-line-parameter.patch | 4 ++--
...E-Compat-4.0-used-balloon
When adding or editing a storage device in Datacenter->Storage in the
web interface, the subdirectory depth can be set in the advanced menu.
Signed-off-by: Noel Ullreich
---
www/manager6/storage/Base.js | 11 +++
1 file changed, 11 insertions(+)
diff --git a/www/manager6/storage/Base.js
This patch fixes #623, allowing isos/vztmpl/snippets in subdirectories.
This feature is opt-in and can be set from the API, web interface or
with `pvesm`.
I addressed the security concerns raised by Fabian, now parent
directories in the path (i.e. `/my/path/../somewhere/`) are forbidded.
I have ke
Tests now also test if traversing subdirectories works. That means
checking if:
- parent directories in paths are caught
- checking that nested files are found
- checking that files below the maximum depth set are not found
Signed-off-by: Noel Ullreich
---
src/test/filesystem_path_test.pm | 1
Add the `subdir-depth` to the filesystems that can hold
isos/vztmpl/snippets.
Signed-off-by: Noel Ullreich
---
src/PVE/Storage/CIFSPlugin.pm | 1 +
src/PVE/Storage/CephFSPlugin.pm| 1 +
src/PVE/Storage/DirPlugin.pm | 1 +
src/PVE/Storage/GlusterfsPlugin.pm | 1 +
src/PVE/Storage/N
This patch allows `get_subdir_files` to recursively call itself, so that
subdirectories of set depth can be searched. We allow searching for
isos, vztmpl and snippets but not backups.
As a security measure, when parsing a given path, parent
directories (`/../`) are forbidden.
The feature is opt-i
Looks good from my side now, maybe someone else wants to chime in
regarding the splitting of the setting in the gui (but IMHO that's good)
consider the series:
Reviewed-by: Dominik Csapak
Tested-by: Dominik Csapak
___
pve-devel mailing list
pve-dev
Small series of patches to drop the use of properties and methods that were
deprecated in GTK 3.
Maximiliano Sandoval (4):
stop using Gtk3::Misc->set_alignment
stop using Gtk3::HBox and Gtk3::VBox
stop using Gtk3::HSeparator
stop using Gtk3::Window->set_has_resize_grip
proxinstall | 84 +
The property GtkWindow::has-resize-grip was deprecated in GTK 3.14,
instead we make the window unresizable.
Note that we set the value of resizable only after the window has been
mapped and knows how much space it needs, also note that this only works
correctly if the window is properly displayed
This was deprecated in GTK 3.2, the replacement Gtk3::Separator was
added in GTK 3.0.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/proxinstall b/proxinstall
index 105904f..d2f3d26 100755
--- a/proxinstall
+++ b/pr
These widgets were deprecated in GTK 3.2 and replaced by Gtk3::Box which
was introduced in 3.0.
The packing changes require to horizontally expand the WebKit2::WebView
widget.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 49 +
1 file chan
This was deprecated in GTK 3.14, instead we use
GtkLabel:{xalign,yalign}. Note that by default they are both set to 0.5.
Signed-off-by: Maximiliano Sandoval
---
proxinstall | 23 +++
1 file changed, 11 insertions(+), 12 deletions(-)
diff --git a/proxinstall b/proxinstall
ind
Am 15/06/2023 um 13:38 schrieb Fiona Ebner:
> There's still some context changes not covered by earlier series. No
> functional change intended.
>
> Signed-off-by: Fiona Ebner
> ---
> ...VE-add-savevm-async-for-background-state-snapshots.patch | 4 ++--
> .../pve/0020-PVE-Add-dummy-id-command-li
Am 15/06/2023 um 13:36 schrieb Friedrich Weber:
> Since commit 8e623a2930f7aee4b3309b1f297613a250ee4698, the inequality
> check for content-dirs prints a warning if a content directory path
> could not be resolved, i.e., if `abs_path` returns undef. Among other
> things, `abs_path` returns undef if
Am 12/06/2023 um 16:27 schrieb Fiona Ebner:
> This avoids confusing errors about other properties when the storage
> type doesn't match. By highlighting that the type doesn't match, users
> should know right away what the issue is.
>
> Signed-off-by: Fiona Ebner
> ---
>
> Changes in v2:
> *
Am 15/06/2023 um 13:12 schrieb Dominik Csapak:
> On 6/15/23 11:51, Wolfgang Bumiller wrote:
>> On Thu, Jun 15, 2023 at 11:32:15AM +0200, Dominik Csapak wrote:
>>> if a real json boolean is sent via the api, $value is a
>>> JSON::PP::Boolean here instead of a string/scalar
>>>
>>> so we should valid
if a real json boolean is sent via the api, $value is a
JSON::PP::Boolean here instead of a string/scalar
so we should validate that too
the $value itself can be used normally in conditions like
if ($value) {
if the value is stringified, the result is either '1' or '0' so even if
we ha
On Thu, Jun 15, 2023 at 02:28:02PM +0200, Thomas Lamprecht wrote:
> Am 15/06/2023 um 13:12 schrieb Dominik Csapak:
> > On 6/15/23 11:51, Wolfgang Bumiller wrote:
> >> On Thu, Jun 15, 2023 at 11:32:15AM +0200, Dominik Csapak wrote:
> >>> if a real json boolean is sent via the api, $value is a
> >>>
Works fine.
Seems to be a good addition for organizing ISOs, vztmpl, etc.
You could add a selector for a directory or a "destination path"
text-input to the file upload as a follow-up patch.
Tested-by: Markus Frank
On 6/15/23 14:03, Noel Ullreich wrote:
This patch fixes #623, allowing isos/vz
applied, thanks
On Thu, Jun 15, 2023 at 02:36:43PM +0200, Dominik Csapak wrote:
> if a real json boolean is sent via the api, $value is a
> JSON::PP::Boolean here instead of a string/scalar
>
> so we should validate that too
>
> the $value itself can be used normally in conditions like
>
>
The current inequality check for content-dirs does not correctly
handle the case in which `abs_path` returns undef. This can result in
confusing warnings:
storage [...] uses directory for multiple content types [...]
Fix this by skipping paths for which `abs_path` returns undef. This
matches t
--- Begin Message ---
Hi,
We have a customer that allows people to build EthernetOverIP tunnels,
and bridge the endpoints of those tunnels to their hypervisor interface.
This causes the hypervisor to transmit BPDU's, which causes the
uplink-switches to trigger on security and shut the uplink.
configuring pbs-entries-max can avoid failing backups due to a high
amount of files in folders where a folder exclusion is not possible
Signed-off-by: Alexander Zeidler
---
Changes from v1:
Add 'pbs-entries-max' according to the new shortened permission check
PVE/API2/Backup.pm | 2 +-
configs
Alter style to make the parameter check more concise
Signed-off-by: Alexander Zeidler
---
Changes from v1:
Shorten permission check
PVE/API2/Backup.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/PVE/API2/Backup.pm b/PVE/API2/Backup.pm
index 45eb47e2..70753c2e 100644
---
Signed-off-by: Alexander Zeidler
---
Changes from v1:
Improve description
Move description to 'performance' section
Remove arrow alignment
src/PVE/VZDump/Common.pm | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/PVE/VZDump/Common.pm b/src/PVE/VZDump/Common.pm
index a6fe483..
configuring pbs-entries-max can avoid failing backups due to a high
amount of files in folders where a folder exclusion is not possible
Signed-off-by: Alexander Zeidler
---
Changes from v1:
Reword loginfo message and include new set value
Fix indentation
src/PVE/VZDump/LXC.pm | 5 +
1 file
Users reported[1] about failing backups to PBS when having a large amount of
files in a directory.
proxmox-backup-client has already 'entries-max' implemented. The default value
is also a kind of DOS prevention feature. Overriding this parameter needs at
least 'Sys.Modify' permissions.
This pa
Am 15/06/2023 um 15:22 schrieb Friedrich Weber:
> The current inequality check for content-dirs does not correctly
> handle the case in which `abs_path` returns undef. This can result in
> confusing warnings:
>
> storage [...] uses directory for multiple content types [...]
>
> Fix this by ski
Am 15/06/2023 um 12:42 schrieb Stoiko Ivanov:
> The first two patches fix the setting of zfs_arc_min and zfs_arc_max via
> the kernel commandline - afaics they should not have been honored until
> now - verification was done in the debug-image by inspecting
> `grep '^c_' /proc/spl/kstat/zfs/arcstat
49 matches
Mail list logo
