Ewoud Kohl van Wijngaarden

> @ekohl Please, remember that UML is about abstraction. I want to make these diagrams as simplified as possible to make them easy to understand for first-time users. Savvy? For...

I started a review because you explicitly asked for it in https://community.theforeman.org/t/rfc-diagrams-of-provisioning-workflows/39045, even though I was very busy with unblocking the Foreman 3.12 branching that happens this week. Then I...

Sharing my research here. First I did some small investigation where the classes are used. At least for core. For now this is just a list of tasks. I'd like...

I'd even suggest that if you really want to do this (and that's a big if) then we should modify the procedure to avoid the attribute.

Ah, you're right. I'm so glad we'll drop modularity soon when we drop EL8 in 3.13

> If this is enough to resolve the issue then I'd prefer it over changing the puppet modules to expect the keytab to be in a different place, especially if...

@asteflova you mentioned that it's more common to deploy IPA and then link your IPA and AD installations. I think it's worth considering making that the recommended AD integration strategy...

> ... and I think I've just realized why the procedure is there. Spoiler alert: It's kind of my fault and I'll send a PR to fix it soon. 

Another use case: you can implement all 3 procedures in different tabs. One tab for the UI, one for CLI and another for Ansible. asciidoctor-tabs can also be configured to...

> Unfortunately, this Asciidoctor function is not supported downstream. If I recall correctly, the blocker was Pantheon, which could not render the tabs. Maybe things will be different when we...