I tested this patch series on a fully up-to-date Proxmox VE 8.4.5
cluster of 3 nodes which I then updated to 9.0 BETA. I tested:
- Having both patched and unpatched nodes
- Migrating VMs from patched nodes to unpatched nodes.
- Having only patched nodes.
My test results and remarks:
1. The upd
Thanks for your feedback! Some answers inline.
On 08.05.25 11:14, Fabian Grünbichler wrote:
On May 6, 2025 11:09 am, Laurențiu Leahu-Vlăducu wrote:
In addition to the existing timed snapshots, it is now possible to
create named snapshots. For some use cases, it makes sense to give
snapshots a
purpose (for example, further testing).
Partially fixes #6252
Signed-off-by: Laurențiu Leahu-Vlăducu
---
src/bin/proxmox-offline-mirror-helper.rs | 6 +-
src/bin/proxmox_offline_mirror_cmds/mirror.rs | 32 ---
src/medium.rs | 4 +-
src/mirror.rs
Some comments inline
Otherwise, please consider:
Reviewed-by: Laurențiu Leahu-Vlăducu
Tested-by: Laurențiu Leahu-Vlăducu
On 04.03.25 12:58, Markus Frank wrote:
Signed-off-by: Markus Frank
---
v14:
* return HTML encoded comment
www/manager6/Makefile | 1 +
www/manager6/dc
Some comments inline
Otherwise, please consider:
Reviewed-by: Laurențiu Leahu-Vlăducu
Tested-by: Laurențiu Leahu-Vlăducu
On 04.03.25 12:58, Markus Frank wrote:
Signed-off-by: Markus Frank
---
v14:
* disable expose-xattr when expose-acl is set
* added missing writeback cache option
www
-by: Laurențiu Leahu-Vlăducu
Reviewed-by: Laurențiu Leahu-Vlăducu
On 04.03.25 12:57, Markus Frank wrote:
Virtio-fs is a shared file system that enables sharing a directory
between host and guest VMs. It takes advantage of the locality of
virtual machines and the hypervisor to get a higher
virtualization software also allow automatically mounting
such things, so I think it makes sense to at least explain how to do it,
even if we don't do it by default.
Some further comments inline
Otherwise, please consider:
Reviewed-by: Laurențiu Leahu-Vlăducu
On 04.03.25 12:57, Markus
repository instead.
Other than that, please consider:
Reviewed-by: Laurențiu Leahu-Vlăducu
Tested-by: Laurențiu Leahu-Vlăducu
On 11.03.25 12:52, Alexander Abraham wrote:
A user reported a bug where they were attempting to login into our
app for PVE and they used a password with two spaces at the end
Hello everyone,
I would need some feedback on a feature that was requested multiple
times by different users over the years. Specifically, many people have
complained that synchronizing Active Directory groups to PVE
partially/mostly fails due to many groups containing spaces by default:
http
nals with its UTF-16 encoding.
pve-common:
Laurențiu Leahu-Vlăducu (2):
fix #3256: SectionConfig: ensure UTF-8 encoding for general configs
SectionConfig: add unit test for UTF-8 configs
src/PVE/SectionConfig.pm| 10 +++---
test/section_config_test.pl | 25 +
://perldoc.perl.org/perlunifaq#When-should-I-decode-or-encode?
Signed-off-by: Laurențiu Leahu-Vlăducu
---
src/PVE/Storage/PBSPlugin.pm | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/PVE/Storage/PBSPlugin.pm b/src/PVE/Storage/PBSPlugin.pm
index 0808bcc..6d89440 100644
--- a/src/PVE
expect bytes instead of UTF-8.
Also, config files are now explicitly encoded as UTF-8 when writing
the config, preventing issues the other way around.
For more information, please read:
https://perldoc.perl.org/perlunifaq#When-should-I-decode-or-encode?
Signed-off-by: Laurențiu Leahu-Vlăducu
The unit test should prevent the issues explained in bug #3256 from
happening in the future.
Signed-off-by: Laurențiu Leahu-Vlăducu
---
test/section_config_test.pl | 25 +
1 file changed, 25 insertions(+)
diff --git a/test/section_config_test.pl b/test
13 matches
Mail list logo
