In I1bc8b00eb98055a6e0f06af287cfa10c02ee2094 the
sample ironic.conf file was modified to prepare
for the transition to oslo-config-generator. Since
bifrost uses the sample config file, we need to
modify our regexps to match both the new and old
sample config file. We also need to replace lines
with the new style.
Change-Id: I1757cc26c7f687b8070ca1f9c61a933fbcee8774
The conditionals on the DIB steps were lacking | bool and
as such they were previously not possible difficult to
override on the command line.
Change-Id: I69a263e7bedbadec52c88df72f592e3b05c3e52d
Diskimage-builder passes the localization environment variables
through to the image being built. Ansible is explicitly setting
"en_US.UTF-8" by default. Depending on the DIB build underway,
and the base image, programs executing in that environment
may fail to execute if localization is not present in the base
image.
As such, we will override the settings for the DIB step.
Closes-Bug: #1563338
Change-Id: I05f0eeb71d3c227b2d8ece52c603c8cfe480c46c
Remove a reference to a nonexistent playbook. Also remove
LOG_LOCATION since we now use collect-test-info.sh.
Change-Id: I646c7d83fa9f16cbe0a95309ab52aa833ebbad66
adding paramiko to requirements.txt, while not a direct dependency of
Bifrost it is a dependency of ansible. Adding to requirements ensures
that it is installed.
Change-Id: I4cd18c43fa2a30f6003c374d3989c6d378ad39ef
Closes-bug: 1559586
The nature of multi-host interactions and testing can
be extremely racey. Since IPA can offer SSH, the wait
step can be short circuited resulting in us not waiting
long enough.
As such, introducing a longer wait _and_ re-check of SSH
connectivity.
While this does not completely fix the underlying race, it
is intended to help the test be more reliable in CI.
Partial-Bug: #1559764
Change-Id: Ief9ff9b2d158fa5322b5bb9be74105ba3d13b7fe
Presently the post-test information collection script is only
collecting a single test virtual machine's console log.
As the DHCP test operates with multiple VMs, we need to collect
all console logs to help identify potential test failures.
Change-Id: I8ddfb7870ea57b51f6340db4ba388c13cc35c059
The ansible add_host module only triggers once per play and as
such we need to loop over the hosts in our baremetal group, and
explicitly add each host.
Change-Id: I603586fe0d46bdf0e61ee00c14f74811a2c02ba8
Direct the mitaka release notes generation to an
origin/stable/mitaka branch for when the branch
has been cut.
Change-Id: I55c61e8fa63dacc1b22d9b91694013704d711706
The inspection module has been submitted for inclusion in ansible,
and as such a duplicate is no longer required. Removing duplicate
and adding call to retreieve the module if missing.
Change-Id: Ib4b395bee5ae80c4cafc9628710b536db895d952
network_metadata's parameter "nics" is expected to be a list. In some
instances it will be left as a string unless the type is explicitly
set. Also the default of '' is incorrect since a string is not expected
but a list. Omitting it is the correct way, since it is not rquired.
Change-Id: If3376ee24507ed6f22e269e4440d2e371ff33eb4
In some work we're doing, we override addressing_mode with the
string 'static'. This should lead to getting network_info.json in our
configdrives, but it doesn't because the test logic in these tasks isn't
right. It has been tested, and the undefined test short circuits, so
we don't ever get errors whether the value is defined as dhcp, static,
or undefined.
Change-Id: I01f42c003dae21b3f25e2f760214ba876ae1aea9
This change allows mysql_username and mysql_password to be passed to
bifrost-ironic-install via the environment variables mysql_user and
mysql_pass. This is useful when there is a preexisting mysql installation
with passwords alreay set.
Change-Id: I6dfd3ec0e769f0af6b157cde106885eabc24d120
The DHCP test ultimately relies upon bifrost putting in place
the correct DHCP configuration to reliably boot machines, and then
(hopefully) ensure that no unknown machines are given leases.
This is in order to help simulate the Infracloud configuration.
In order to do this, and hopefully get the CI test into a passing
state, I have increased the amount of memory allocated to each VM
as the realistic minimum for IPA is 1 GB, and introduced specific
stops and starts to VMs present during the test sequence in order
to limit the number of concurrent running virtual machines.
Change-Id: Ia612414edd80d154200e9df12b410a327db19cff
Corrected the netstat data collection as the --version flag causes
a non-zero exit code with netstat.
Added interface counter data collection to allow for additional
troubleshooting context.
Change-Id: I7e0e26cc735d6c0ad9052547bc0eefe3ed3877ae
DIB based tests appear to fail when uuidgen is not present.
On Debian based systems, this is packaged in uuid-runtime.
Change-Id: Ib040406779a4144c6c0409cefa5e420c15ad2bcb
Add two README files to avoid any confusion about the intent of the
scripts/ and tools/ directories.
Change-Id: I50d94088649ec6b6d7f0799f40760e03b5aad371
When we use a single VM, there is no need to wait extra
time for keyscan to complete. Add a flag to wait this
additional time when we perform multinode testing, such
as in DHCP test.
Change-Id: I10a54a91af145d6d44873865f5b7df48c68c59c7
When we implemented reno support, we did so early on and never put
in place the proper index for the release notes. With the upcoming
release, this needed to be corrected.
Change-Id: I73f7f7c3b399c8bc1805c8caab88876dafdd80dc
Colon characters are magical characters in yaml, and as such we
can't have them in release notes.
This is also called the realization that TheJulia needs to put
reno check job in-place.
Change-Id: I6eee79af773574db7f113e99223d75cbef7e8b0b
The flag "create_ipa_image: true" may be ignored if the IPA image file
is already present on the local disk.
However, because "download_ipa" defaults to true and runs first, the IPA
image file is always present before the tests to determine whether to
create it or not (unless one specifically sets "download_ipa: false").
This change causes the "create_ipa_image: true" flag to override
the behavior of the "download_ipa" flag.
Change-Id: I5f8e8f1d8893a279aae5b733cde5c07265197f5d
Changing the standard test to invoke inspection in
order to help raise any inspector related issues faster
and improve overall quality.
Change-Id: I85af6c21e17a965323051eaca77e41817ce02875
As a result of the playbooks combined with the use of the
sqllite database file, the inspector functionality was partially
broken and could not be re-run without clean-up, in the form of
the sqllite database file being removed, between test runs.
Closes-Bug: #1554174
Change-Id: I77d33546dd97348298072b5047502b6705dfca11
We may not have python when we install. One example
is on Ubuntu 15.10 which only has python3. Ansible
currently requires python 2.6 or 2.7. Install it
via the package manager if it's missing.
Change-Id: Ic824f9cd131eca4d771c22d25d5646731ce78ae3
Partial-Bug: #1515071
It will run a play where it will spin up five vms, and will
truncate inventory to consider only three. The play will
configure dns using dynamic inventory, and will take an
extra parameter to specify if we want static ips or not.
It will perform a test on dnsmasq leases file, to ensure that
the dhcp behaviour is working as expected, depending on
the settings.
Change-Id: If56488f78524ceba201b50f76df306620744cda5
