From: Max Carrara <m.carrara@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH v4 ceph master, ceph quincy-stable-8, pve-storage, pve-manager 0/16] Fix #4759: Configure Permissions for ceph-crash.service
Date: Tue, 5 Mar 2024 16:07:42 +0100 [thread overview]
Message-ID: <20240305150758.252669-1-m.carrara@proxmox.com> (raw)
Fix #4759: Configure Permissions for ceph-crash.service - Version 4
===================================================================
Notable changes since v3
------------------------
* Both parser and writer for 'ceph.conf' now have unit tests which run
during make targets like e.g. `make deb`, thanks to `dh_auto_test`
* The parser for 'ceph.conf' now correctly un-escapes comment literals
(found while developing unit tests)
* The writer for 'ceph.conf' now correctly escapes comment literals
(found while developing unit tests)
* The helper script called in 'postinst' of pve-manager for updating
'ceph.crash' in 'ceph.conf' now correctly handles an existing key
being referenced directly and removes it (thanks Friedrich!)
* The aforementioned helper script has more verbose output, showing
explicitly what's being done to the configuration
* The 'postinst' hook now prints an empty line before and after it
runs to make it a little more visible
* The 'postinst' hook now also restarts 'ceph-crash.service' if the
user hasn't disabled it (thanks Friedrich!)
For a detailed list of changes, please see the comments in the
individual patches.
Older Versions
--------------
v1: https://lists.proxmox.com/pipermail/pve-devel/2024-January/061546.html
v2: https://lists.proxmox.com/pipermail/pve-devel/2024-February/061646.html
v3: https://lists.proxmox.com/pipermail/pve-devel/2024-February/061802.html
Summary of Changes
------------------
ceph (master):
Max Carrara (2):
debian: add patch to fix ceph crash dir permissions in postinst hook
patches: add patch that reorders clients used by ceph-crash
...ly-adjust-permissions-of-var-lib-cep.patch | 54 +++++++++++++++++++
...h-crash-change-order-of-client-names.patch | 30 +++++++++++
patches/series | 2 +
3 files changed, 86 insertions(+)
create mode 100644 patches/0016-debian-recursively-adjust-permissions-of-var-lib-cep.patch
create mode 100644 patches/0017-ceph-crash-change-order-of-client-names.patch
ceph (quincy-stable-8):
Max Carrara (2):
debian: add patch to fix ceph crash dir permissions in postinst hook
patches: add patch that reorders clients used by ceph-crash
...ly-adjust-permissions-of-var-lib-cep.patch | 54 +++++++++++++++++++
...h-crash-change-order-of-client-names.patch | 30 +++++++++++
patches/series | 2 +
3 files changed, 86 insertions(+)
create mode 100644 patches/0025-debian-recursively-adjust-permissions-of-var-lib-cep.patch
create mode 100644 patches/0026-ceph-crash-change-order-of-client-names.patch
pve-storage:
Max Carrara (9):
cephconfig: align our parser more with Ceph's parser
cephconfig: support line-continuations in parser
cephconfig: allow writing arbitrary sections
cephconfig: support escaped comment literals
cephconfig: emit warning for lines that fail to parse
cephconfig: change code style inside config writer
cephconfig: change order of written sections
cephconfig: remove leading whitespace on write to Ceph config
test: add tests for 'ceph.conf' parser and writer
src/Makefile | 1 +
src/PVE/CephConfig.pm | 95 +++--
src/PVE/Makefile | 4 +
src/PVE/test/Makefile | 9 +
src/PVE/test/ceph_conf_parse_write_test.pl | 402 +++++++++++++++++++++
5 files changed, 490 insertions(+), 21 deletions(-)
create mode 100644 src/PVE/test/Makefile
create mode 100755 src/PVE/test/ceph_conf_parse_write_test.pl
pve-manager:
Max Carrara (3):
ceph: introduce '/etc/pve/ceph'
fix #4759: ceph: configure ceph-crash.service and its key
bin/make: gather helper scripts in separate variable
PVE/API2/Ceph.pm | 5 ++
PVE/API2/Ceph/MON.pm | 8 +++
PVE/Ceph/Tools.pm | 47 ++++++++++++++-
bin/Makefile | 6 +-
bin/pve-init-ceph-crash | 129 ++++++++++++++++++++++++++++++++++++++++
debian/postinst | 26 ++++++++
6 files changed, 218 insertions(+), 3 deletions(-)
create mode 100755 bin/pve-init-ceph-crash
--
2.39.2
next reply other threads:[~2024-03-05 15:08 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-05 15:07 Max Carrara [this message]
2024-03-05 15:07 ` [pve-devel] [PATCH v4 master ceph 1/16] debian: add patch to fix ceph crash dir permissions in postinst hook Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 master ceph 2/16] patches: add patch that reorders clients used by ceph-crash Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 quincy-stable-8 ceph 3/16] debian: add patch to fix ceph crash dir permissions in postinst hook Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 quincy-stable-8 ceph 4/16] patches: add patch that reorders clients used by ceph-crash Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 05/16] cephconfig: align our parser more with Ceph's parser Max Carrara
2024-03-19 9:38 ` Fabian Grünbichler
2024-03-19 15:58 ` Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 06/16] cephconfig: support line-continuations in parser Max Carrara
2024-03-19 9:37 ` Fabian Grünbichler
2024-03-19 15:59 ` Max Carrara
2024-03-20 16:59 ` Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 07/16] cephconfig: allow writing arbitrary sections Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 08/16] cephconfig: support escaped comment literals Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 09/13] cephconfig: emit warning for lines that fail to parse Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 10/16] cephconfig: change code style inside config writer Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 11/16] cephconfig: change order of written sections Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 12/16] cephconfig: remove leading whitespace on write to Ceph config Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-storage 13/16] test: add tests for 'ceph.conf' parser and writer Max Carrara
2024-03-19 9:36 ` Fabian Grünbichler
2024-03-19 16:00 ` Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-manager 14/16] ceph: introduce '/etc/pve/ceph' Max Carrara
2024-03-19 10:04 ` Fabian Grünbichler
2024-03-19 16:01 ` Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-manager 15/16] fix #4759: ceph: configure ceph-crash.service and its key Max Carrara
2024-03-19 10:04 ` Fabian Grünbichler
2024-03-19 17:41 ` Max Carrara
2024-03-20 8:05 ` Fabian Grünbichler
2024-03-20 9:25 ` Max Carrara
2024-03-05 15:07 ` [pve-devel] [PATCH v4 pve-manager 16/16] bin/make: gather helper scripts in separate variable Max Carrara
2024-03-08 12:37 ` [pve-devel] [PATCH v4 ceph master, ceph quincy-stable-8, pve-storage, pve-manager 0/16] Fix #4759: Configure Permissions for ceph-crash.service Friedrich Weber
2024-03-11 16:45 ` [pve-devel] partially-applied-series: " Thomas Lamprecht
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240305150758.252669-1-m.carrara@proxmox.com \
--to=m.carrara@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox