Skip to content

Commit

Permalink
new-linode: clean up outdated comments
Browse files Browse the repository at this point in the history
  • Loading branch information
skyfaller committed May 4, 2022
1 parent ac478e2 commit 30b9d7f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
7 changes: 1 addition & 6 deletions roles/new-linode_harden/tasks/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,6 @@
- force change password
# note that handlers are executed in the order they are written down, NOT in the order they are notified
# https://stackoverflow.com/questions/35130051/order-of-notify-handlers
# TODO: make sure handlers execute in the correct order, by e.g. calling the 2nd handler from the first handler
# don't rely on the order in which handlers are written down, which seems fragile
# https://github.com/sunrisemovement/devops/issues/7

- name: Install PAM on Alpine Linux
package:
Expand All @@ -32,6 +29,4 @@
- restart ssh
- force change password

# Before you run any other Ansible plays, you'll have to ssh in to change your password.

# Also, consider switching to Linode's latest kernel to improve security.
# Switch to Linode's latest kernel to improve security and ensure all functions work.
2 changes: 0 additions & 2 deletions roles/new-linode_initiate-users/tasks/main.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
# tasks on remote server once it has been created, as root #
# TODO: DISABLE RUNNING THESE TASKS IF SERVER ALREADY EXISTED (how tho?)
# https://github.com/sunrisemovement/devops/issues/9

- name: Don't do anything until server is reachable via SSH
wait_for_connection:
Expand Down

0 comments on commit 30b9d7f

Please sign in to comment.