summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #168 from detiber/client_config_changeThomas Wiest2015-04-233-100/+334
|\ | | | | Fixes for latest osc client config changes
| * Fixes for latest osc client config changesJason DeTiberus2015-04-223-100/+334
|/ | | | - also pylint fixes
* Merge pull request #182 from kwoodson/git_hooksKenny Woodson2015-04-221-1/+1
|\ | | | | Adding pylint binary call.
| * Adding pylint binary call instead of python moduleKenny Woodson2015-04-221-1/+1
| |
* | Merge pull request #181 from kwoodson/git_hooksKenny Woodson2015-04-224-0/+519
|\| | | | | Adding git hooks for jenkins repository validation.
| * Adding commit hooks for jenkinsKenny Woodson2015-04-224-0/+519
|/
* Merge pull request #169 from detiber/gce_iptables_defaultsJhon Honce2015-04-223-45/+46
|\ | | | | fixes to better deal with gce image defaults
| * pylintJason DeTiberus2015-04-211-38/+44
| |
| * fixes to better deal with gce image defaultsJason DeTiberus2015-04-213-8/+3
| | | | | | | | | | | | | | - remove exception if INPUT rules are not found, gce centos-7 image is stripped of default rules - ignore_errors for systemctl mask operation, fails with permission denied on gce centos-7 image.
* | Merge pull request #176 from wshearn/online_varsWesley Hearn2015-04-221-5/+5
|\ \ | | | | | | Update openshift-cluster/vars for online defaults
| * | Update openshift-cluster/vars for online defaultsWesley Hearn2015-04-221-5/+5
| | |
* | | Merge pull request #175 from wshearn/use_ops_docker_mirrorWesley Hearn2015-04-222-0/+12
|\ \ \ | |/ / |/| | Use docker-registry.ops when deploying as online
| * | Use docker-registry.ops when deploying as onlineWesley Hearn2015-04-222-0/+12
|/ /
* | Merge pull request #174 from wshearn/fix_reposWesley Hearn2015-04-223-44/+1
|\ \ | | | | | | Remove duplicated repos and fix enterprise-v3 repo
| * | Remove duplicated repos and fix enterprise-v3 repoWesley Hearn2015-04-223-44/+1
|/ /
* | Merge pull request #166 from detiber/awsTerminateThomas Wiest2015-04-214-121/+69
|\ \ | | | | | | aws terminate playbook improvements
| * | aws terminate playbook improvementsJason DeTiberus2015-04-204-121/+69
| |/ | | | | | | | | | | | | | | | | | | | | | | | | - Reduce duplication in terminate playbooks between openshift-master and openshift-node (they both now just include playbooks/aws/terminate.yml - update openshift-cluster terminate playbook to include the new shared terminate playbook, also delete all cluster hosts at once instead of treating masters and nodes differently. - remove env, host-type and env-host-type tags from instance before terminating (since most users can't terminate, we are mostly just renaming instances to -terminate and stopping them, so this prevents "terminated" hosts from being returned by the dynamic inventory, at least after the cache is refreshed)
* | Merge pull request #172 from detiber/aws_vpcThomas Wiest2015-04-212-8/+33
|\ \ | | | | | | add vpc support to ec2 cluster, add more overrides for variables
| * | add vpc support to ec2 cluster, add more overrides for variablesJason DeTiberus2015-04-212-8/+33
| | |
* | | Merge pull request #170 from detiber/docker-networkThomas Wiest2015-04-211-4/+17
|\ \ \ | | | | | | | | openshift-sdn-node fix to use /etc/sysconfig/docker-network
| * | | openshift-sdn-node fix to use /etc/sysconfig/docker-networkJason DeTiberus2015-04-211-4/+17
| | |/ | |/|
* | | Merge pull request #165 from detiber/noEmptyNodesThomas Wiest2015-04-211-2/+2
|\ \ \ | | | | | | | | Fix issue with nodes being set to an empty string when generating master config
| * | | Fix issue with nodes being set to an empty string when generating master configJason DeTiberus2015-04-201-2/+2
| |/ /
* | | Merge pull request #164 from detiber/bugFixRunOnMasterThomas Wiest2015-04-211-7/+10
|\ \ \ | | | | | | | | Fix common node config playbook when ansible is run on the first master
| * | | Fix common node config playbook when ansible is run on the first masterJason DeTiberus2015-04-201-7/+10
| |/ /
* | | Merge pull request #163 from detiber/todoForSyncThomas Wiest2015-04-211-0/+3
|\ \ \ | | | | | | | | Todo for sync
| * | | Add TODO for making node certificate sync more efficientJason DeTiberus2015-04-201-0/+3
| |/ /
* | | Merge pull request #162 from detiber/nix_deployment-type_tagThomas Wiest2015-04-213-4/+1
|\ \ \ | | | | | | | | Remove deployment-type tags
| * | | Remove deployment-type tagsJason DeTiberus2015-04-203-4/+1
| |/ /
* | | Merge pull request #173 from detiber/onlineRepoWesley Hearn2015-04-213-6/+9
|\ \ \ | |_|/ |/| | Update online v3 repo file
| * | Update online v3 repo fileJason DeTiberus2015-04-213-6/+9
|/ /
* | Merge pull request #161 from detiber/noMoreMatersWesley Hearn2015-04-212-2/+2
|\ \ | |/ |/| Fix typos... master not mater
| * Fix typos... master not materJason DeTiberus2015-04-202-2/+2
|/
* Merge pull request #139 from detiber/configUpdatesMasterThomas Wiest2015-04-20110-1125/+1614
|\ | | | | Massive refactor, deployment-type support, config updates, reduce duplication
| * Fixup typosJason DeTiberus2015-04-152-2/+2
| |
| * Merge pull request #19 from lhuard1A/move_pool-refreshJason DeTiberus2015-04-152-4/+3
| |\ | | | | | | Move `virsh pool-refresh`
| | * Move `virsh pool-refresh`Lénaïc Huard2015-04-152-4/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | The `pool-refresh` command is used to ask libvirt to rescan the content of a volume pool. This is used to make `libvirt` take into account volumes that were created outside of livirt control i.e.: not with a `virsh` command. `pool-refresh` is useless after a `pool-create` as the content is scanned at creation. `pool-refresh` is mandatory after having created files inside an existing pool.
| * | Merge pull request #20 from lhuard1A/locale_proofJason DeTiberus2015-04-152-2/+2
| |\ \ | | | | | | | | Make the error message checks locale proof
| | * | Make the error message checks locale proofLénaïc Huard2015-04-152-2/+2
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On a computer which has a locale set, the error messages look like this: ``` $ virsh net-info foo erreur :impossible de récupérer le réseau « foo » erreur :Réseau non trouvé : no network with matching name 'foo' ``` ``` $ virsh pool-info foo erreur :impossible de récupérer le pool « foo » erreur :Pool de stockage introuvable : no storage pool with matching name 'foo' ``` The classical way to make those tests locale proof is to force a given locale. Like this: ``` $ LANG=POSIX virsh net-info foo error: failed to get network 'foo' error: Réseau non trouvé : no network with matching name 'foo' ``` ``` $ LANG=POSIX virsh pool-info foo error: failed to get pool 'foo' error: Pool de stockage introuvable : no storage pool with matching name 'foo' ``` It looks like the "Network not found" or "Storage pool not found" parts of the message are generated by the `libvirtd` daemon and are not subject to the locale of the `virsh` client. The clean fix consists in patching `libvirt` so that `virsh` sends its locale to the `libvirtd` daemon. But in the mean time, it is safer to have our playbook match the part of the message which is not subject to the daemon locale.
| * | Merge pull request #21 from lhuard1A/fix_ansible_libvirt_metadataJason DeTiberus2015-04-152-5/+7
| |\ \ | | | | | | | | Fix libvirt metadata used to store ansible tags
| | * | Fix libvirt metadata used to store ansible tagsLénaïc Huard2015-04-162-6/+8
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to https://libvirt.org/formatdomain.html#elementsMetadata , the `metadata` tag can contain only one top-level element per namespace. Because of that, libvirt stored only the `deployment-type-{{ deployment_type }}` tag. As a consequence, the dynamic inventory reported no `env-{{ cluster }}` group. This is problematic for the `terminate.yml` playbook which iterates over `groups['tag-env-{{ cluster-id }}]` The symptom is that `oo_hosts_to_terminate` was not defined. In the end, as Ansible couldn’t iterate on the value of `groups['oo_hosts_to_terminate']`, it iterated on its letters: ``` TASK: [Destroy VMs] *********************************************************** failed: [localhost] => (item=['g', 'destroy']) => {"failed": true, "item": ["g", "destroy"]} msg: virtual machine g not found failed: [localhost] => (item=['g', 'undefine']) => {"failed": true, "item": ["g", "undefine"]} msg: virtual machine g not found failed: [localhost] => (item=['r', 'destroy']) => {"failed": true, "item": ["r", "destroy"]} msg: virtual machine r not found failed: [localhost] => (item=['r', 'undefine']) => {"failed": true, "item": ["r", "undefine"]} msg: virtual machine r not found failed: [localhost] => (item=['o', 'destroy']) => {"failed": true, "item": ["o", "destroy"]} msg: virtual machine o not found failed: [localhost] => (item=['o', 'undefine']) => {"failed": true, "item": ["o", "undefine"]} msg: virtual machine o not found failed: [localhost] => (item=['u', 'destroy']) => {"failed": true, "item": ["u", "destroy"]} msg: virtual machine u not found failed: [localhost] => (item=['u', 'undefine']) => {"failed": true, "item": ["u", "undefine"]} msg: virtual machine u not found failed: [localhost] => (item=['p', 'destroy']) => {"failed": true, "item": ["p", "destroy"]} msg: virtual machine p not found failed: [localhost] => (item=['p', 'undefine']) => {"failed": true, "item": ["p", "undefine"]} msg: virtual machine p not found failed: [localhost] => (item=['s', 'destroy']) => {"failed": true, "item": ["s", "destroy"]} msg: virtual machine s not found failed: [localhost] => (item=['s', 'undefine']) => {"failed": true, "item": ["s", "undefine"]} msg: virtual machine s not found failed: [localhost] => (item=['[', 'destroy']) => {"failed": true, "item": ["[", "destroy"]} msg: virtual machine [ not found failed: [localhost] => (item=['[', 'undefine']) => {"failed": true, "item": ["[", "undefine"]} msg: virtual machine [ not found failed: [localhost] => (item=["'", 'destroy']) => {"failed": true, "item": ["'", "destroy"]} msg: virtual machine ' not found failed: [localhost] => (item=["'", 'undefine']) => {"failed": true, "item": ["'", "undefine"]} msg: virtual machine ' not found failed: [localhost] => (item=['o', 'destroy']) => {"failed": true, "item": ["o", "destroy"]} msg: virtual machine o not found failed: [localhost] => (item=['o', 'undefine']) => {"failed": true, "item": ["o", "undefine"]} msg: virtual machine o not found etc… ```
| * / explicitly use python2 for libvirt dynamic inventoryJason DeTiberus2015-04-151-1/+1
| |/
| * fix missed absolute path reference to mktempJason DeTiberus2015-04-151-1/+1
| |
| * Configuration updates for latest builds and major refactorJason DeTiberus2015-04-14111-1198/+1686
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Configuration updates for latest builds - Switch to using create-node-config - Switch sdn services to use etcd over SSL - This re-uses the client certificate deployed on each node - Additional node registration changes - Do not assume that metadata service is available in openshift_facts module - Call systemctl daemon-reload after installing openshift-master, openshift-sdn-master, openshift-node, openshift-sdn-node - Fix bug overriding openshift_hostname and openshift_public_hostname in byo playbooks - Start moving generated configs to /etc/openshift - Some custom module cleanup - Add known issue with ansible-1.9 to README_OSE.md - Update to genericize the kubernetes_register_node module - Default to use kubectl for commands - Allow for overriding kubectl_cmd - In openshift_register_node role, override kubectl_cmd to openshift_kube - Set default openshift_registry_url for enterprise when deployment_type is enterprise - Fix openshift_register_node for client config change - Ensure that master certs directory is created - Add roles and filter_plugin symlinks to playbooks/common/openshift-master and node - Allow non-root user with sudo nopasswd access - Updates for README_OSE.md - Update byo inventory for adding additional comments - Updates for node cert/config sync to work with non-root user using sudo - Move node config/certs to /etc/openshift/node - Don't use path for mktemp. addresses: https://github.com/openshift/openshift-ansible/issues/154 Create common playbooks - create common/openshift-master/config.yml - create common/openshift-node/config.yml - update playbooks to use new common playbooks - update launch playbooks to call update playbooks - fix openshift_registry and openshift_node_ip usage Set default deployment type to origin - openshift_repo updates for enabling origin deployments - also separate repo and gpgkey file structure - remove kubernetes repo since it isn't currently needed - full deployment type support for bin/cluster - honor OS_DEPLOYMENT_TYPE env variable - add --deployment-type option, which will override OS_DEPLOYMENT_TYPE if set - if neither OS_DEPLOYMENT_TYPE or --deployment-type is set, defaults to origin installs Additional changes: - Add separate config action to bin/cluster that runs ansible config but does not update packages - Some more duplication reduction in cluster playbooks. - Rename task files in playbooks dirs to have tasks in their name for clarity. - update aws/gce scripts to use a directory for inventory (otherwise when there are no hosts returned from dynamic inventory there is an error) libvirt refactor and update - add libvirt dynamic inventory - updates to use dynamic inventory for libvirt
* | Merge pull request #156 from kwoodson/ans_autokeysKenny Woodson2015-04-161-0/+1
|\ \ | | | | | | Adding ansible-tower-cli to tower.
| * | Adding ansible-tower-cli rpm to towerKenny Woodson2015-04-161-0/+1
| |/
* | Merge pull request #157 from tdawson/201504-tower-amiTroy Dawson2015-04-161-1/+1
|\ \ | |/ |/| update tower ami image to latest libra-ops-rhel7
| * update tower ami image to latest libra-ops-rhel7Troy Dawson2015-04-161-1/+1
|/
* Merge pull request #152 from net-engine/aws_readmeThomas Wiest2015-04-142-3/+22
|\ | | | | Launch openshift on AWS issues
| * Add extra information for AWS READMERicardo Bernardeli2015-04-132-3/+22
| | | | | | Make security group an environment variable with default to ‘public’