Third in a series of commits to add Codespell to Ironic Repos. This commit adds the Tox Target to CI
A future commit could potentially add a git-blame-ignore-revs file if their are lots of spelling mistakes that could clutter git blame.
Change-Id: I43818291edfde18c217ab4acb4d951b8a85f6fcd
Since ironic-inspector is pending deprecation and does not see a lot of
changes, simply disable the jobs that do not pass. A release note is
issued about the status of the project.
Also: Re-adjust the memory footprint of the VMs since we use
tinycore linux on this job, *and* we were previously introsepcting
4 VMs at 1GB each. The count has been changed to two.
Change-Id: I11d10a9f83552b5875812fa65d87cc5c72081537
This was merely obscuring a bug in pbr. setuptools doesn't do the
auto-discovery when pbr is in use. Remove it.
Also fixes the environments list where it is now validated for what is
installed by removing functional-py38, and fixing the job to just be
"functional".
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Depends-on: https://review.opendev.org/c/openstack/pbr/+/869082
Change-Id: Ife999840777f3c553f569d81cd826ef9d5a6765d
This is an automatically generated patch to ensure unit testing
is in place for all the of the tested runtimes for antelope. Also,
updating the template name to generic one.
See also the PTI in governance [1].
[1]: https://governance.openstack.org/tc/reference/project-testing-interface.html
Change-Id: I49f40f464b9b029927d2fc7ec33de5e554661d9e
Sync with ironic configuration to fix grenade job
Drop lower-constraints.txt and its testing (was [1])
Fix ironic-inspector-tempest-managed-non-standalone sine we're here
[1] https://review.opendev.org/c/openstack/ironic-inspector/+/840075
Change-Id: I9b08c6e61c39a0e5616e51256f76a19dc5fd00c0
Since we run inspector jobs in ironic gate, we need to
include `ironic/tests` in the irrelevant-files so
we don't run the jobs when there is only changes in unit tests.
Change-Id: I98757ab78716689355f70c7735480d4dc8e04320
The lower-constraints test was removed because of an issue where pip
could not correctly determine the required packages versions to install,
ending in an almost infinite loop that would end up in timeout, failure,
and general mayhem.
Recently the issue has been fixed and, if properly configured, the
lower-constraints test can provide good indication of which minimum
versions are required to support the current code.
This patch adds the test back to the current development branch, and it
runs only on master.
The lower-constraints file will stay in the future stable branches.
Change-Id: I2e247ff2d68705d04d40c7ea653a8d3e0daf17d8
Since pymemcache 3.4.0, localhost will first translate to ipv6 ([::1])
if possible instead of ipv4 (127.0.0.1). Since memcached is running in
the ipv4 interface, we need to point tooz to the right endpoint.
Depends-On: https://review.opendev.org/c/openstack/requirements/+/772684
Signed-off-by: Moisés Guimarães de Medeiros <moguimar@redhat.com>
Change-Id: Iddf558cd598e41ffead11e164ce2fb1720d15f34
As discussed during the upstream ironic community meeting on
Monday Dec 14 2020, the lower-constraints job is being removed.
Change-Id: I943b63b9c6a980011b35bb028c64bee18d772750
The pymemcache lib was updated recently in upper-constraints to 3.4.0
and since then the ironic-inspector-tempest-managed-non-standalone job
has been broken.
There seems to be an issue between tooz memcache driver and pymemcache,
the change in upper-constraints has been reverted [1] and the failing
version blacklisted in global-requirements.
Re-enabling ironic-inspector-tempest-managed-non-standalone as it
will now pass correctly.
[1] https://review.opendev.org/760103
Change-Id: I4944860c265f9e5060c8ab276e7962a93a27da88
The pattern of adding a column and then reading a table with it
no longer works in SQLAlchemy 1.3.20. This has been reported
upstream [1] and apparently we're doing the wrong thing by reusing
a column object twice.
We need to disable the non-standalone job since it's really broken
now, and this fix is blocking bifrost.
[1] https://github.com/sqlalchemy/sqlalchemy/issues/5669
Change-Id: I2fb07413e8f421f39b24acf1272771ee2097b195
Currently ironic-base defaults to 2 and our tests try to introspect
all of them. This puts unnecessary strain on the CI systems, return
the number back to 1.
Change-Id: I178190ee1e9d99e60994a307044105f306d4cab3
Uncompressing the ramdisk on a single core CPU can take a
substantial amount of time and elongate the overall initial
period between VM start and OS becoming operational.
Additionally, multiple CPUs allows for the software with-in
the job to execute more quickly with multiple running processes.
Change-Id: I814b12cbc05c6a110dc5593113b948e3f89e80c1
- Removed ironic-inspector-non-standalone-tempest
- add the non-standalone case to ironic-inspector-tempest-managed and
make voting in check and gate.
- Renamed ironic-inspector-tempest-managed to match the new scenario.
- Keep a copy of ironic-inspector-tempest-managed
Change-Id: Icf9406c2b9e2829a36fa37e49b2692a76d67bdb6
The default value is inherited from ironic-base.
Depends-On: I8496ccc32c633e167be2083448d4f267865f07fa
Change-Id: Ib1985274bbc679a621d5a68f9bcf5491512913cd
The same timeout values are shared between different jobs and
they should just be the default for the base job.
Change-Id: Ic41fe7ade21ece2d1110ef8a6ccb74df6ddd2b6f
Most of the failures in this job is because tempest couldn't
finish the test on a given time, we increased the introspection
timeout but we forgot to allow tempest to run for more than
20 min (default)
Change-Id: I25435fe93f2f6d6cd50befe9cdf6bb1e548cef88
The job is often failing during introspection because of a timeout error,
so increasing the timeout value to prevent multiple rechecks.
Change-Id: I4da76ec3cf7b82f7aed30c087bbfe31cbea91f45
Remove the value of IRONIC_RAMDISK_TYPE when present to make
the base job and all children take the type from ironic-base
which now is set to use the dib image.
Change-Id: I59f00c0b135f209ca6258efe9c3e3eb722fff4cb
- Add ipa-builder to grenade
- Disabled smoke tests since they are not lightweight when
using Ironic.
- Using test_baremetal_introspection for the tempest run
pos-upgrade.
- added LIBVIRT_STORAGE_POOL_PATH since some cloud providers
may not have enough disk space.
- lowered the number of VMs to 3 (we don't have many tests)
Change-Id: Ifee10541f369d5bc850ce3d92305810cd3eedd34
An upcoming ironic change changes the default boot option for nodes
to local booting as opposed to the long standing default netboot.
Some jobs still use cirros, which cannot be local booted from a
partition image as it is designed to write the ramdisk contents
to the partition upon the initial network boot.
Change-Id: I0d0c043de449048c3cc38ed7d7dac3e9e85974ef
The ironic-inspector-base job is based on ironic-base job which
will default to dib.
To prevent breaking the ironic-inspector CI we set the ramdisk
type to tinyipa for now and we'll convert the jobs here later.
Change-Id: If2be23d4e59f4c4821005b037a6002428b2e7eb4
Takashi Kajinami
OpenStack Release Bot