Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

working extra_vars.yml for Smart Management workshop #1824

Open
criscofats opened this issue Sep 30, 2022 · 3 comments
Open

working extra_vars.yml for Smart Management workshop #1824

criscofats opened this issue Sep 30, 2022 · 3 comments
Assignees

Comments

@criscofats
Copy link
Contributor

Area

Other

Proposed Changes

I attached a sanitized extra_vars.yml file that allows for the successful provisioning of the Smart Management workshop. There are many variables missing from the example extra_vars.yml that is provided on https://github.com/ansible/workshops/tree/main/provisioner
extra_vars.txt

@IPvSean
Copy link
Contributor

IPvSean commented Oct 4, 2022

@heatmiser can you provide this?

@rclements-redhat
Copy link
Contributor

  1. What is the "demo: all" used for? I didn't have to use that in my file.
  2. the key 'ec_name_prefix' value is not sanitized. It says 'test-demo311f'
  3. provided_sha_value: ea28..8c01 is correct. I had to make this adjustment as well

@rclements-redhat
Copy link
Contributor

rclements-redhat commented Oct 14, 2022

  1. ee_images key and the associated list value is provided in the devel branch https://github.com/ansible/workshops/blob/devel/roles/control_node/defaults/main.yml#L19
  2. ee_default_image key and the associate value is provided in the devel branch https://github.com/ansible/workshops/blob/devel/roles/control_node/defaults/main.yml#L25

We could still include them in extra_vars for the override, but they could be commented out, with a comment referencing https://github.com/ansible/workshops/blob/devel/roles/control_node/defaults/main.yml to let people know where the main keys are defined.

Lastly, we have a bug #1794 that will update the newest images to version 22 instead of 21.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants