public inbox for kdevops@lists.linux.dev
 help / color / mirror / Atom feed
From: Daniel Gomez <da.gomez@kernel.org>
To: Luis Chamberlain <mcgrof@kernel.org>
Cc: Chuck Lever <chuck.lever@oracle.com>,
	Viacheslav Dubeyko <Slava.Dubeyko@ibm.com>,
	"slava@dubeyko.com" <slava@dubeyko.com>,
	"kdevops@lists.linux.dev" <kdevops@lists.linux.dev>
Subject: Re: [RFC PATCH] storage-subsystem: correct playbooks for Fedora distro
Date: Mon, 25 Aug 2025 14:16:31 +0200	[thread overview]
Message-ID: <30cd0793-18ce-435b-a6c1-94ddad1ea6ed@kernel.org> (raw)
In-Reply-To: <aKowyNaIG1olssoT@bombadil.infradead.org>

On 23/08/2025 23.21, Luis Chamberlain wrote:
> On Sat, Aug 23, 2025 at 02:33:26PM +0200, Daniel Gomez wrote:
>> On 23/08/2025 01.14, Luis Chamberlain wrote:
>>> So let's order our priorities and see who is willing to help tackle
>>> some of these challenges:
>>>
>>>   * nixos -- I have a branch now, seems to work but I gotta debug ssh
>>
>> That's good! Can you clarify if you refer to controller node NixOS support? or
>> guest support?
> 
> Guest support. I'm also adding mirroring support.
> 
>>> BTW I'm curious about the goal of making systemd services optional,
>>> did you mean things like systemd-journal and things like that? And
>>> mirroring?
>>
>> Yes, all these services we depend on. But I don't think we can sandbox systemd
>> services. But ensuring they all can run as user services allows us to remove
>> sudo.
> 
> I am not sure some of them can be run as user services. Like:
> 
>   * systemd-timesyncd.service
>   * systemd-journal-remote.service
> 
> So we'd have to figure out a way to support that. Optional, and by
> default we don't use them?

I think they need to be enabled by default in user mode. Optionally disabled
and/or system wide installed.

> 
>> For system systemd services, should they be part of the RPM/deb package?
> 
> Indeed this is what makes this complex.
> 
> Think about the bug reports. What is the path of least bugs. That is
> what we'd need to settle with I think to move forward. In theory this
> will be nixos and each distro's version of libvirt, and no exta systemd
> stuff or mirroring. The user would have to set that up themselves if
> they want it?

Then we can have multiple kdevops packages like:

kdevops
kdevops-systemd (all)
or:
kdevops-systemd-mirror
kdevops-systemd-timesyncd
...

Where the package manager recommends to install them but not force the users
to do it so. I think for systemd user services in ~/.config/systemd/user we can
keep them inside Ansible/kdevops playbook.

> 
>>> What about ssh configurations do we need to review? Perhaps to not
>>> require passwordless sudo?
>>
>> I also meant sandboxing. I think Chuck prefers using ssh <kdevops-vm> instead of
>> ssh -F <kdevops-vm-config> <kdevops-vm>, which makes sense. But some users may
>> not want kdevops settings scattered across their system.
>>
>> What about a kdevopsctl thing? e.g. kdevopsctl ssh vm.
> 
> We already use a custom ssh config which we include.
> Is that really so terrible?

Absolutely not!

I think is hard to have idempotency on files that the user may update themselves
like the .ssh/config. We already have support for it and never fail to me:

cat ~/.ssh/config
# Automatically added by kdevops
# kdevops_version: 5.0.2-g5258e41
Include ~/.ssh/config_kdevops_*

> I think such a thing could optional.

Agreed. That's all.

> Or is it just that cloud needs to move over to use that too?

Not AFAIK.

> 
>   Luis

  reply	other threads:[~2025-08-25 12:16 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-14 19:09 [RFC PATCH] storage-subsystem: correct playbooks for Fedora distro Viacheslav Dubeyko
2025-08-15  0:35 ` Luis Chamberlain
2025-08-15 13:48   ` Chuck Lever
2025-08-15  8:47 ` Daniel Gomez
2025-08-15 11:18   ` Daniel Gomez
2025-08-15 17:28   ` Viacheslav Dubeyko
2025-08-15 17:40     ` Chuck Lever
2025-08-15 18:16       ` Viacheslav Dubeyko
2025-08-15 18:29         ` Chuck Lever
2025-08-15 18:39           ` Viacheslav Dubeyko
2025-08-15 18:46             ` Chuck Lever
2025-08-22  2:18           ` Luis Chamberlain
2025-08-22  8:34             ` Daniel Gomez
2025-08-22 13:50               ` Chuck Lever
2025-08-22 23:14                 ` Luis Chamberlain
2025-08-23 12:33                   ` Daniel Gomez
2025-08-23 21:21                     ` Luis Chamberlain
2025-08-25 12:16                       ` Daniel Gomez [this message]
2025-08-27 21:21                         ` Luis Chamberlain
2025-08-24 17:00                     ` Chuck Lever
2025-08-25  7:21                       ` Luis Chamberlain

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=30cd0793-18ce-435b-a6c1-94ddad1ea6ed@kernel.org \
    --to=da.gomez@kernel.org \
    --cc=Slava.Dubeyko@ibm.com \
    --cc=chuck.lever@oracle.com \
    --cc=kdevops@lists.linux.dev \
    --cc=mcgrof@kernel.org \
    --cc=slava@dubeyko.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