all lists on 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal