Ansible role in JetBrew to provision the EDPM nodes#12
Ansible role in JetBrew to provision the EDPM nodes#12smandaRH wants to merge 11 commits intoredhat-performance:mainfrom
Conversation
smandaRH
force-pushed
the
provision
branch
2 times, most recently
from
d419eec to
f212c7c
Compare
| # Starting node index for compute node provisioning (0-indexed) | ||
| # Default is 4 (starts from 5th node: 1 bastion + 3 OCP nodes = index 4) | ||
| # Set this in group_vars/all.yml to start provisioning from a different node | ||
| compute_start_node_index: 4 |
There was a problem hiding this comment.
This var is defined in group_vars/all.sample.yml alsoo..
if this is intended, change the var name to "start_node", keep default value to 5 and change the comments as necessary.
There was a problem hiding this comment.
still i can see the same name in group_vars/all.yml
smandaRH
force-pushed
the
provision
branch
2 times, most recently
from
7c14258 to
d90eed8
Compare
smandaRH
force-pushed
the
provision
branch
2 times, most recently
from
3d5a1f5 to
b4b987e
Compare
smandaRH
force-pushed
the
provision
branch
2 times, most recently
from
1ae568f to
9f5938f
Compare
edpm provisioning role adding to jetbrew
ansible/edpm_provisioning.yml
Outdated
| all_nodes: "{{ (ocp_inventory.content | from_json).nodes }}" | ||
| total_num_nodes: "{{ (ocp_inventory.content | from_json).nodes | length }}" | ||
|
|
||
| - name: Set effective start index (from start or fallback variable) |
There was a problem hiding this comment.
As discussed this tasks should get triggered only when user has provided negative index for provision_start_node_index
There was a problem hiding this comment.
addressed the comment
ansible/roles/foreman/tasks/main.yml
Outdated
| ansible.builtin.set_fact: | ||
| foreman_user: "{{ all_nodes[0].pm_user }}" | ||
| foreman_password: "{{ all_nodes[0].pm_password }}" | ||
| supermicro_regex: '.*(1029u|1029p|5039ms|6018r|6029p|6029r|6048p|6048r|6049p).*' |
There was a problem hiding this comment.
support incase new machine type is added to lab. Provide a option to override from group_vars/all.yml or update README
There was a problem hiding this comment.
updated supermicro_regex to default to role.
4 participants
No description provided.