This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.

This is a mechanically generated change to replace openstack.org
git:// URLs with https:// equivalents.

This is in aid of a planned future move of the git hosting
infrastructure to a self-hosted instance of gitea (https://gitea.io),
which does not support the git wire protocol at this stage.

This update should result in no functional change.

For more information see the thread at

 http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html

Change-Id: I28efa8961cb37a022b1a8558402f2eeaf022c846

On centos7.2, it doesn't happen every time, but sometimes, partprobe
fails with error `Device or resource busy` in _fix_gpt_structs. In fact,
this is not a problem, I tried ignoring this error and ran remain code.
Things went well. The target of running `partprobe` is to check GPT
table, we can run `sgdisk -v` instead.

This patch is to replace `partprobe` command in _fix_gpt_structs to
avoid `Device or resource busy` error.

Change-Id: I43182f17b1c6229132814313f7582ab30245f6bb
(cherry picked from commit b9a109c6)

When using volumes, and having empty ones, the volumes are failing to
mount when using conv=sparse with dd . This is causing security
hardened volumes :
http://git.openstack.org/cgit/openstack/tripleo-image-elements/tree/elements/overcloud-secure/block-device-default.yaml

To fail when mounted via iscsi

This reverts commit ab0492a3.

Story: #2003755
Task: #26441
Change-Id: I03f53a50ada9aabe9e3fbb71106d5190ae110e01
(cherry picked from commit 3234f432)

Currently the search for config drive is just done for config-2
in lowercase. If we pass the label with vfat, the label will
be in uppercase by convention. So the comparison will fail, and
this needs to be case insensitive to work.

Change-Id: I1dd196841fc3599333d68459eac606bd3b104b7b
Story: #2003547
Task: #24834
(cherry picked from commit 65b3df1a)

This is a mechanically generated patch to complete step 1 of moving
the zuul job settings out of project-config and into each project
repository.

Because there will be a separate patch on each branch, the branch
specifiers for branch-specific jobs have been removed.

Because this patch is generated by a script, there may be some
cosmetic changes to the layout of the YAML file(s) as the contents are
normalized.

See the python3-first goal document for details:
https://governance.openstack.org/tc/goals/stein/python3-first.html

Change-Id: Ia0cac32cd9f490ba6ba9fd8aa43e2e5cc61f7c52
Story: #2002586
Task: #24302

Whole disk images with GPT partition tables would encounter
errors after the initial deploy as secondary copy of the
GPT partition table did not match the first table. This is
something we explicitly need to act upon for ironic.

Original commit message:

As suggested by TheJulia, the `_fix_gpt_structs` code
which is currently not accessible/callable from outside of ironic-lib
should be exposed so that other code like ironic-python-agent
can use it such as https://review.openstack.org/#/c/568524/

Continuation of backport commit message:

To enable backporting of this patch such that we can address
the bug that it fixes in older releases, the method has been
preserved as a hidden method with a leading underscore.

Change-Id: Icd824982d23966605fb179270bbf428cab23a41d
(cherry picked from commit e4f16012d98907c76e0994a0999c3332c41c8ead)

We do not need tox_install.sh, pip can handle constraints itself
and install the project correctly. Thus update tox.ini and remove
the now obsolete tools/tox_install.sh file.

This follows https://review.openstack.org/#/c/508061 to remove
tools/tox_install.sh.

Change-Id: Ic2615a261e433ce7ec8c0ef1d9c2061286a05739
(cherry picked from commit b5923cf7)

Zuul no longer requires the project-name for in-repo configuration.
Omitting it makes forking or renaming projects easier.

Change-Id: Ie349dda7d6c247e6bb9791259c494c7d4f412bfd

When using whole disk images, one generally wants the *virtual*
size of the image to be close to the size of the overcloud nodes'
physical disks (since no partition/logical volume/filesystem
expansion is done). As an example, one might end up with a 4GB
overcloud image (QCOW file) that has a virtual size of 100GB or
more.

Ironic uses qemu-img to convert this QCOW file to a "raw" image,
which is a *sparse* file. When the image is copied to an overcloud
node's disk (via iSCSI), we currently write all 100GB (mainly of
zeroes) across the network to the node's disk.

Adding conv=sparse to the dd command makes it skip the "holes" in
the image file; it will only the write the portions of the image
that actually contain data across the network.

Closes-Bug #1743651

Change-Id: Ief3688b210c3b19ce8be45c5f9571b7ba6e79127
(cherry picked from commit ab0492a3)

Use the tempest plugin from openstack/ironic-tempest-plugin as we
have moved the tempest code there. Soon the tempest code will be
deleted from openstack/ironic.

Change-Id: I4b0ca6da3ca546b05ae68716ca34405c1f7f7329
(cherry picked from commit bcc08885)

Create a new base job: legacy-ironic-lib-dsvm-base

Define 'irrelevant-files' in legacy-ironic-lib-dsvm-base and remove
them from project.yaml. This means we only define 'irrelevant-files'
once. Also makes project.yaml much easier to read.

Fix the 'irrelevant-files' section to make sure we do test when
changes are made to 'requirements.txt'.

Sort the jobs lists in project.yaml.

Conflicts:
    zuul.d/project.yaml

Change-Id: I0534be1ff4ac4d3d2e40f282966683c7b60fc7ba
(cherry picked from commit bff1d934)

This patch moves the legacy ironic-lib CI jobs into the ironic-lib
tree instead of storing them in openstack-infra/openstack-zuul-jobs
to give us control as we migrate the legacy jobs to the new ansible
roles.

Change-Id: Id9b0c22b73135ca0e9a952e909e8295550ad899f
(cherry picked from commit 5ab2d5c6)

Change-Id: I12547bc73edfb7f1ba20828f62d535f6bbb06cac

Change-Id: I82959556ef3edc67a316137a1bc51987938bbc67

Change-Id: I7eb18504bdbd6453114d2cd48f1bd0bb66a1feb0

Change-Id: I43eb0e3a0c79fdbef3e971410503c700678dcbfa

Change-Id: I1fea1922376f5dcb1c1efa2da20e33c82ba64d2d

Change-Id: I3fbf025ece33e47a57db4c6442e2f2636ba49c69

Change-Id: I52055e7b3c0ed8f56d05ca624339b7e9aca19222

Change-Id: I5c5b575324f71a87936036e8f4c16009fd9eaaf1

Config option 'backend' with parameter choices =['noop', 'statsd'],
that means oslo.config will ensure value is one of them, otherwise
a ValueError will be raised, so don't need handle and test config
option 'backend' with invalid value.

Change-Id: Ibda809416c565d57f19179d7f13f4cee8f8145ff

Change-Id: I10d320ebb5d47f00d2e7a200676d1ca28915f667

Change-Id: I30576714cd6195c3ec98daa5c81ff9527247d901

This change removes the now unused "warnerrors" setting,
which is replaced by "warning-is-error" in sphinx
releases >= 1.5 [1].

[1] http://lists.openstack.org/pipermail/openstack-dev/
2017-March/113085.html

Change-Id: Ic311ad8dc0ea211acdce770fb1f1726994f1c1a4

NVMe drives use different device naming convention for partitions:

> lsblk /dev/nvme0n1
NAME        MAJ:MIN RM  SIZE RO TYPE MOUNTPOINT
nvme0n1     259:0    0  1.8T  0 disk
|-nvme0n1p1 259:2    0  1.3G  0 part
`-nvme0n1p2 259:1    0   64M  0 part

The ironic-python-agent fails to provision NVMe drives,
because it does not expect the 'p' letter before the partition number.

This patch fixes that.

Change-Id: I5061f6d5b00f404d4aaadc6b18675039ba2ed854
Closes-Bug: #1695265

In testcase, setUp will be called automatically. This patch used to
remove setUp functions that do nothing. Besides, it will keep code clean.

Change-Id: I015e4faa2e185a49609078c281f449c65a0d8b27

Change-Id: Iad2e7193d963ce628fcef11183caa323cdfffc38

Apparently, wipefs still leaves some metadata around, which can break ceph.
Use sgdisk's zapping feature to be absolutely sure.

Change-Id: I51aea8fb5ba02d63a05a3794405caf000ff28e75
Closes-Bug: #1690458

Change-Id: I6d2eb4ba997c8f7d208a689acc99925a19ba23bf

This change introduces a new base test class that mocks out
utils.execute() and similar functions and forces an exception if it
gets called. If utils.execute() is mocked by the test case then it
will work. What this does is prevent un-mocked access to
utils.execute() / processutils.execute() and similar subprocess
library functions.

Inspired by Julian Edwards' patch to ironic-python-agent

Change-Id: Ie5bda8f4c65cf4dfb40b91c11b61485c954addde

Change-Id: Ic1215a9ef301e5db92f660f91a6c823e64e1b853

Change-Id: Ibbd5852508f4b20aa988516103e1caf50ddd3f21