summaryrefslogtreecommitdiffstats
path: root/playbooks/openstack
Commit message (Collapse)AuthorAgeFilesLines
* flake cleanupTzu-Mainn Chen2018-01-291-0/+3
|
* code simplification and lint cleanupTzu-Mainn Chen2018-01-291-20/+31
|
* add cinder mountpoint to inventoryTzu-Mainn Chen2018-01-271-1/+15
|
* Merge pull request #6817 from mtnbikenc/fix-1506750OpenShift Merge Robot2018-01-231-1/+1
|\ | | | | | | | | | | | | Automatic merge from submit-queue. [1506750] Ensure proper hostname check override Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1506750
| * [1506750] Ensure proper hostname check overrideRussell Teague2018-01-221-1/+1
| |
* | Merge pull request #6504 from mbruzek/mbruzek-openshift-openstackRussell Teague2018-01-231-6/+8
|\ \ | | | | | | The provision_install file ends in yml not yaml! Ansible requirement …
| * \ Merge branch 'master' into mbruzek-openshift-openstackMatt Bruzek2018-01-188-28/+172
| |\ \
| * | | The provision_install file ends in yml not yaml! Ansible requirement ↵Matt Bruzek2017-12-151-7/+9
| | | | | | | | | | | | | | | | clarification.
* | | | Install base_packages earlierMichael Gugino2018-01-221-2/+2
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, openshift_facts requires pyyaml to be installed. This package is installed via init/base_packages.yml, which is currently called after init/facts.yml. This results in a situation where installs will fail due to missing python dependency. This commit splits init/facts.yml into two, and allows base_packages.yml to be run before the openshift_facts.py plugin is executed.
* | | Fix OpenStack readmeTomas Sedovic2018-01-191-1/+1
| | | | | | | | | | | | | | | | | | The ansible-playbook command in the OpenStack readme is missing a trailing backslash after the dynamic inventory. This prevents being able to copy/paste the command into the terminal and just run it as is.
* | | Drop the testing repo var from openstack readmeTomas Sedovic2018-01-181-5/+1
| |/ |/| | | | | | | | | This was introduced as a temporary fix for the origin+centos deployments. It is no longer necessary and it's confusing to newcomers.
* | Document the inventory changeTomas Sedovic2018-01-161-1/+9
| |
* | Move the OpenStack dynamic inventory from sampleTomas Sedovic2018-01-161-0/+0
| | | | | | | | | | | | | | | | | | | | Keeping the dynamic inventory in the `sample-inventory` alongside the `group_vars` poses a problem: when we update the inventory, our users won't get the latest version. The dynamic inventory should be independent of the OpenShift or OpenStack configuration and the users should be able to either pass it explicitly or create a symlink if they want.
* | Adjust openstack provider dependencies versionsBogdan Dobrelya2018-01-151-4/+3
| | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* | Fix openstack provider playbook name in docsBogdan Dobrelya2018-01-151-1/+1
| | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* | Merge pull request #6695 from mbruzek/openstack_md_fixesOpenShift Merge Robot2018-01-121-7/+7
|\ \ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Spelling and grammar changes to the advanced-configuration.md file. I noticed some spelling errors when trying to read the OpenStack `advanced_configuration.md` file so I wanted to contribute the fixed spelling.
| * | Spelling and grammar changes to the advanced-configuration.md file.Matt Bruzek2018-01-101-7/+7
| | |
* | | Fix typo in the advanced config docsTomas Sedovic2018-01-101-1/+1
| | |
* | | Write guide on setting up PVs with CinderTomas Sedovic2018-01-101-0/+106
| | |
* | | Allow using server names in openstack dynamic invTomas Sedovic2018-01-101-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | When deploying on OpenStack with internal DNS configured, this will set `openshift_hostname` to the Nova server name instead of its IP address. Without those two matching, the OpenStack cloud provider configuration will fail and the OpenShift nodes will not start.
* | | Specify the Cinder version in the inventoryTomas Sedovic2018-01-101-0/+1
|/ / | | | | | | | | | | | | As described in[1], OpenShift currently only works with Block Storage API v2 and the version autodetection is failing to figure that out. [1]: https://github.com/openshift/openshift-docs/issues/5730
* | Merge pull request #6580 from tomassedovic/openstack-fixesOpenShift Merge Robot2018-01-083-2/+10
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Openstack fixes This includes a few fixes for the OpenStack provider. It should fix #6555 and possibly also #6560.
| * | Import prerequisites.yml for OpenStackTomas Sedovic2018-01-021-0/+3
| | |
| * | Return a openshift_node_labels as a dictTomas Sedovic2018-01-021-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | The OpenStack dynamic inventory was setting the `openshift_node_labels` value as a string which causes a failure with the `lib_utils_oo_dict_to_keqv_list` filter. Fixes #6555
| * | Fix yaml syntax error in the sample inventoryTomas Sedovic2018-01-021-2/+2
| | |
* | | Merge pull request #6627 from sdodson/import_roleScott Dodson2018-01-052-8/+8
|\ \ \ | | | | | | | | Migrate to import_role for static role inclusion
| * | | Migrate to import_role for static role inclusionScott Dodson2018-01-052-8/+8
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Ansible 2.2, the include_role directive came into existence as a Tech Preview. It is still a Tech Preview through Ansible 2.4 (and in current devel branch), but with a noteable change. The default behavior switched from static: true to static: false because that functionality moved to the newly introduced import_role directive (in order to stay consistent with include* being dynamic in nature and `import* being static in nature). The dynamic include is considerably more memory intensive as it will dynamically create a role import for every host in the inventory list to be used. (Also worth noting, there is at the time of this writing an object allocation inefficiency in the dynamic include that can in certain situations amplify this effect considerably) This change is meant to mitigate the pressure on memory for the Ansible control host. We need to evaluate where it makes sense to dynamically include roles and revert back to dynamic inclusion if and where it makes sense to do so.
* | | Merge pull request #6532 from jmencak/heat_stack-cnsOpenShift Merge Robot2018-01-052-1/+13
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Automatic merge from submit-queue. OpenStack provisioning -- support cns. Initial support for CNS nodes during OpenShift on OpenStack provisioning.
| * | OpenStack provisioning -- support cns.Jiri Mencak2018-01-022-1/+13
| | |
* | | set repos after registration: convert to match task -> import_role model.Mark Lamourine2017-12-181-3/+11
|/ /
* / Fix rhel_subscribe booleanMichael Gugino2017-12-151-2/+2
|/ | | | | | | This commit fixes a variable name to it's correct name used in a when condition. Also makes use of 'is defined' consistent.
* Allow 2 sets of hostnames for openstack providerBogdan Dobrelya2017-12-111-5/+16
| | | | | | | | | | | | | Support private/public hostnames suffixes for DNS records. Real hostnames, Inventory variables, Nova servers and ansible hostnames will ignore the custom suffixes. Those are only for nsupdates sent to external DNS servers. Related change: add openshift_openstack_public_dns_domain to the role defaults to not rely on the group vars example only. Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com> Co-authored-by: Tomas Sedovic <tsedovic@redhat.com>
* Cleanup byo referencesRussell Teague2017-12-082-3/+3
|
* Include Deprecation: Convert to import_playbookRussell Teague2017-12-073-6/+6
|
* Merge pull request #6361 from mtnbikenc/consolidate-deploy-clusterScott Dodson2017-12-071-5/+2
|\ | | | | Playbook Consolidation - byo/config.yml
| * Playbook Consolidation - byo/config.ymlRussell Teague2017-12-061-5/+2
| |
* | Merge pull request #6310 from tomassedovic/openstack-rhel-subscriptionMichael Gugino2017-12-072-21/+14
|\ \ | |/ |/| Optionally subscribe OpenStack RHEL nodes
| * Remove shell environment lookupTomas Sedovic2017-12-011-2/+2
| |
| * Optionally subscribe OpenStack RHEL nodesTomas Sedovic2017-11-292-21/+14
| |
* | Merge pull request #6340 from bogdando/openstack_cleanup_dnsScott Dodson2017-12-062-64/+6
|\ \ | | | | | | Cleanup not required dns bits
| * | Fix and cleanup not required dns bitsBogdan Dobrelya2017-12-052-64/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Do not manage hostnames for openstack cloud provider, let cloud-init to do its job. * Make python-dns / dnspython dependency check conditional. * Drop not used dns node flavor and image. * Do not manage dns nodes and sec groups in heat stacks. * Keep supporting dynamic updates for private DNS records, yet only limited to an external DNS managed elsewhere (not deployed by the openshift_openstack provider). So users may still benefit from this feature, sending nsupdates to private and public servers as they want it. * Fix openstack default for external nsupdate keys. It should be undefined by default as the dns-populate logic is based on that. * Fix dns records generation for openstack provider's populate-dns * Update docs
* | | Remove all references to pacemaker (pcs, pcsd) and ↵Andrew Butcher2017-12-052-2/+0
|/ / | | | | | | | | | | | | openshift.master.cluster_method. With pacemaker removed there is no longer a need for openshift.master.cluster_method. We only have one option.
* | Merge pull request #6341 from bogdando/openstack_cleanup_staticMichael Gugino2017-12-052-86/+0
|\ \ | | | | | | Cleanup unused openstack provider code
| * | Cleanup unused openstack provider codeBogdan Dobrelya2017-12-042-86/+0
| | | | | | | | | | | | | | | | | | | | | Remove references for bastion, ssh UI tunnek and static inventory. Update docs. Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* | | Document testing repos for dev purposesBogdan Dobrelya2017-12-052-2/+8
| | | | | | | | | | | | Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* | | Use openshift testing repos for openstackBogdan Dobrelya2017-12-042-1/+2
|/ / | | | | | | | | | | | | | | Use testing openshift repos to sidestep packaging issues for 3.6 clients being used in 3.7 deployments or something. Poke host requirements Centos 7.4 as 7.3 can't see that testing repo. Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
* | Update the doc textTomas Sedovic2017-11-292-5/+6
| |
* | Use IP addresses for OpenStack nodesTomas Sedovic2017-11-285-78/+44
|/ | | | | | | | | | | | | Unlike other cloud providers, OpenStack VMs are not able to resolve each other by their names. If you try to run the playbooks against nodes without a pre-created /etc/hosts or a DNS that provides the hostname/ip resolution, it will fail. By setting the `openshift_hostname` variable to each node's IP address, we're able to deploy a functional cluster without running a custom DNS. It is still possible to provide an external server with nsupdate keys and have it be populated, but that is no longer a hard requirement.
* Fix openstack initRussell Teague2017-11-162-8/+3
|
* Namespace the docker volumesTomas Sedovic2017-11-071-7/+7
|