ansible/lightbulb

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..?

tima commented

@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.