{"action":"created","issue":{"url":"https://api.github.com/repos/chef-training/chef-essentials/issues/127","labels_url":"https://api.github.com/repos/chef-training/chef-essentials/issues/127/labels{/name}","comments_url":"https://api.github.com/repos/chef-training/chef-essentials/issues/127/comments","events_url":"https://api.github.com/repos/chef-training/chef-essentials/issues/127/events","html_url":"https://github.com/chef-training/chef-essentials/issues/127","id":122133465,"number":127,"title":"Feedback from a Recent Delivery of the Content","user":{"login":"burtlo","id":244426,"avatar_url":"https://avatars.githubusercontent.com/u/244426?v=3","gravatar_id":"","url":"https://api.github.com/users/burtlo","html_url":"https://github.com/burtlo","followers_url":"https://api.github.com/users/burtlo/followers","following_url":"https://api.github.com/users/burtlo/following{/other_user}","gists_url":"https://api.github.com/users/burtlo/gists{/gist_id}","starred_url":"https://api.github.com/users/burtlo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/burtlo/subscriptions","organizations_url":"https://api.github.com/users/burtlo/orgs","repos_url":"https://api.github.com/users/burtlo/repos","events_url":"https://api.github.com/users/burtlo/events{/privacy}","received_events_url":"https://api.github.com/users/burtlo/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-12-14T21:18:41Z","updated_at":"2015-12-14T23:00:17Z","closed_at":"2015-12-14T22:51:46Z","body":"Shared by Jessica DeVita and Jody Wolfborn:\r\n\r\nEssentials Slide Notes\r\n\r\n* [x]\tSlide 1-8, remove Backspace\r\n* [x]\tSlide 2-14 “This company” should be “This computer”\r\n* [x]\tSlide 2-16 doesn’t say which type of example it is: “GE” and “Labs” not on every slide? \r\n* [ ]\tSlide \"Desired Configuration”, change “environs” to “environments”?\r\n* [ ]\tSlide 2-21, combine with previous slides? redundant\r\n* [ ]\tSlide 2-40, add “package ‘vim’” and “package ‘emacs’”\r\n* [ ]\tSlide 3-8, why is there a line break between “package ‘emacs’” and “package ‘tree’”?\r\n* [x]\tSlide 3-17, why is “recipes” highlighted, when it’s talking about the README?\r\n* [x]\tSlide 3-22, why are there two README.md files? Also a .gitignore file?\r\n* [ ]\tSlide 3-15 to 3-25 - Discuss weirdness on slides\r\n* [ ]\tSlide 3-35, add “Use chef generate recipe” to the “create a recipe” portion\r\n* [x]\tSlide 3-37, confusing that it says “modify the Apache Recipe” rather than “server.rb” recipe\r\n* [x]\tSlides 3-39 through 3-41 have the wrong slide numbers\r\n* [x]\tSlide 3-41, add “you” to “How often would *you* commit”\r\n* [x]\tSlide 4-12, add “s” to final “Cookbook” (also, maybe talk a little about the syntax “mv workstation cookbook”?)\r\n* [x]\tSlide 4-14\r\n* [x]\tSlide 4-20, add # before “reserved”\r\n* [x]\tSlide 5-25, explicitly say “edit kitchen.yml”?\r\n* [x]\tSlide 5-29, 5-30, 5-39 and 5-40, either add more info about the “[INSTANCE…]” to talk track or remove from the slide\r\n* [x]\tSlide 5-36, update to be more clear on what it’s asking, etc.?\r\n* [ ]\tSlide 5-40, specify which cookbooks you should be kitchen destroy-ing\r\n* [x]\tSlide 5-42, “software” instead of “softwares”\r\n* [x]\tSlide 5-44, remove “s” from “Test Helper Tools”\r\n* [x]\tSlide 5-55, update this to remove the sentence fragments, change “Event” to “Even”\r\n* [x]\tSlide 5-60, add the rest of the packages that are installed in the Workstation::setup (vim, emacs, nano, and /etc/motd?)\r\n* [x]\tSlide 5-61 and 5-62, make sure it’s clear that they need to be in the workstation directory\r\n* [x]\tSlide 5-25 and 5-26, implies that everything below “run_list” should be deleted; talk-track should make it clear that the “workstation::default” is still part of the run list\r\n* [x]\tSlide 6-35, clarify whether you’re supposed to delete the various package commands?\r\n* [ ]\tSlide 6-36, how to test that the variable is being passed appropriately?\r\n* [x]\tSlide 6-39, “Lets” should be “Let’s”\r\n* [x]\tSlide 7-4, why is “httpd” changed to “apache2”?\r\n* [x]\tSlide 7-8, why are we “escaping our feelings”?\r\n* [x]\tSlide 7-16, is this a demo or an example?\r\n* [x]\tSlide 7-21 to 7-23, why are we going over this information again??? We’ve already used “chef generate”\r\n* [ ]\tSlide 7-36, can pass any Ruby into Angry Squid? Can you use Angry Squid in Recipes? “Pretty much, yup”,\r\n* [x]\tSlide 7-42, add “Resource” after “Template” (otherwise confusion as to whether we’re actually converting the file to a template)\r\n* [x]\tSlide 7-35, make it more clear that we’re supposed to update the index.html.erb to have the “hello, world” text from the server.rb\r\n* [ ]\tSlide 7-50, call out explicitly what the -s is trying to do, and find out why there’s not a -s in the apache template creation\r\n* [ ]\tSlide 7-50, can we change the source from the actual file to be the setup.rb recipe instead?\r\n* [ ]\tSlide deck for Chapter 8 is not available in Word format?\r\n* [ ]\tSlide 9-15, Why talk about conEmu?\r\n* [ ]\tSlide 9-29, “knife client list” needs to be run from chef-repo directory\r\n* [x]\tSlide 10-37, is there an unnecessary colon between “recipe” and the bracket?\r\n* [ ]\tSlide 10-25, we taught them how to use variables, these slides shouldn’t be hard-coded\r\n* [ ]\tSlide 10-32, should call out explicitly that versions don’t match what they’ve coded on their haproxy settings\r\n* [ ]\tSlide 11-12, widen the page in order to increase font size to something readable and make this variables rather than hard-coded\r\n* [x]\tSlide 11-12, ‘ipaddress' should be an IP address, rather than the hostnames\r\n* [x]\tSlide 12-8, setting up the starter kit on Windows servers automatically creates a roles directory within the chef-repo directory; add “IF NEEDED” to title\r\n* [x]\tSlide 12-9, why do we call this “proxy\" instead of “load_balancer\"?\r\n* [x]\tSlide 12-14, put all the text on the same line?\r\n* [ ]\tSlide 12-27, talk track doesn’t explain nested roles or what the right answer is\r\n* [x]\tSlide 13-4, typo on talk-track “add them TO a list”\r\n* [ ]\tSlide 13-5, node numbers are incorrect; seems to indicate way more machines than we provide trainees\r\n* [x]\tSlide 13-11, typo in talk-track “we ask about A specific attribute”\r\n* [x]\tSlide 13-20, provide a gist with final recipe\r\n* [x]\tSlide 13-29, talk track doesn’t explain what’s meant by “terminate a node without removing it”\r\n* [x]\tSlide 14-2, should say “create an acceptance and a production environment”\r\n* [x]\tSlide 14-4 and 5, typo in box refers to Node 8 (node 1&2 to prod and 3 to acceptance)\r\n* [x]\tSlide 14-6 and 7, font is too small\r\n* [x]\tSlide 14-12, “create a new environment file” to be consistent with language on slide for creating an acceptance environment”\r\n* [x]\tSlide 14-16, add highlighting to be consistent with other slides\r\n* [x]\tSlide 14-49, no talk track for these questions\r\n* [x]\tSlide 14-50, typo “what happens when you USE and in a query”; talk track, what example should we provide here?\r\n"},"comment":{"url":"https://api.github.com/repos/chef-training/chef-essentials/issues/comments/164587542","html_url":"https://github.com/chef-training/chef-essentials/issues/127#issuecomment-164587542","issue_url":"https://api.github.com/repos/chef-training/chef-essentials/issues/127","id":164587542,"user":{"login":"burtlo","id":244426,"avatar_url":"https://avatars.githubusercontent.com/u/244426?v=3","gravatar_id":"","url":"https://api.github.com/users/burtlo","html_url":"https://github.com/burtlo","followers_url":"https://api.github.com/users/burtlo/followers","following_url":"https://api.github.com/users/burtlo/following{/other_user}","gists_url":"https://api.github.com/users/burtlo/gists{/gist_id}","starred_url":"https://api.github.com/users/burtlo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/burtlo/subscriptions","organizations_url":"https://api.github.com/users/burtlo/orgs","repos_url":"https://api.github.com/users/burtlo/repos","events_url":"https://api.github.com/users/burtlo/events{/privacy}","received_events_url":"https://api.github.com/users/burtlo/received_events","type":"User","site_admin":false},"created_at":"2015-12-14T23:00:17Z","updated_at":"2015-12-14T23:00:17Z","body":"```\r\nSlide 6-36, how to test that the variable is being passed appropriately?\r\n```\r\n\r\nWe aren't really passing variables when we use them within the string using the string interpolation. The best way would be to test that they are present in the file in the integration test. In interest of time we don't go into that it. Sometimes when I teach that content I go into writing that test using the file matchers and talk about using a regular expression with the website [Rubular](http://rubular.com/)."}}