Skip to content

Commit a71d1d0

Browse files
authored
Review chapter_11_ansible.asciidoc
1 parent e8abda8 commit a71d1d0

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

chapter_11_ansible.asciidoc

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,11 @@ It's just as easy to fall into the Refactoring-Cat trap on the server!
206206
207207
////
208208

209+
// JAN: It feel a bit like something is missing here. You're talking about servers and DNS records. Yet, there's no real call to action for a reader.
210+
// I ended up executing the initial Ansible command when I realized I should've already spun up the server and set DNS records.
211+
// I'd suggest being more specific about what should I do as a reader. No need to go step by step. Just to clearly state: "Spin up your publicly accessible server", "Now set DNS A records that point to your server"
209212

213+
// JAN: It would also be beneficial to state at least some range of server size (e.g., 2GB of memory). e.g., the smallest EC2 instances can have quite some troubles when running Django in Docker
210214

211215
=== Configuring Domains for Staging and Live
212216

0 commit comments

Comments
 (0)