Slide deck wording does not align with the example that follows
thisdwhitley opened this issue · 2 comments
The text on slide 33 of the ansible-essentials.html deck states:
If a configuration file gets changed notify a service restart task that it needs to run.
But the example on the following slide (34) shows a handler utilized on the task that installs a package.
The example would make more sense in relation to the prior slide if the notify
was in the "copy" task:
- name: latest index.html file is present
copy:
src: files/index.html
dest: /var/www/html/
notify: restart httpd
Additionally, the package
module could be used in place of yum
but that is purely personal preference.
I find that it is easy enough to change the html file, but I have to wonder if that is actually generated from something else and if I should instead update the source of the file..?
@dswhitley: Thanks for that observation.
Slide 33 wasn't meant to be that direct of a connection to what's on 34, but I suppose it could. That example is a copy of one that's in examples/
. I'd prefer not to change that unless necessary since it has repercussions beyond that slide. I'd just assume we rephrase what's on 33 to something like "If a package get's installed or updated notify a service restart task that it needs to run. That connects the two with least amount of changes.
The reveal decks are currently just static HTML. As our content grows and more slides are shared amongst them we may have to move to a (simple) generated model. We're not there yet so feel free to do any pull requests on what you see there.
Not a fan of package
and prefer explicitly naming the package manager we want to use.