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

Enhance inventory_cloud j2 template to support deployment on openstack vs local, hetzner #48

Closed
cmoulliard opened this issue Apr 19, 2018 · 1 comment · Fixed by #50
Closed
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@cmoulliard
Copy link
Member

Enhance inventory_cloud j2 template to support deployment on openstack vs local, hetzner.

A few modifications are required :

a) openshift_hostname
openshift_hostname=node_name -> openshift_hostname=ip

Should be good to see if we can also define a hostname for local, hetzner deployments.
If we use the Centos ISO created, then hostname command executed on the terminal of the vm returns : cloud and for hetzner -> CentOS-74-64-minimal
We should perhaps specify it as a name

b) Node, master, etcd

[masters]
10.8.250.104 openshift_public_hostname=10.8.250.104 openshift_hostname=172.16.195.12
Change To : -->
192.168.99.50 openshift_public_hostname=192.168.99.50 openshift_ip=192.168.99.50

[etcd]
10.8.250.104
Change To : -->
192.168.99.50 openshift_ip=192.168.99.50

[nodes]
10.8.250.104 openshift_node_labels="{'region':'infra','zone':'default', 'node-role.kubernetes.io/compute': 'true'}" \
   openshift_public_hostname=10.8.250.104 \
   openshift_hostname=172.16.195.12
   
Change To : -->

192.168.99.50 openshift_node_labels="{'region':'infra','zone':'default', 'node-role.kubernetes.io/compute': 'true'}" \
   openshift_public_hostname=192.168.99.50 \
   openshift_ip=192.168.99.50

See diff file

screenshot 2018-04-19 14 04 23

@cmoulliard cmoulliard added the enhancement New feature or request label Apr 19, 2018
@cmoulliard cmoulliard added this to the 1.0 milestone Apr 19, 2018
@geoand
Copy link
Member

geoand commented Apr 19, 2018

I will update the code introduced in #47 with this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants