| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, building ramdisks installs ironic-python-agent without any
upper-constraints. This causes the package to be installed with newer,
untested dependencies.
This commits introduces a tool to generate a local upper-constraints
file based on predefined strategies (below). Additionally, the fallback
to the openstack/requirements uses the URL defined in tox.ini instead of
redefining it. This prevents having to keep track of two separate
variables when releasing.
upper-constraints lookup strategies (in order):
* UPPER_CONSTRAINTS_FILE points to a local file
* UPPER_CONSTRAINTS_FILE points to a URL
* /opt/stack/new/requirements/upper-constraints.txt
* upper-constraints.txt from openstack/requirements git repository
Partial-bug: #1616554
Change-Id: Ib5c0c57cafdb6ffd7456e61f3b1bb5fa57520e5a
(cherry picked from commit a0ca6ce157b8a84f282d8abbdd27c75f48d991ff)
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The patch defines the variables before they are used in the
code.
Closes-bug: 1555547
Change-Id: I27c5e926cf4001211c4edd794a09b26559c63f71
(cherry picked from commit bb605784dd5570310dedfdbd1b0d8a4ecf967e07)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This configures tox to use upper-constraints for everything, except
environments used for post jobs (as constraints aren't supported there
yet).
This also doesn't enable constraints for our image builds, as it isn't
possible to pass an environment variable into the Dockerfile to allow
folks to choose their own constraints file (and we don't want to force
people to rely on git.o.o for building images).
Closes-Bug: #1611528
Change-Id: Ic78db2ce5ab516606cbe63c1e35a4d6394367f16
(cherry picked from commit 4e35b0ee64a249c8461efc1c4374dd6cd9253220)
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Creating specific partitions will cause a udev event to trigger,
causing systemd to attempt to mount the new partition at /usr/share/oem.
This mount will not succeed, but will unmount /usr/share/oem,
causing ironic-python-agent to die.
Masking this unit disables this behavior.
Change-Id: Ifea3482f36a354cbf988e6f6003f4284c5c2a21d
Closes-bug: #1592163
(cherry picked from commit da0a8aff88e67fce7603b03a220b77b451349e7d)
|
|/
|
|
| |
Change-Id: I6da6f7605e4bd27b802ca67de9a4ed64502743a1
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
qemu-image has been installed from testing to fix issues when
qemu-img can't convert image:
qemu-img: error writing zeroes at sector 0: Invalid argument
The issue has been fixed in qemu 2.2.0-rc2, reference [0].
Which has been added to debian backports.
This patch fixes issue with building coreos image, when qemu-utils
can't be installed from testing due to dependency problems.
Reference:
[0] http://git.qemu-project.org/?p=qemu.git;a=commitdiff;h=f3a9cfddae
[1] https://bugs.launchpad.net/cinder/+bug/1389728
Change-Id: Ie62a88e1d012aba1392b19c18137529c4c3ebf31
Related-Bug: #1490086
Closes-bug: #1570090
Closes-Bug: #1569835
(cherry picked from commit 52066d0a4920d2ccb3c6aef165e4a843ede4a809)
|
|
|
|
|
|
|
|
|
| |
The current link points to a section of the docs that is just a pointer
to another section of our docs. Change this to point directly to the
correct docs, saving users an extra click.
Change-Id: I6814fda9292921aeedb90041563b3dbb997de9c6
(cherry picked from commit 99f70370b7cbe7a89a2d23ec32d872bac36745ae)
|
|
|
|
|
|
|
|
|
|
|
| |
After finally being able to test this locally, I've found that the
'cgroup-lite' package is what we were missing, not lxc.
Additionally, the output is written by root, not the current user, so
use sudo when modifying/writing files in imagebuild/coreos/UPLOAD.
Change-Id: I8cf5ce5bd5345ccea5e8869f2628f8616d0fe758
Closes-Bug: #1559383
(cherry picked from commit 473b6b4661480e0a2e2c363553e48d62c7145272)
|
|
|
|
|
|
|
|
|
|
| |
This adds $BRANCH_PATH (from the dependent patch) to the file names of
the build output, so that we can upload images for each branch we
maintain.
Change-Id: I8c767fa1b56a566c1acabab8552a51ffeb4e1d7d
Depends-On: I540d2f139c80bb7f382315258dbe3a7a14be74c8
Partial-Bug: #1561756
(cherry picked from commit 7862d1458097e26148f11210d0458ec4968ce7f5)
|
|\ |
|
| |
| |
| |
| | |
Change-Id: Ibaa2716f15dfdd7884cd017e5351b823bd1076f9
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The intention of the iscsi extension _wait_for_iscsi_daemon function
is to wait till the tgtd process boots completely before running any
iscsi commands through the tgtadm command line utility. The tgtd
process utilises a unix socket for communication with the user and the
_wait_for_iscsi_daemon function asserts the tgtd process status based
on the socket presence. The socket name/path to use is not specified
by iscsi extension configuration but rather falls-back to a
(system-wide) default value[1]. Since tgtd version 1.0.60, this path
changed from /var/run/tgtd.ipc_abstract_namespace to
/var/run/tgtd/socket[2] and now causes a false-negative while waiting
for the tgtd process to boot[3].
This patch follows implementation suggestion from the bug report[3] to
avoid relying on the socket presence but rather to check the tgtd
status by trying to communicate through the tgtadm tool.
[1] https://github.com/openstack/ironic-python-agent/blob/
5bbb9ded082f2cfde5e8877e9f33294eb93a5fb7/
ironic_python_agent/extensions/iscsi.py#L47 [2]
https://github.com/fujita/tgt/commit/
d1aa4dcfd691c9409dbad5db49ce6754ce9c1b5a
[3] https://bugs.launchpad.net/ironic-python-agent/+bug/1505923
Closes-Bug: #1505923
Partial-Bug: #1551250
Depends-On: I4c19c96d2d5cd8accb9ed8df6241fc9f0c229e0e
Change-Id: If3284e0b441fe6c2da507640c08a4bbefd9dad2b
(cherry-picked from commit fb920f41ed181c7ec74e3367f2735bccce07e574)
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
This patch enables virtual media deploy even if the expected virtual
floppy device name "/dev/disk/by-label/ir-vfd-dev" is capitalized to
"/dev/disk/by-label/IR-VFD-DEV".
This change is necessary because some bare metal hardware such as
FUJITSU RX1330 series create the capitalized virtual floppy device
name.
Change-Id: I436b3f9524a83814e082f8dd421df3e53b33f792
Closes-Bug: #1541167
(cherry picked from commit 73f81f298218b394e8e8be23d5ec59da809613c2)
|
|\
| |
| |
| | |
stable/liberty
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This release note was not backported with the change, as the change
landed before we started doing reno.
This is a partial cherry-pick of
b9df861eb78d2494f528c1fbc52f584ef2a31e6d.
Change-Id: I72fc2f52f06faec20d2981820c93f3f4d4891abd
|
|\ \
| |/
|/|
| | |
into stable/liberty
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Somehow it didn't pop earlier. Updated tests to contain some creepy
russian letters :)
Closes-Bug: #1517913
Change-Id: I4c6712ea1e813d1f0f0d0aedaccfa1187526e0ec
(cherry picked from commit 5fa258b7087e610625e0277efb197f87db08f1ea)
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
(cherry picked from commit e8488c253499e77a00f7363abaf4a7ca68e41f8e)
Add mitaka reno page
As Dmitry pointed out here: https://review.openstack.org/#/c/258727/
we need a page without a branch specified.
(cherry picked from commit ffcdcd4019c3df9d0a62034e1973a01be09ae1c2)
Squashed, because there's no reason for this to be two commits.
Change-Id: Id6bfe132dcc8fc3efbe0d96820eebf7d174b2736
|
|/ /
| |
| |
| | |
Change-Id: I3ae1adaee1c0252aa8393143e0ff57fa4e80872f
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
Now pyudev raises DeviceNotFoundByFileError which does not inherit
from EnvironmentError, so our 'except' block in hardware.py no longer
catch the exception. It broke unit tests, but it can also potentially
break the deploy.
This patch updates hardware.py to catch both old a new exceptions.
Change-Id: Iaefd6089f6f766a241054d8e132b2f3098c8130d
Closes-Bug: #1522756
(cherry picked from commit 2fc6ce22f8b531d33bb3d7f7876dd42276231797)
|
|
|
|
| |
Change-Id: I23ed0151728cb2f6313f2888015ca4cb50fc5810
|
|
|
|
|
|
|
|
| |
Update default branch in .gitreview and remove the version setting from
setup.cfg to make pbr understand that it can use post-versioning on the
branch.
Change-Id: I3e4d602d0fb0fa4db4ea6257ab2035b9ac4a0cc3
|
|
|
|
| |
Change-Id: Ia5982574312b48a9e96a9deaaec163dcf75e637f
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This is a port of downstream inspector ramdisk plugins we found helpful.
* logs - sends journald logs with inspection data.
* extra-hardware - uses hardware-detect utility to collect bigger
hardware inventory and to run benchmarks.
Change-Id: If05402606c45185d618279eef46e68c51209f82b
|
|/
|
|
|
|
|
|
|
|
|
| |
This uses the doc8 style checker to check that document files
conform to certain requirements, as documented at
https://pypi.python.org/pypi/doc8.
The README was modified because it had several lines that were
were longer than 79 characters, and some links were fixed.
Change-Id: Ia21b680ec227357ada58326fea104696561475d5
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patches updates the get_clean_steps() method to make the
erase_devices step abortable. Erasing devices is something that can be
cancelled without damaging the machine.
When a clean step is aborted the provision state of the Ironic node
will go to CLEANFAIL state. The operator can then do what is needed to
fix the problem (i.e network booting issues) and restart the cleaning
later on.
Partial-Bug: #1455825
Change-Id: Ic181ac3712810c6f6925e8b627ee79e77ecf4d83
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This fix enables creation of config drive for UEFI only whole disk
images.
It will not work with the hybrid images that support booting in BIOS
and UEFI boot mode.
Change-Id: Ib4dd8c082a50e1dbaf0df91477b062716cb780ff
Closes-Bug: #1486887
Depends-On: I81400305f166d62aa4612aab54602abb8178b64c
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This patch corrects the default ironic api port number to 6385 from
6835. The default value of 'ipa-api-url' is not used in production
environment since conductor and bare metal node have its own IP
address, but could be used in development environment.
Therefor this patch fixes this trivial error.
- default=APARAMS.get('ipa-api-url', 'http://127.0.0.1:6835'),
+ default=APARAMS.get('ipa-api-url', 'http://127.0.0.1:6385'),
Change-Id: Idb85e95b354a68111f94e8cddd5c1b6227e676ef
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
If an image cannot be downloaded for some reason, it is helpful for
operators to have the image path, checksum, and calculated checksums
available easily from the API.
Change-Id: I6a2fb46726245cebd730b5c51d4f25f8465f1658
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Currently we only use these disk properties for root device hints.
However, they'll be really useful for inspector, especially for also
implementing root device hints.
Change-Id: I48aa6b6d2d198d16f2f8e387970f7230066cf8a2
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Create a SerializableComparable class derived from the Serializable
class.
Added the following functions to the SerializableComparable class:
'__eq__'
'__ne__'
Disable the '__hash__' function in the SerializableComparable class as
some derived classes are mutable.
Use the SerializableComparable class in hardware.py and
extensions/base.py
This should make unit testing users of the class easier when doing a
self.assertEqual() or self.assertNotEqual()
Added some initial unit testing for encoding.py
Change-Id: If0f14b3bfe7f1391f65dd730a16a534afed0da82
|
| | | |
| | | |
| | | |
| | | | |
Change-Id: I9ade8873a2aa239f7605040385171189af86f01b
|
|\ \ \ \
| |_|/ /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
1. cleanly separate deprecated and non-deprecated properties
2. add root disk to inspection data, so that we can have a proper
fallback when root device hints are not given.
Change-Id: Ie19b82ff2a914873ff4b2395b02643e086b934b1
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | |
| | | |
| | | |
| | | | |
Change-Id: Ie7d171e891f57fc7f8b92de1761ff7ef6ec4aaa9
|
|\ \ \ \ |
|
| | |/ /
| |/| |
| | | |
| | | | |
Change-Id: I7ec05e501ec40802efa14cabe14752972919c7a9
|
| |/ /
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
A GET to /v1 in the API returns a link to a URL for a repository that is no
longer maintained. This commit changes the link to point to the correct
documentation.
Change-Id: I8779cd1840627ff8759527b7514306293682aaae
|
|/ /
| |
| |
| |
| |
| |
| | |
Includes links for launchpad, wiki and dev docs.
Closes-Bug:1495668
Change-Id: Ic0559d50425b3ab68fe34d9e88e78c599c29100a
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The get_memory() method relies on dmidecode to get the total amount of
RAM present in the node but, but by default dmidecode is not installed
in the image.
Same for get_bmc_address() and ipmitool.
This patch fixes both cases by installing the "dmidecode" and "ipmitool"
packages when generating the CoreOS image of IPA.
Closes-Bug: 1494367
Change-Id: Ibded7bd533d604cf21a3b60afd396dc7b8b8750b
|
|\ \ |
|