Ewoud Kohl van Wijngaarden

Results 1159 comments of Ewoud Kohl van Wijngaarden

Oh, I just noticed: we did build it for EL8 but not for EL9.

> Now that I read your comment and thought about it in this context, this might be caused by us disabling inteface updates from facts? That sounds very plausible. >...

Wouldn't it be better if we linked to the exact template that failed? We may need to restructure the errors for that but that would be better to debug. Bonus...

> Isn't this a different story? This is about a way to copy and paste error messages. Perhaps. As a user I'm surprised they're copyable now. Looks like an underdeveloped...

@Thorben-D is the screenshot in the opening message still up to date with the current code?

> * [Fixes #37433 - Set indentation when calling subscription_manager_setup #10200](https://github.com/theforeman/foreman/pull/10200) fixed the indentation problem, but exposed a lingering problem with `save_to_file`, since this helper method does not work well...

I had a look at the default `cloud.cfg` in EL9: ```yaml # The modules that run in the 'init' stage cloud_init_modules: - migrator - seed_random - bootcmd - write_files -...

I've just rebased this to resolve the conflicts because I needed this for something. Not sure if I have a direct desire to complete this.

In ca34010f2d34205493489a14f42d1e2334cc8077 @ShimShtein rewrote this I'd expect that to be used. This whole snippet is supposed to be unused and I think keeping it was an omission.

I'm not sure that's a big deal. If you can become root, you can then run things as any user anyway. Security wise I don't think there's a difference.