summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* lvm-direct support for awsJason DeTiberus2015-04-237-5/+123
| | | | | | | | | | | | | | | | | - Create a separate docker volume in aws openshift-cluster playbooks - default to using ephemeral storage, but allow to be overriden - allow root volume settingsto be overriden as well - add user-data cloud-config to bootstrap the installation/configuration of docker-storage-setup - pylint cleanup for oo_filters.py - remove left over traces to the deployment_type tags which were previously removed - oo_get_deployment_type_from_groups filter in oo_filters.py - cluster list playbooks references to oo_get_deployment_type_from_groups filter
* Merge pull request #180 from kwoodson/pylint_multi_ec2Kenny Woodson2015-04-232-46/+53
|\ | | | | Multi_ec2 fixes for pylint and path change. Also added refresh-cache.
| * Adding refresh-cache option and cleanup for pylint. Also updated for ↵Kenny Woodson2015-04-222-46/+53
| | | | | | | | aws/hosts/ being added.
* | Merge pull request #183 from kwoodson/git_hooksKenny Woodson2015-04-231-1/+3
|\ \ | | | | | | Testing hooks.
| * | Testing jenkins hooksKenny Woodson2015-04-231-1/+3
| | |
* | | Merge pull request #178 from detiber/fix_aws_defaultsThomas Wiest2015-04-239-58/+445
|\ \ \ | | | | | | | | Allow variable overriding for ec2 deployment_vars
| * | | Allow variable overriding for ec2 deployment_varsJason DeTiberus2015-04-229-58/+445
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - users can now override the deployment_vars variables with the assocated ec2_* variables - added deployment_type and env specific vars files that load some ec2_* overrides - added the ability to search for amis by ami_name - this allows us to specify a base name with a wildcard to have the playbook choose the latest available image for that image name - added a copy of the ec2_find_ami module that will be in ansible 2.0 until we can make ansible 2.0 a requirement.
* | | | 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
| |
generated by cgit v1.2.3 (git 2.25.1) at 2024-09-19 16:12:55 +0000