Skip to content

Commit 4eef9c9

Browse files
committed
Merge remote-tracking branch 'origin/main'
2 parents c6a6da7 + 68ce94b commit 4eef9c9

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

chapter_11_ansible.asciidoc

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,12 @@ It's just as easy to fall into the Refactoring-Cat trap on the server!
249249
250250
////
251251

252+
// JAN: It feels 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.
253+
// I ended up executing the initial Ansible command when I realized I should've already spun up the server and set DNS records.
254+
// 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"
255+
// JAN: It would also make sense to tell the reader to point both domains to the same server
252256

257+
// 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
253258

254259
=== Configuring Domains for Staging and Live
255260

@@ -336,6 +341,8 @@ The "hate" part is that the actual syntax is surprisingly fiddly to get right:
336341
the difference between lists and key/value maps is subtle and I can never quite remember it honestly.]
337342
for.
338343

344+
// JAN: It would be worth mentioning what to do to be able to execute this ansible command. I haven't noticed anything about adding a record to authorized_keys
345+
339346
// DAVID: I didn't know if this was an example or a file to create.
340347
// I think it's the latter, it would be good to make that clear.
341348

@@ -369,6 +376,10 @@ for.
369376
----
370377
====
371378

379+
// JAN: I'm stuck here. Docker is not installed on EC2 instance with Ubuntu. Googling it, it seems like this is some outdated Docker installation process
380+
The one described here is much closer to Docker's installation guide: https://www.digitalocean.com/community/tutorials/how-to-use-ansible-to-install-and-set-up-docker-on-ubuntu-22-04
381+
382+
372383
<1> An Ansible playbook is a series of "tasks"
373384
(so in that sense it's still quite sequential and procedural),
374385
but the individual tasks themselves are quite declarative.

0 commit comments

Comments
 (0)