Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

[Provisioner] Instructor_inventory to closely match student_instance_inventory #335

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

nicholas-chia
Copy link
Contributor

…ce_inventory

This update makes the instructor inventory clearer by explicitly stating username and admin_password to the inventory for each student login.

I usually upload instructor inventory to gist during workshop. Removes ec2-user to avoid any confusion.

Change from "inventroy" to "inventory".
…ce_inventory

This update makes the instructor inventory clearer by explicitly stating username and admin_password to the inventory for each student login.

I usually upload instructor inventory to gist during workshop. Removes ec2-user to avoid any confusion.
This role will email the instructor_inventory.txt when the extra_var email_instructor is set to True. Default is set to False.
This is set to False by default and does not email the instructor_inventory.txt by default.
This will email the instructor_inventory.txt with Sendgrid. Default is set to False. To enable, set email_instructor to "True" in the extra_vars.yml file. This update requires email_instructor role.
sync lightbulb to my fork
Correct irritating typo in task name "inventroy"
@gundalow
Copy link
Collaborator

gundalow commented Dec 3, 2018

@nicholas-chia Looks like you have unrelated commits in here

@gundalow gundalow changed the title Propose updating instructor_inventory to closely match student_instan… [Provisioner] Instructor_inventory to closely match student_instance_inventory Dec 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants