This is cleaned up as of commit
2351d624ae, and should be sufficient for
the final release. If something else with a release note lands, we can
make any changes there in a later patch.
Change-Id: Ia1ef442f3cc0d64c8a7c6fbb5aaf3f0160f214de
After set_bios_config is invoked, there is no way to check the progress
of the remote job on the DRAC card. list_unfinished_jobs on the
vendor-passthru interface provides a way to check its status.
Change-Id: I963504dbbec36e44312ccbf4455c45d6ec60908b
Closes-Bug: #1621515
First, the tested function signature was wrong. We didn't catch it in gate,
as we mock proliantutils, but it does break e.g. Debian package build.
Second, the arguments override was not actually working. We didn't catch
it in gate, because the new values were the same as the defaults.
Closes-Bug: #1626089
Change-Id: I2e4899e368b0b882dcd59bf33fdca98f47e5b405
For now we have to carry both in both trees, and they have gone out
of sync. This changes updates the copy here with two new commands.
Change-Id: Ife131cf825518012a3b04d2c7d713beeb4f8f770
This has never been needed by Ironic, and likely never will. It drags in
numpy, and makes the whole of Ironic that much more bloated.
The requirement was first added in
84a7dc47e2 and was presumably copied from
Nova whilst forking Ironic.
Change-Id: Ied4591e38cbe151b9c2fbca28ec9d006ed4afcfb
This adds a note that network security must be disabled, or certain
ports must be allowed, for provisioning and cleaning networks.
Closes-Bug: #1622727
Change-Id: I8415591d31209f8e3fbd9a4dcce30bd64bf8b24b
setUp will be automatically called around each
testcase, so this is to remove setUp that doing
nothing additional than super to keep code clean.
Change-Id: I8ce1dd0cc0cbba760caa6df6516b300622bb67fe
The commit 1027c1bc44
introduced a line that was already being done (line 1212).
This removes the duplicated line.
Change-Id: I0d05ecb1236dc68995f47204b686fe398b177f79
Partial-Bug: #1493328
Fixes the documentation to indicate that the notification
event types can have status 'error'. 'fail' is incorrect.
(11fcf6b710/ironic/objects/notification.py (L48))
Change-Id: I13c3f40430a4d7f8c07fdd9861b2d8712b8247a6
The constructor for the ipmitool classes are the same but they are
duplicated all over this patch is refactoring that code into a common
function that can be called from the classes constructors.
The VendorPassthru class wasn't checking for the 'timing' option support
of the ipmitool command, it should, this patch is fixing that too.
Change-Id: I456dbb9ef11230d722ff1b1fe5aa142237e0d187
This patch is changing the default bootloader for PXE + UEFI to Grub2.
Prior to this patch Elilo was the default but, Elilo is no longer
maintained with the last release dating 2014.
Ironic should have sane defaults therefore we should not rely on an
unmaintained project.
Change-Id: Ib34fcbd37e4698bf272582c96b1540b511b5a892
This will work for UEFI only or BIOS only images. It will not
work for hybrid images; which are capable of boot from BIOS
and UEFI boot mode.
Partial-Bug: #1493328
Change-Id: I9a0775a147d5bac711b58b4b6cf0135ec68509f7
The exception NotificationEventTypeError was used when an invalid
status was used. But it is not possible to assign an invalid status
without causing a ValueError.
Add a unit test to show that changing status to an invalid value does
raise a ValueError.
Remove check for valid status value.
Change-Id: I58fb50f963ca4a8e8d0185b9165c9bcda3bb7a25
This will cause them to log a warning at startup, that it is unsupported
and may be removed later.
The drivers marked unsupported are any driver that uses:
* SSHPower / SSHManagement
* NativeIPMIPower / NativeIPMIManagement
* seamicro.Power / seamicro.Management
* IBootPower
* SNMPPower
* VirtualBoxPower / VirtualBoxManagement
* AMTPower / AMTManagement
* MSFTOCSPower / MSFTOCSManagement
* WakeOnLanPower
See the release note in this change for the full list of the setup.cfg
names for the drivers.
The remaining drivers are tested via OpenStack CI, and/or have a
third-party CI listed here[0] and are making an honest effort toward
having stable CI.
[0] https://wiki.openstack.org/wiki/ThirdPartySystems
Closes-Bug: #1526410
Change-Id: I9c09be6b7a734426a4c0b18cb61ece5389428cd5
The notification spec is being updated to rename "phase" to "status" and make
it a required field. This updates the notification base code and docs to
reflect this modification. Version for the notification EventType base class is
also bumped.
Change-Id: I11b6ea44a01d2a830b592784ccc63a1de9bf2ec1
Depends-On: Ib25bd5a3a600c13f7b265d86ad253af3dfa5552f