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

Update test-intertenant for 3.2 #3

Open
wants to merge 1 commit into
base: cloudwatt
Choose a base branch
from

Conversation

k-ayache
Copy link

No description provided.

target_num2: 2
asn: "217.1.0.0"
vars_files:
- ansible-vars.yml
Copy link
Owner

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
Copy link
Owner

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

Copy link
Owner

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

@@ -6,11 +6,11 @@ variable "key_pair" {}

# Backend
Copy link
Owner

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:
Copy link
Owner

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'
Copy link
Owner

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)

@k-ayache k-ayache force-pushed the inter-tenant-3.2 branch 2 times, most recently from 70e2782 to 7f08a27 Compare March 22, 2019 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants