summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-03-14Merge "xci: config: evn-vars: Dynamically determine the Ansible version"Markos Chandras1-1/+1
2018-03-14Merge "xci: Fix group for the OPFNV host"Markos Chandras4-4/+4
2018-03-14kubespray: Do not grep log text to determine failureFatih Degirmenci1-7/+4
Pipefail should be sufficient to determine the failure so this change gets rid of grepping the logs for failure and unreachable to leave the evaluation to bash itself. Change-Id: I0fcd6016ddc19eb11b3eb2402347032a11c5a32a Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-13Merge "Set VM node names and log the installer version based on installer"Fatih Degirmenci4-5/+9
2018-03-13Merge "xci: osa: Switch git repositories to github.com"Markos Chandras3-79/+83
2018-03-13xci: Merge configure-network and syncronize-time rolesMarkos Chandras18-12/+31
Those roles were too small and only used as part of the host bootstrapping process. As such, we merge them into a common 'bootstrap-host' role which can be used to prepare the hosts after the initial deployment Change-Id: Ifc84cf40b98ced91b31aae699dc28e9642380550 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-13xci: Fix group for the OPFNV hostMarkos Chandras4-4/+4
Ansible gets upset if we use the same name for a host and a group so add the OPNFV host to the deployment group. This fixes the following warning: [WARNING]: Found both group and host with same name: opnfv Change-Id: Idbaff712d40e8234721332aee6d10d5ba44c62ce Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-13Merge changes from topics 'allow-pass-ansible-arguments', ↵Markos Chandras5-20/+20
'allow-pass-ansible-arguments-force-verify' * changes: xci: OSA: Fix warning about missing inventory file xci: Pass the XCI_PATH variable to all Ansible calls xci: Rename XCI_ANSIBLE_VERBOSITY to XCI_ANSIBLE_PARAMS
2018-03-13xci: osa: Switch git repositories to github.comMarkos Chandras3-79/+83
CI sometimes fails with the following error "msg": "fatal: unable to access 'https://git.openstack.org/openstack/dragonflow/': Failed to connect to git.openstack.org port 443: Connection timed out" We assume that the upstream OpenStack mirrors may get very busy at times or the connectivity between the CI and the OpenStack repos is not very stable. As such, lets switch to github.com for the majority of OSA repositories such as the roles and the services which may improve the situation. Change-Id: Ia4668d692077a785c76adeda128eb4cf9f4516a7 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-13Do not grep log text to determine failureFatih Degirmenci1-10/+1
Pipefail should be sufficient to determine the failure so this change gets rid of grepping the logs for failure and unreachable to leave the evaluation to bash itself. Change-Id: Ie928438c1503b086159276af4308f5549b58cb71 Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-12Set VM node names and log the installer version based on installerFatih Degirmenci4-5/+9
The VM nodes are named using OpenStack terminology but since we now have Kubernetes scenario, it is appropriate to name the nodes accordingly. Apart from that, we normally log the version of OSA which is not correct for kubernetes deployments. Logging Kubespray version would be correct for this scenario. Change-Id: I3964574c8592450f9fa3126f19e27bb78fd9d174 Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-12Merge "Combine vars setting installer type in INSTALLER_TYPE"Fatih Degirmenci2-2/+2
2018-03-12Merge "Combine vars setting installer type in INSTALLER_TYPE"Fatih Degirmenci7-20/+20
2018-03-12remove .ansible directoryPeriyasamy Palanisamy1-9/+10
It's needed to remove .ansible directory which could be a leftovers by some other ansible execution enviroment which would affect XCI deployment. Change-Id: Ic334ef1ca53878721762b23b238fe9a8aeefa0e0 Signed-off-by: Periyasamy Palanisamy <periyasamy.palanisamy@ericsson.com>
2018-03-12xci: config: evn-vars: Dynamically determine the Ansible versionMarkos Chandras1-1/+1
There is no good reason why we should pin the Ansible version without the ability to override it. OpenStack Ansible is doing a great work testing Ansible releases as early as possible so we could benefit from that work and use whatever Ansible our pinned OSA uses. Change-Id: I4677dbefc68f7140417ece34d013ca9674baab43 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-12Combine vars setting installer type in INSTALLER_TYPEFatih Degirmenci2-2/+2
Change-Id: Ie196d1df537d09f0f91e43ab5e0305a45d543815 Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-12Combine vars setting installer type in INSTALLER_TYPEFatih Degirmenci7-20/+20
Change-Id: I330bc036f901d4ba61bc94ee6e085cadf54b4d8b Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-12xci: OSA: Fix warning about missing inventory fileMarkos Chandras1-1/+1
This is similar to 346079ea1b8dbda0c5e282c18f30cbac7e907d1e ("xci: Fix warning about missing inventory file") This playbook only affects localhost so make it explicit. Fixes: [WARNING]: Host file not found: inventory [WARNING]: provided hosts list is empty, only localhost is available Change-Id: If8548a36498ea3191de33cff94acbaee6ed04ce3 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-12xci: Pass the XCI_PATH variable to all Ansible callsMarkos Chandras2-3/+5
The XCI_PATH variable is used by various playbooks to find the root directory of the XCI repository so it's much cleaner to pass it on every Ansible call. Change-Id: Ifc0b77fa767e48bd1dc6daa44c6251c02983fd4d Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-12xci: Rename XCI_ANSIBLE_VERBOSITY to XCI_ANSIBLE_PARAMSMarkos Chandras5-20/+18
We don't quite need a specific env variable just for Ansible verbosity so we can rename this variable to make it clear that it can be used to pass any Ansible option to XCI. Change-Id: Ie20517d4b563bfc6daeb27848168d36da7014cee Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-12xci: Drop the python-crypto dependencyMarkos Chandras4-9/+0
The upstream pw-token-gen tool doesn't need python-crypto anymore since e9f957861b4160640f6debb2b939084ec43b43b2 ("Make pw-token-gen.py more random") so we no longer need to install that package. Change-Id: Ib53f246db999ff8ecfed2e3f62143c780c483fbd Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-12Merge "xci: Fix warning about missing inventory file"Markos Chandras1-2/+2
2018-03-12Merge "xci: roles: configure-network: Avoid passing empty string to module"Markos Chandras1-1/+1
2018-03-12Merge changes from topic 'set-proper-constrains'Markos Chandras8-38/+44
* changes: xci: OSA: Ensure proper constrains are applied to pip installations xci: Move functest required packages to the functest role
2018-03-12Merge "xci: scripts: vm: Update package manager cache and ignore errors"Markos Chandras1-4/+10
2018-03-10xci: scripts: vm: Update package manager cache and ignore errorsMarkos Chandras1-4/+10
We need to make sure that the local package cache is updated before we try to install packages. Moreover, from time to time, something in the repos may be broken so we don't want to break the CI job. As such, make that step non-fatal as simply send the output to stdout so we can see what's wrong. Change-Id: I148e3b5435c604bb0d54339935255740c8ce07c0 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-10xci: roles: configure-network: Avoid passing empty string to moduleMarkos Chandras1-1/+1
The package module gets upset if we pass it an empty string as a package name Mar 09 15:15:33 TASK [configure-network : Ensure networking packages are present] ************** Mar 09 15:15:49 fatal: [opnfv]: FAILED! => {"changed": false, "failed": true, "msg": "No package matching '' found available, installed or updated", "rc": 126, "results": ["iproute-3.10.0-87.el7.x86_64 providing iproute is already installed", "No package matching '' found available, installed or updated"]} As such, lets pass an existing package for non-Debian distros to make the module happy. Change-Id: Ib1128057097193278cfe50f79daf35332b54e781 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-10xci: OSA: Ensure proper constrains are applied to pip installationsMarkos Chandras3-0/+6
We need to make sure that the pip packages that we install are compatible with the OSA components. Change-Id: I87e80dc9b8fb862d9f9039d835b0908f752de3ca Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-10xci: Move functest required packages to the functest roleMarkos Chandras7-38/+38
The docker packages that we install in the OPNFV VM are needed by functest so add them to the related role. Change-Id: I6ebe76fd030859f757d41ecf20c30ab76888ee9c Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-10Merge "bifrost: Do not run virsh commands if it's not installed"Markos Chandras1-2/+4
2018-03-10Merge "bifrost: Move DIB configuration variables to bifrost"Markos Chandras2-19/+20
2018-03-10Merge "bifrost: Optimize the OPNFV bifrost playbook"Markos Chandras1-31/+11
2018-03-09xci: Fix warning about missing inventory fileMarkos Chandras1-2/+2
We no longer have a global inventory file and we don't need one right now since the bootstrapping opearations happen on the localhost anyway so we could be explicit and get rid of the following warning: [WARNING]: Host file not found: inventory [WARNING]: provided hosts list is empty, only localhost is available Change-Id: I5cfe7b0971397665cf9eae9c54985c44c4506449 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-09Merge "xci: roles: configure-network: Wait for connection when configuring ↵Fatih Degirmenci1-6/+35
the network"
2018-03-09bifrost: Optimize the OPNFV bifrost playbookMarkos Chandras1-31/+11
We could combine several plays into a single one when they are executed to the same set of nodes. This has the added benefit that we avoid all the Ansible warm-up operations when a new play is started. Change-Id: Ibcad07d1b74f1d7cc5a9c60e4b2a6130fb6178fd Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-09bifrost: Do not run virsh commands if it's not installedMarkos Chandras1-2/+4
Check if 'virsh' is available before trying to remove the VMs: Fixes the following warning ./scripts/destroy-env.sh: line 36: virsh: command not found ./scripts/destroy-env.sh: line 37: virsh: command not found ./scripts/destroy-env.sh: line 36: virsh: command not found ./scripts/destroy-env.sh: line 37: virsh: command not found ./scripts/destroy-env.sh: line 36: virsh: command not found ./scripts/destroy-env.sh: line 37: virsh: command not found Change-Id: I9c82c8b8b40462584c6324871284ba12d660ae9d Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-09bifrost: Move DIB configuration variables to bifrostMarkos Chandras2-19/+20
The DIB variables only make sense on bifrost so they should live in the bifrost provision script. Change-Id: I05baca7b9c6889a0d401a9288998991324d270af Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-09xci: roles: configure-network: Wait for connection when configuring the networkMarkos Chandras1-6/+35
Ansible some times loses connection with the host after we configure its network interface. This shouldn't happen since the host is actually up and running. As a result of which, we need to fire-and-forget the restart of the network service and then try to reach the SSH port. We also need to make sure that all required packages are installed. Finally, we drop the 'ignore_errors' parameter since we really need to know when the networking configuration has failed. Change-Id: I366192737b5c4e01964eaf187396eababbdc808e Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-09Merge "xci: Rename provision-vm-nodes playbook"Markos Chandras2-1/+1
2018-03-09Merge "Add subproject INFO.yaml"Fatih Degirmenci1-0/+42
2018-03-09xci: Rename provision-vm-nodes playbookMarkos Chandras2-1/+1
The provision-vm-nodes playbook doesn't do any VM provisioning. It basically just bootstraps the bifrost repository so rename it to make it clear what this is about. Change-Id: I1c10bc6135686c5197d553a885412a94312a41bc Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-08bifrost: Ensure OS images are present in the PXE directoryMarkos Chandras3-6/+11
The XCI deployment script may wipe the PXE directories so we need to place the OS images after this has happened so we don't download them every time. Change-Id: Iccf943bc75592e7f5dddea59dfee14b2b080e7ef Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-08Add subproject INFO.yamlFatih Degirmenci1-0/+42
This change adds INFO.yaml for releng-xci Gerrit Project which is constructed by subset of committers of Releng project who are active in xci. New committers to releng-xci Gerrit Project will be nominated on Gerrit via separate changes and voted by releng-xci committers that are listed within releng-xci/INFO.yaml file as of today. A single mail with all the nominations will be sent to opnfv-tsc and opnfv-tech-discuss mailing lists announcing nominations. A followup mail will also be sent to announce the results. Change-Id: I1d49585276304b29ac86b03aff5fd45974a02ccc Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-07bifrost: Use the pre-built XCI OS images instead of dibMarkos Chandras4-6/+56
We already have images for all 3 distros which we could simply re-use for all the XCI VM deployments instead of building new ones everytime with DIB. The images will be copied to the new VM from the cache directory if they are available otherwise we will simply download them during the XCI execution phase. Change-Id: I2a8391650558511668654c6b54a10db316f867a2 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-07Merge "Bump bifrost sha and set UPPER_CONSTRAINTS_FILE"Markos Chandras2-1/+4
2018-03-07Merge "Remove opnfv vm from known_hosts"Fatih Degirmenci1-0/+4
2018-03-07Bump bifrost sha and set UPPER_CONSTRAINTS_FILEFatih Degirmenci2-1/+4
The lack of UC means that pip is attempting to install libvirt-python 4.1.0, which is failing on Ubuntu due to https://bugs.launchpad.net/openstack-requirements/+bug/1753539. Currently upper-constraints limits libvirt-python to 4.0.0, which does not have this issue. Change-Id: Idc97a0d9928cebafc9167614723236d2584cc93f Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-02xci: Make few of OpenStack-Ansible components optionalMarkos Chandras4-4/+9
Ironic and Horizon are not quite needed for a functional deployment and they are not currently required by functest so we can remove them from the default deployment. Change-Id: I171483f7b774951f84687529e98cb519afa48043 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-03-02xci: Bump SHAs to the latest stable/queensFatih Degirmenci3-64/+64
Rocky is still in early stages of development so bump SHAs to the latest version for Queens. Change-Id: I0721e6188889c3548c266c042414397385b85ca2 Signed-off-by: Fatih Degirmenci <fdegir@gmail.com>
2018-03-02Merge "Pin kubespray to working sha"Fatih Degirmenci1-1/+3