-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update test-intertenant for 3.2 #3
base: cloudwatt
Are you sure you want to change the base?
Conversation
target_num2: 2 | ||
asn: "217.1.0.0" | ||
vars_files: | ||
- ansible-vars.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Must not be hard-coded. Need to switch easily between environments
target_num2: 2 | ||
asn: "217.1.0.0" | ||
vars_files: | ||
- ansible-vars.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The stuff about the ansible-vars.yml is still not solved. You must rely on *.tfvars file to swicth easily from one environment to another.
Think about how you would do it when you have to test quicly between lab and int
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/*.tfvars/ansible vars file/g
Like in the orchestration repo
cloudwatt/test_snat_lbaas/input.tf
Outdated
@@ -6,11 +6,11 @@ variable "key_pair" {} | |||
|
|||
# Backend |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Modifications here are not related to inter-tenant stuff 🤔
This modification should not be tracked
Create route-target between tenants by starting the script setup_inter_tenant_x-x.yml in bastion of the enviroment (sudo password is asked): | ||
there are 2 files: setup_inter_tenant_2-21.yml for contrail-2.21 and setup_inter_tenant_3-2.yml for contrail-3.2 | ||
|
||
First, create the file ansible-vars.yml that seems like: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be readapted after your change on vars file
|
||
``` | ||
[bastion] | ||
bastion_server_A ansible_host=xx.xx.xx.182 ansible_user=cloud ansible_ssh_private_key_file=/path/private-key ansible_ssh_common_args='-o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/cloud/<SSH_USER>/gc
s//path/private-key/<SSH_KEY>/gc
The user will follow that it is the same information like in terrafrom-inventory (line 37)
70e2782
to
7f08a27
Compare
7f08a27
to
9ca671b
Compare
No description provided.