Add support for virtual uc on first node#175
Add support for virtual uc on first node#175asyedham wants to merge 3 commits intoredhat-performance:masterfrom
Conversation
| - name: set undercloud hostname | ||
| set_fact: | ||
| undercloud_hostname: "{{ undercloud_host }}" | ||
| instack_firstnode: "{{ (instackenv_content.nodes[0].pm_addr | replace('mgmt-','') | replace('-drac', '')) }}" |
There was a problem hiding this comment.
we need to give support for other labs also, since our CI is not running on these labs
There was a problem hiding this comment.
we need to give support for other labs also, since our CI is not running on these labs
@masco can you please share the instackenv for those machines . I will add a task for it
|
@masco have added the option to specify the virtual undercloud. Could you please review |
group_vars/all.yml
Outdated
| #This enables user to have a undercloud in VM | ||
| #NOTE: now the vm is created in the ansible host, so | ||
| #do not enable this while running from your desktop | ||
| virtual_uc_hostname: redhat.com |
There was a problem hiding this comment.
the above comment is for the below variable, so move this to one line below
There was a problem hiding this comment.
and make this variable optional, if users using scale or aliase lab, we no need to force the user to set this one.
|
@masco Could you please have look. If this needs any further changes as I'm testing it |
| - name: Clean network interfaces | ||
| shell: | | ||
| /root/clean-interfaces.sh --nuke | ||
| changed_when: false |
There was a problem hiding this comment.
Will it throw error if clean-interfaces.sh not present on the hypervisor, for example, CI machine?
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
3 participants
fixes
#173
#174