Timothée Ravier

Results 529 comments of Timothée Ravier

From `podman-run(1)`: ``` Note: if the user only has access rights via a group, accessing the device from inside a rootless container will fail. Use the --group-add keep-groups flag to...

> One option is to deprecate `ssh_authorized_keys` and create e.g. `ssh_authorized_keys_source` of type `SSHAuthorizedKeyResource`. But looking at how `Resource` is defined now, it's just the underlying Ignition `Resource` struct with...

> For previous discussion on expanding the types of resources Ignition can fetch, see [coreos/ignition#884](https://github.com/coreos/ignition/issues/884), [coreos/ignition#986](https://github.com/coreos/ignition/pull/986), and [coreos/ignition#1097](https://github.com/coreos/ignition/issues/1097). It would have made sense to support fetching more broadly in the...

What this could potentially look like: ``` systemd: contents_local (object): - path (string): password: users: - ssh_authorized_keys_local (list of objects): - path (string): ```

Took a quick look and this LGTM but I'm not up-to-date on Butane code and logic to do an in-depth review. I'll give it a try soon however to see...

I don't think the discussion about merging config files should impact this PR as far as I understand.

Sorry for the late reply. I really want to have this but I haven't had the time to look at it again. I'll see if we can find someone to...

Something else that might help with reviewing this change is splitting it into two commits, one for systemd and one for ssh keys.

Cross linking an example until we add proper support in Butane/Ignition: https://discussion.fedoraproject.org/t/layering-packages-via-rpm-ostree-as-part-of-provisioning/29862

> * Rather than `crashkernel: 128M`, maybe something like `reserved_mib: 128`? The [full option syntax](https://www.kernel.org/doc/html/latest/admin-guide/kdump/kdump.html?highlight=kdump#extended-crashkernel-syntax) enables setting memory size dependent values and offset but I don't know if that's something...