summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Update .gitreview for bugfix/16.1bugfix-16.1-eolbugfix/16.1OpenStack Release Bot2020-12-301-0/+1
| | | | Change-Id: Id96fbf6f033faf97b33a87d28b2269436f3ff2ef
* Merge "Inject TLS certificate when using virtual media"16.1.0Zuul2020-12-196-5/+77
|\
| * Inject TLS certificate when using virtual mediaDmitry Tantsur2020-12-156-5/+77
| | | | | | | | | | | | | | | | A new option allows embedding a CA certificate in the virtual media ISO to allow fully secure TLS between ironic and IPA. Depends-On: https://review.opendev.org/763207 Change-Id: Idaacf44fd829c441d708b11704a97f9cd2b7a74c
* | Merge "Support port name"Zuul2020-12-1912-14/+199
|\ \
| * | Support port nameKaifeng Wang2020-11-2912-14/+199
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MAC address is not user friendly for port management, having a name field is also a feature parity with other resources. This patch implements db related change. Change-Id: Ibad9a1b6bbfddc0af1950def4e27db3757904cb1 Story: 2003091 Task: 23180
* | | Merge "Include HeartbeatMixin in the ramdisk deploy"Zuul2020-12-182-1/+7
|\ \ \
| * | | Include HeartbeatMixin in the ramdisk deployDmitry Tantsur2020-12-162-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise cleaning gets stuck forever because there is no heartbeat implementation to use. Change-Id: I3589038001b98c866f47339ac8d65edfb891ed79
* | | | Merge "Document that DHCP-less deploy does work with debian-minimal"Zuul2020-12-182-7/+12
|\ \ \ \
| * | | | Document that DHCP-less deploy does work with debian-minimalDmitry Tantsur2020-12-162-7/+12
| |/ / / | | | | | | | | | | | | Change-Id: I9ec4f9ceac0c5517ce59c6db3681ef9dc643ec14
* | | | Merge "Consistently use utils functions for policy auth"Zuul2020-12-1813-105/+47
|\ \ \ \
| * | | | Consistently use utils functions for policy authSteve Baker2020-12-1713-105/+47
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The check_policy function exists in api utils, along with other more complex policy utility functions. This change replaces direct calls to authorize with calls to check_policy. Having authorize calls consolidated in api utils may help with the upcoming secure-rbac work. Change-Id: If4779b08b9f360f4c2f4675c605aa519f6ea4778
* | | | Merge "Update outdated descripton for `default_boot_option`"Zuul2020-12-171-2/+2
|\ \ \ \
| * | | | Update outdated descripton for `default_boot_option`ericxiett2020-12-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default value of `default_boot_option` has already been 'local', so update the note. Change-Id: Ia846fa82121cda942697a240eec5bbd7bd93e68e
* | | | | Merge "remove lower-constraints in tox"Zuul2020-12-171-6/+0
|\ \ \ \ \
| * | | | | remove lower-constraints in toxlikui2020-12-171-6/+0
| | |/ / / | |/| | | | | | | | | | | | | Change-Id: If798d35d47bbc13d2e6e751467d16848c44d2975
* | | | | Merge "Remove detect_vendor decorator"Zuul2020-12-171-1/+1
|\ \ \ \ \
| * | | | | Remove detect_vendor decoratorJulia Kreger2020-12-161-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | And move a lock creation to vendor storage, which... should be okay. mgoddard reported issues in one of his projects where ironic was reporting that there were errors on the power sync process, ultimately due to the decorator from the very first iteration of the detect vendor code. Change-Id: Icd0dc3fe59a3815d6f78d881eb280f406e7d0c69
* | | | | Merge "Complete the REST API POST documentation"Zuul2020-12-178-38/+207
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Complete the REST API POST documentationSteve Baker2020-12-168-38/+207
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change adds missing POST request parameters and makes other minor corrections to the REST API documentation. Its possible that some of the parameters for /v1/nodes were never intended to be available for the POST call, but this docs change is to document what has been implemented, not the original intention. Change-Id: I1e35586d20bd3eda4d727931235340dd408c7508
* | | | | Merge "Remove lower-constraints job"Zuul2020-12-162-149/+0
|\ \ \ \ \
| * | | | | Remove lower-constraints jobRiccardo Pittau2020-12-152-149/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As discussed during the upstream ironic community meeting on Monday Dec 14 2020, the lower-constraints job is being removed. Change-Id: I116d99014a7bf77ca77b796ea3b759800dd808ce
* | | | | | Merge "Prepare release 16.1"Zuul2020-12-168-20/+39
|\ \ \ \ \ \
| * | | | | | Prepare release 16.1Dmitry Tantsur2020-12-148-20/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia37075d4aa2f39ca0862b03ca02c85bac17400e5
* | | | | | | Merge "Revert "devstack: build DIB images with CentOS Stream by default""Zuul2020-12-151-1/+1
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Revert "devstack: build DIB images with CentOS Stream by default"Dmitry Tantsur2020-12-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 05f2c8b79f0d6b7e9200bbc531ff621d2029da2e. It is being reverted as the centos stream images contain extra, un-necessary libraries and packages installed which swells the ramdisk size up substantially and is causing failures in CI as the compressed image size expanded by about 100MB, and uncompressed the stream images are 1.1GB. Change-Id: Icc3a18ed12d309fd9a00f02d5e703dfeda50e86b
* | | | | | | Merge "Fix release guide and include intermediate branches"Zuul2020-12-151-3/+11
|\ \ \ \ \ \ \
| * | | | | | | Fix release guide and include intermediate branchesRiccardo Pittau2020-12-151-3/+11
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the command to submit new releases and adds command reference for the intermediate bugfix branches. Change-Id: I79a039a6effcf8bd13e5c3ab5a231d5b515c8297
* | | | | | | Merge "Document the current status of the DHCP-less deploy"Zuul2020-12-152-0/+11
|\ \ \ \ \ \ \
| * | | | | | | Document the current status of the DHCP-less deployDmitry Tantsur2020-12-152-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I could not make the ramdisk side work on a variety of distributions, the same problem has been confirmed by another contributor. Until we get a working procedure for building a ramdisk, add an ugly warning to the documentation. Change-Id: I12100539b9987fcb47ba81c75ec96ed501d50c82
* | | | | | | | Merge "Rewrite DHCP-less documentation"Zuul2020-12-156-47/+91
|\ \ \ \ \ \ \ \ | |/ / / / / / /
| * | | | | | | Rewrite DHCP-less documentationDmitry Tantsur2020-12-156-47/+91
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current documentation is basically unreadable and contains a few factual errors. Rewrite it for simplicity and move to a separate file since at least 2 hardware types claim its support. The patch does not concern the current status of the feature, an easily revertable follow-up will be posted for that. Change-Id: I3404378333316b0736ce07610a1dbbd7847bac00
* | | | | | | Merge "CI: switch the multinode job to tempest-multinode-full-base"Zuul2020-12-152-3/+4
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | CI: switch the multinode job to tempest-multinode-full-baseDmitry Tantsur2020-12-142-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The non-base job is designed for the integrated gate and may have unnecessary side effects. It has recently overriding the OVS agent bridge settings, breaking our job. Make the job voting again. Change-Id: Ied8cafd32c3e634d498467ebe878a411f0b24e6d
* | | | | | | Merge "IPMI: Handle vendor set boot device differences"Zuul2020-12-145-16/+320
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | |
| * | | | | | IPMI: Handle vendor set boot device differencesJulia Kreger2020-12-146-16/+321
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Supermicro machines, when in UEFI mode, have a different device number, in binary, to represent the hard disk from other vendors such as Fujitsu which actually has somewhat similar code in their driver. This means we need to be somewhat cognizent of the vendor of the BMC and possibly update the device mapping based upon that vendor. This may ultimately fix a number of IPMI related problems, because there is a reliance upon the text output of ipmitool, which only reads the bytes retured by the BMC, which may not be reality after the next reset, espescialy if ipmitool doesn't know of the UEFI operating difference. Change-Id: Ie19db9e0cf1eafdfc9bb46248f4d457337821f94 Story: 2008241 Task: 41085
* | | | | | | Merge "Avoid a full install in tox environments that do not need it"Zuul2020-12-141-0/+6
|\ \ \ \ \ \ \
| * | | | | | | Avoid a full install in tox environments that do not need itDmitry Tantsur2020-12-141-0/+6
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usage of usedevelop=True results in all requirements.txt dependencies always installed (without constraints) even when requirements.txt is not explicitly added. Override usedevelop for these environments. Add requirements.txt to the 'docs' environment since it imports the whole ironic in the process. Change-Id: Ibab7d9cff1bd6eccc022873d1c5f59e27e54abaf
* | | | | | | Merge "Use ddt for existing policy tests"Zuul2020-12-141-103/+194
|\ \ \ \ \ \ \
| * | | | | | | Use ddt for existing policy testsSteve Baker2020-12-141-103/+194
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change aims to replicate the existing policy check tests using ddt so that they can be represented in a declarative format. The number of tests are expected to grow significantly with secure-rbac, so the aim of this change is to explore how best to represent the tests before the new tests are written. Change-Id: Id1b60f5525773eadbe1b85797eeefdb70d73ae1d
* | | | | | | | Merge "devstack: build DIB images with CentOS Stream by default"Zuul2020-12-141-1/+1
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | / / / | | |_|_|/ / / | |/| | | | |
| * | | | | | devstack: build DIB images with CentOS Stream by defaultDmitry Tantsur2020-12-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I50edd6b2740a26d00be19abc58c3ff770417fb68
* | | | | | | Do not pin Python version in tox configRiccardo Pittau2020-12-141-1/+1
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | We want to use the default for each system. Change-Id: I80e411e11b3b1db990ff91a1cf90c662170dcab6
* | | | | | Merge "Add secure boot support to ilo-uefi-https"Zuul2020-12-144-7/+29
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Add secure boot support to ilo-uefi-httpsvmud2132020-11-264-7/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds secure boot support to ilo-uefi-https boot interface. Change-Id: I1d08b88496764bbee5cf0a1d306eb7be31d0d373 Story: #2008258 Task: #41114
* | | | | | Merge "Fix lower-constraints with the new pip resolver"Zuul2020-12-134-10/+13
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | Fix lower-constraints with the new pip resolverIury Gregory Melo Ferreira2020-12-114-10/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * move pep8 dependencies from test-requirements to tox.ini, they're not needed there and are hard to constraint properly. * add oslo.cache to l-c to avoid bump of dependencies Change-Id: Ia5330f3d5778ee62811da081c28a16965e512b55
* | | | | | Add TLS troubleshooting guide entryDmitry Tantsur2020-12-101-0/+51
| |_|_|/ / |/| | | | | | | | | | | | | | Change-Id: Ied66562bb2475513ddb8c712dedc5f50fc6cad4f
* | | | | Merge "Test patching booleans with string values"Zuul2020-12-061-0/+32
|\ \ \ \ \
| * | | | | Test patching booleans with string valuesSteve Baker2020-11-271-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The baremetal client encodes boolean patch values as strings ("True", "False") but there is no unit test coverage which confirms that this actually works. This change adds that test coverage. Change-Id: I9e428ad973e88d3e1ef1e04e49a7b00a4e2d43fd
* | | | | | Merge "Improve object_to_dict arguments"Zuul2020-12-066-59/+63
|\ \ \ \ \ \ | |/ / / / /