public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>,
	Stoiko Ivanov <s.ivanov@proxmox.com>
Subject: [pve-devel] applied-series: [PATCH installer 0/4] adaptation to kernel cmdline handling and 2 small ZFS related improvements
Date: Thu, 16 Nov 2023 17:46:36 +0100	[thread overview]
Message-ID: <d2ab7076-63ee-4daa-9e16-5f42d8bd6f7b@proxmox.com> (raw)
In-Reply-To: <20231116150041.1089432-1-s.ivanov@proxmox.com>

Am 16/11/2023 um 16:00 schrieb Stoiko Ivanov:
> This patch-series contains mostly unrelated patches, which still can be
> tested well together:
> * patch 1/4 fixes the parsing of our kernel cmdline parameters during the
>   install (they were ignored if being the last on the line).
>   additionally everything not used by our installer now end up in the
>   target system's kernel cmdline - so if you need to add something for
>   the system to boot - you do not need to do so 3 times (for the install,
>   for the first boot, for the boot-loader config)
> * patch 2/4 is a minor glitch I introduced, which caused me to debug the
>   wrong thing for too long
> * patch 3/4 has been a long-standing and quite sensible request
> * patch 4/4 fixes a minor inconvenience on machines with local users and
>   ZFS on root (dmesg gets filled with journald-messages upon
>   journal-rotation and user-logins)
> 
> Tested the changes a bit (4/4 on my machine, 3/4 was straight-forward, 2/4
> with a quick look, 1/4 with a few installer-options and net.ifnames=0)
> 
> Stoiko Ivanov (4):
>   fix #4747: pass kernel cmdline parameters to target system
>   unconfigured.sh: set serial to a number to prevent warning
>   fix #1410: zfs: create /var/lib/vz as separate dataset
>   zfs: set acltype=posix for root-dataset
> 
>  Proxmox/Install.pm        | 16 ++++++++++------
>  Proxmox/Install/Config.pm | 25 +++++++++++++------------
>  unconfigured.sh           |  1 +
>  3 files changed, 24 insertions(+), 18 deletions(-)
> 


applied this one a bit ago, please send further fixes as follow up, thanks!




      parent reply	other threads:[~2023-11-16 16:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-16 15:00 [pve-devel] " Stoiko Ivanov
2023-11-16 15:00 ` [pve-devel] [PATCH installer 1/4] fix #4747: pass kernel cmdline parameters to target system Stoiko Ivanov
2023-11-16 15:00 ` [pve-devel] [PATCH installer 2/4] unconfigured.sh: set serial to a number to prevent warning Stoiko Ivanov
2023-11-16 15:00 ` [pve-devel] [PATCH installer 3/4] fix #1410: zfs: create /var/lib/vz as separate dataset Stoiko Ivanov
2023-11-16 16:31   ` Stoiko Ivanov
2023-11-16 15:00 ` [pve-devel] [PATCH installer 4/4] zfs: set acltype=posix for root-dataset Stoiko Ivanov
2023-11-16 16:46 ` Thomas Lamprecht [this message]

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=d2ab7076-63ee-4daa-9e16-5f42d8bd6f7b@proxmox.com \
    --to=t.lamprecht@proxmox.com \
    --cc=pve-devel@lists.proxmox.com \
    --cc=s.ivanov@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
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal