Per review feedback on
https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/798143
it is believe the setting is no longer required for Wallaby branches.

Change-Id: I9face242ac4501a37a36534dd05eba66466fcccc

At this point, it is relatively safe to remove the
train jobs for ironic-tempest-plugin. The branches
are EOL, and we're unlikely to make any changes which
testing against Train, that can't be identified or
revealed in later community supported stable branches.

Change-Id: I069a9ec15d94871c7b610e130b13e9403110f728

The min version of tox is updated to 3.18.0 to replace whitelist_externals
with allowist_externals option [1]

[1] https://github.com/tox-dev/tox/blob/master/docs/changelog.rst#v3180-2020-07-23

Change-Id: I3715bdc7079d6a8754be514a2131a3dae06769c2

Change-Id: Id0d91c09f3d9f6cb5207b02ec7ebf3d2075f366d

The ironic tempest plugin was an early plugin and manually
invoked override plugin clients and then attached them in
the setup_clients method. However, the newer format is to
use get_service_clients, which creates and attach client
classes using the prepared credentials supplied by the
credentials attribute on the test classes.

In order to support even the most basic testing handling
and testing of Scope Enforcement as part of Secure RBAC,
then the we need to leverage the newer (last 3-4 years)
model of instantiating and leveraging clients in tempest.
This is because we need to be able to get a system scoped
admin token to be able to test actions as a system scoped
admin user. Not to be confused with "admin", which is
project scoped.

This newer style of client support does necessitate some
legacy style or direct client invocations to be retooled
so they do not attempt to directly invoke without the
required context.

Additionally, to support even the most basic handling of
the Secure RBAC's effort, we need to be able to know
when to leverage *and* then leverage that client.
We do that through the enforce_scope parameter
in upstream tempest.

Depends-On: https://review.opendev.org/c/openstack/tempest/+/798130
Change-Id: I5188fc756f1b524e9d1b32ef0474e29a9cf90b57

Setuptools v54.1.0 introduces a warning that the use of dash-separated
options in 'setup.cfg' will not be supported in a future version [1].
Get ahead of the issue by replacing the dashes with underscores. Without
this, we see 'UserWarning' messages like the following on new enough
versions of setuptools:

  UserWarning: Usage of dash-separated 'description-file' will not be
  supported in future versions. Please use the underscore name
  'description_file' instead

[1] https://github.com/pypa/setuptools/commit/a2e9ae4cb

Change-Id: I651339c8103cb46f7d4eb0107bb300a6da29dbb1

A long time ago, in a galaxy far far away, the use of
/extra/vif_port_id was deprecated in favor of vif attachment and
detachment. This change updates a test which was using the
/extra/vif_port_id field value to bind a vif, and then list/detach
it using the vif attach/detach/list API interfaces.

Since the action is to add/list/detach, this seems appropriate and
should be widely supported going back 4+ years.

Change-Id: I0b0cd0aaf9a87803386e096ab768638476f9ca52

Update minversion of tox to 3.9.0 to support inline comments [1]

Move pep8 and coverage requirements to tox.ini

Fix typo in coverage job

[1] https://tox.readthedocs.io/en/latest/changelog.html#v3-9-0-2019-04-17

Change-Id: I3f39b25ee0fd44c1bc51f94a7fbee74de8cba65d

Different implementations of ping behave slightly differently
in order to ensure the packet payload is transmitted across
the wire because some mediums have a minimum packet size.

Typically this results in a frame size of 64 bytes, but
the client may report 56 when the header is removed and only
the packet payload is examined.

But the point of the check is that we got something back.

The below edited for example purposes:

$ ping -c1 -w4 192.168.133.92
PING 192.168.65.92 (192.168.133.92) 56(84) bytes of data.
From 192.168.65.249 icmp_seq=1 Destination Host Unreachable

--- 192.168.65.92 ping statistics ---
1 packets transmitted, 0 received, 0 errors, 100% packet loss, time 3075ms
$ ping -c1 -w4 192.168.65.1
PING 192.168.65.1 (192.168.65.1) 56(84) bytes of data.
64 bytes from 192.168.65.1: icmp_seq=1 ttl=64 time=12.4 ms

And then in CI, we're able to see this be 56 bytes returned
becasue the underlying medium may not have been ethernet or
the client has stripped the header. Which, doesn't matter,
we get A reply from the expected address and we can validate
the test on that alone.

Change-Id: If0d51b52e96f280336d0d92135681db75a6fbd65

The ironic-standalone-redfish-victoria job is using latest tinyipa
based on tinycore 11.x, but still running on bionic.
That causes issues during the ipa-ramdisk build because of kernel
too old in bionic, so setting nodeset to focal.

Change-Id: Ia0fbc97184a4098635085edbdd6d7be5dfd2218c

We need to explicitly tear down the raid sets and not just
remove the hint because the code will try to use the raid
set if it is there.

Depends-On: https://review.opendev.org/#/c/759906
Change-Id: I4c6829833889e03588d5723dadb94338696c9122

Versions of hacking from 3.1.0 until 4.0.0 NOT included are supposed
to require a safe version of flake8, so besides normale issues related
to minor version upgrades, we can safely use those versions.
Also forcing pycodestyle versions to be compatible with flake8 installed
by hacking.

Change-Id: Idb66f2ae0105308747cdf3046b09a22bdf691ea3

This commit adds test case to support UEFI-HTTPS
boot using ilo5 driver with standalone ironic.

Change-Id: Ib54fa2e79800bb56ff14699e8ce4fbc460615670

Stop testing stein in wallaby and adjust other jobs to start
testing explicitely victoria branch.

Change-Id: I3c8be006e0ea136690abb70df7c5e4e561fdff20

There is a race sometimes when power off persists after the allocation
query, causing it to fail[1].

[1] https://zuul.opendev.org/t/openstack/build/aab3ecc1f63f4a52b1015f726fd8d229

Change-Id: I2b854a3a6b0f40f46dfa030b84d58715572f3ad4

Visually indented line with same indent as next logical line.

Change-Id: I9204313f2b8b52929366b00f5ac035ce06603122

Adds a new feature flag to enable it instead of cleaning time.

Make the standalone-redfish job voting since it covers some
of the tests that are not covered by the ipmi one.

Story: #2006963
Task: #40462
Change-Id: Id2e376e38771700d295f632e84ab3e32710d8180

As recommended, since version 3.2.0 tox switches pip invocations
to use the module -m pip instead of direct invocation.
We set min version to 3.2.1 [1] to also fix the behavior of
--parallel--safe-build

[1] https://tox.readthedocs.io/en/latest/changelog.html#v3-2-1-2018-08-10

Change-Id: I7a791c8239ae54b74e714145a69e9af00c96bc6f

Change-Id: I4e86e6264a4457b36ff94c8f817294ffd764c48d

flake8 new release 3.8.0 added new checks and gate pep8
job start failing. hacking 3.0.1 fix the pinning of flake8 to
avoid bringing in a new version with new checks.

Though it is fixed in latest hacking but 2.0 and 3.0 has cap for
flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also
break the pep8 job if new check are added.

To avoid similar gate break in future, we need to bump the hacking min
version.

- http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html

Change-Id: I90fa5ab8085a07c3f19b32cfdbbeb6bfe3953087

- test_baremetal_basic_ops now can test scenarios
for partition/wholedisk images.
- added a new configuration for tempest `boot_mode`.
- if boot_mode is uefi the test_baremetal_basic_ops
will run a verification to make sure the VM is using
uefi.

Change-Id: I920675b8a61a65c850b245283c3613b03f33ee6a

Change-Id: Icbc092e85e9211c409f32767412915a3c7651dc3

Switch to openstackdocstheme 2.2.1 version. Using
this version will allow especially:
* Linking from HTML to PDF document
* Allow parallel building of documents
* Fix some rendering problems

Update Sphinx version as well.

Set openstackdocs_pdf_link to link to PDF file. Note that
the link to the published document only works on docs.openstack.org
where the PDF file is placed in the top-level html directory. The
site-preview places the PDF in a pdf directory.

Change pygments_style to 'native' since old theme version always used
'native' and the theme now respects the setting and using 'sphinx' can
lead to some strange rendering.

openstackdocstheme renames some variables, so follow the renames
before the next release removes them. A couple of variables are also
not needed anymore, remove them.

See also
http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014971.html

Change-Id: I263e2109e455ee98a940cb34356e37473413cad4

Since we split the tests let's add ironic-standalone-redfish jobs
in the check phase

Depends-On: I1d2c3c2b4fe2a6eda64895bf73a6f57175c192b6
Change-Id: Ia1036d8a7fbb5dcf4bec07802d83198c477fbec4

Now we have stable/ussuri branch ready for devstack
and so does for all service projects.

This commit adds the stable/ussuri job on tempest plugin
master gate.

Change-Id: Id758a4b9a47261abc7a37abb87b1137e15d2b451

Update docs building:
* Switch to sphinx-build
* Update requirements for Sphinx and openstackdocstheme for
  python 3, create doc/requirements.txt file.
* Remove unneeded doc and translation sections from setup.cfg
* Remove install_command, it's unneeded, the default is fine.
* Use TOX_CONSTRAINTS instead of obsolete UPPER_CONSTRAINTS

Change-Id: I6ff4427014fc967a6f7b28fc6cbe5a0f4eca601c

If ip_version_for_ssh == 6 then use a IPv6 range for
the tenant network. The range used (fd00:33::/64) should
be routed to the gatway IP for for the router configured
in tempest.conf ("public_router_id").

Use dhcpv6-stateless for IPv6 addressing as
o using iPXE with DHCPv6 has problems as dnsmasq wont
  hand out IP addresses when the DHCPv6 clid or iaid
  changes
o the cirros image doesn't seem to support DHCPv6.

Change-Id: Id67be61534f1384af1a539ef257b0b85d4fff180