Skip to content

Commit 9267ba7

Browse files
committed
add sebastian general comments
1 parent 5f8df39 commit 9267ba7

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

chapter_11_ansible.asciidoc

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,23 @@ A few small things:
7575
billed needlessly.
7676
////
7777

78+
////
79+
SEBASTIAN overall notes
80+
All in all, I am not very fond of the current shape of this chapter.
81+
82+
The summary is great and the choice of technologies is the best I can imagine.
83+
84+
However, I got lost several times while reading through the chapter.
85+
There are too many open loops. For example, SSH is mentioned but then we jump into all other technologies without seeing what SSH is and how it will play with the rest.
86+
87+
mentioning of too many technologies (e.g. Puppet/Chef - IMHO not necessary in 2024).
88+
89+
I think (my opinion) the chapter needs reorganizing so that readers can more quickly see a given piece of tech in action. I'd cut some content.
90+
91+
If you like some more specific suggestions, I can spend more time and provide them.
92+
93+
Also, it's mentioned that the server will be provisioned manually without automation, but then we get ansible infra/ansible-provision.yaml. I must say I'm not following when provisioning starts and where it ends. In my book (figuratively speaking), installing docker falls under the definition of provisioning (which was meant NOT to be automated) while the remaining steps, like exporting and reimporting docker image are something different (deployment?).
94+
////
7895

7996
.🚧 Warning, chapter under construction 🚧
8097
*******************************************************************************

0 commit comments

Comments
 (0)