Age | Commit message (Collapse) | Author | Files | Lines |
|
[This is similar to upstream https://review.openstack.org/389079]
This somewhat reverts 0a82e3d8d29f85229e1ed29d90de47346bc1d9b1
("Serialize localhost actions during final host deployment").
The only point where we want to serialize the steps is when accessing
common files on the host. In the bifrost-deploy-nodes-dynamic role,
all tasks are accessing files based on the 'inventory_hostname'
variable, which means there is no conflict there and tasks on these
role can run in parallel. This allows bifrost to provision machines
in parallel during pxe boot which speeds up the entire operation.
Change-Id: I64b1f2832c333c89e33e8f9d19f03e4d653d74b4
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
If this is the first time we run bifrost on a host or if we haven't
run bifrost since the last reboot, restarting the dnsmasq service will
fail because of conflicts with the dnsmasq process running by libvirt.
This is later on 'fixed' by bifrost by killing the one from libvirt.
Make sure that this spurious failure will not prevent the rest of the
cleanup process to move on.
Change-Id: Iebf37881570c27cb07cea3cecdb0fa8ac3fc511b
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
Several tasks in the final playbooks run locally and there is a chance
to encounter race conditions when multiple tasks run in parallel
and all access or modify the same local files. We fix this by splitting
the final steps into multiple plays and serializing those that run
locally.
Change-Id: I38c3465d9c4714d9db772f9ff9e32ec1d4a91db8
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
There is no point in us keeping our own copy of the upstream
bifrost-prepare-for-test-dynamic playbook just for overriding
the wait_timeout variable. Delete the duplicate role and move the
variable definition to the deployment script instead.
Change-Id: Ib8066fefe80e9a3190d878eb5dba7b8d86a85e07
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
This change
- Creates job templates to verify opnfv/bifrost patches
- Introduces type in order to prepare verification on baremetal
- Fixes git cloning so the job shows changes for the corresponding project
- Gets rid of unnecessary slave parameters from slave-params.yml
- Fixes build blocking
- Updates the bifrost-verify.sh script and includes in shell so we don't have
to clone releng repo separately
- Updates destroy-env.sh to create /httpboot and /tftpboot with permissions 755
- Removes daily infra-bifrost job and scripts as we don't use them at the moment
Change-Id: Icb5a543628d3321bfe4e549ab92c67443ac578d8
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|
|
Change-Id: If2af5177e8614651664c7de9cde3e681ec2bb43c
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|
|
It's necessary to run the deployment script using 'sudo' so that
exported variables within the script are preserved in playbooks
once we gain elevated privileges.
Change-Id: I824b44199d57b6acf14804c3ab1825d7aa80da4e
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
Nearly all operations in the destroy-env.sh script need to be executed
as root so make sure that's the case before we try to do anything.
This also fixes the dnsmasq.leases path and we check that the file
exists before we try to wipe it. The dnsmasq is also restarted to take
the empty leases file into consideration. Finally, we improve the
documentation to run the necessary commands using sudo.
Change-Id: I56df821ca8a93307117a94a11abd0867b27d8443
Signed-off-by: Markos Chandras <mchandras@suse.de>
|
|
This will allow to run on different distros depending on
settings exported.
Change-Id: I068d7bb2eebfd009dceeff057c8a65361483788b
Signed-off-by: Yolanda Robla <yroblamo@redhat.com>
|
|
Change-Id: Ic3729d9450edabd809eed53525632e2b36752401
Signed-Off-By: Yolanda Robla <yroblamo@redhat.com>
|
|
It relies on this var being exported, and if the script
is ran manually and this var is not set, it crashes. Adding
a default to this var to avoid this failure.
Change-Id: I5c61a1d9f1aafe2d6c64af70d3237b6c5b0d1e1d
Signed-Off-By: Yolanda Robla <yroblamo@redhat.com>
|
|
Upstream change: https://review.openstack.org/#/c/358477/
Change-Id: Ibfef32aec4b62d8d1d13728abec374a2d4560d81
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|
|
This change
- Fixes readme of puppet-infracloud
- renames destroy_env.sh script
- enables mail notification
- disables removal of DIB images by default
Change-Id: I628b26b976e38772cbbc69b737a545fe801639ba
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|
|
This change
- creates upstream daily job to trigger and control the execution of jobs
that provision nodes using bifrost, deployment using puppet-infracloud,
and simple smoke test job.
- fixes the hashbang placement in 2 of the scripts.
- removes sandbox dummy jobs as the infra jobs can serve as examples now.
Only the node provisioning job is functional at the moment.
Change-Id: I311d159956d8b7ce3f7a500fac5a7dcb459104cf
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|
|
It is needed on rabbit install
Change-Id: I0ebf66883cc7c66771b3b5b4c2cccc590e61e5ca
Signed-off-by: Yolanda Robla <yroblamo@redhat.com>
|
|
The 10g disk space is not enough to spin up some vms and test. With
default flavors, just a m1.small needs 20g of disk. Increasing to
100g of disk in each vm, so we can test with better conditions.
Change-Id: I7b273dc5881cb6af67b7852c1318ea739217602f
Signed-off-by: Yolanda Robla <yroblamo@redhat.com>
|
|
It defaults to 1 cpu, and that means that computes won't be able
to spin up any node because the number of cpus are exhausted.
Currently the playbook doesn't allow to individually set cpus,
so defaulting everything to 4 temporarily.
Change-Id: Ic6058b28f5d75a270e3897157ba034f804af8ab0
Signed-Off-By: Yolanda Robla <yroblamo@redhat.com>
|
|
This change
- increases the node provisioning timeout to 1900 seconds since the original
timeout used by upstream bifrost (900 seconds) is not suited for provisioning
VMs with Ubuntu.
- fixes permissions of the scripts.
- removes .gitkeep.
This change is not upstreamed since bifrost uses cirros for test & development
purposes and it is probably not in their interest to have increased timeout.
Change-Id: I4b59a321fa26f8b5895ec577a19d10f5cdcace0f
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|
|
Change-Id: Ic7e33f53d0111b0e833d7547fff16458506b063b
Signed-off-by: Yolanda Robla <yroblamo@redhat.com>
|
|
Change-Id: I890d36d0db54efcbcb9911707b72bf58416550b3
Signed-off-by: Fatih Degirmenci <fatih.degirmenci@ericsson.com>
|