summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Enforce upper-constraints when building ramdisksliberty-eol1.0.5stable/libertyMathieu Mitchell2016-08-305-4/+99
| | | | | | | | | | | | | | | | | | | | | | | 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)
* Merge "Fixes programmatic error in _install_grub()" into stable/libertyJenkins2016-08-251-3/+2
|\
| * Fixes programmatic error in _install_grub()Nisha Agarwal2016-05-091-3/+2
| | | | | | | | | | | | | | | | | | The patch defines the variables before they are used in the code. Closes-bug: 1555547 Change-Id: I27c5e926cf4001211c4edd794a09b26559c63f71 (cherry picked from commit bb605784dd5570310dedfdbd1b0d8a4ecf967e07)
* | Use constraints for all the thingsJim Rollenhagen2016-08-092-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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)
* | Fix unwanted unmounting of /usr/share/oem1.0.4Brad Morgan2016-06-211-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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)
* | Updated from global requirements1.0.3OpenStack Proposal Bot2016-04-291-1/+1
|/ | | | Change-Id: I6da6f7605e4bd27b802ca67de9a4ed64502743a1
* Install qemu-image from backports repovsaienko2016-04-151-4/+3
| | | | | | | | | | | | | | | | | | | | | | | 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)
* Correct link to enabling agent driversJim Rollenhagen2016-04-011-1/+1
| | | | | | | | | 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)
* Fix full_trusty_build once and for allJim Rollenhagen2016-03-301-7/+9
| | | | | | | | | | | 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)
* Append BRANCH_PATH to filenames of build outputJim Rollenhagen2016-03-291-1/+19
| | | | | | | | | | 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)
* Merge "Updated from global requirements" into stable/libertyJenkins2016-03-241-1/+1
|\
| * Updated from global requirementsOpenStack Proposal Bot2016-01-271-1/+1
| | | | | | | | Change-Id: Ibaa2716f15dfdd7884cd017e5351b823bd1076f9
* | Merge "determine tgtd ready status through tgtadm" into stable/liberty1.0.2Jenkins2016-03-222-34/+26
|\ \
| * | determine tgtd ready status through tgtadmdparalen2016-03-212-34/+26
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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)
* | Fix vfd mount for capitalized device nameNaohiro Tamura2016-02-083-7/+49
|/ | | | | | | | | | | | | 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)
* Merge "Add release note for 9353adda59d7fe9d85465ef195b82a2437814e39" into ↵1.0.1Jenkins2016-01-081-0/+4
|\ | | | | | | stable/liberty
| * Add release note for 9353adda59d7fe9d85465ef195b82a2437814e39Jim Rollenhagen2016-01-081-0/+4
| | | | | | | | | | | | | | | | | | | | 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
* | Merge "Fix "logs" inspection collector when logs contain non-ascii symbols" ↵Jenkins2016-01-072-4/+15
|\ \ | |/ |/| | | into stable/liberty
| * Fix "logs" inspection collector when logs contain non-ascii symbolsDmitry Tantsur2015-12-092-4/+15
| | | | | | | | | | | | | | | | | | 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)
* | Merge "Add reno for release notes management" into stable/libertyJenkins2016-01-0611-0/+311
|\ \
| * | Add reno for release notes managementJim Rollenhagen2016-01-0411-0/+311
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (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
* | | Updated from global requirementsOpenStack Proposal Bot2015-12-161-1/+1
|/ / | | | | | | Change-Id: I3ae1adaee1c0252aa8393143e0ff57fa4e80872f
* | pyudev exception has changed for from_device_fileDmitry Tantsur2015-12-082-3/+16
|/ | | | | | | | | | | | | 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)
* Updated from global requirementsOpenStack Proposal Bot2015-10-191-2/+2
| | | | Change-Id: I23ed0151728cb2f6313f2888015ca4cb50fc5810
* Set up stable/liberty branchDoug Hellmann2015-10-062-1/+1
| | | | | | | | 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
* Updated from global requirements1.0.0OpenStack Proposal Bot2015-10-021-1/+1
| | | | Change-Id: Ia5982574312b48a9e96a9deaaec163dcf75e637f
* Merge "Add "logs" and "extra-hardware" inspection collectors"Jenkins2015-10-024-0/+146
|\
| * Add "logs" and "extra-hardware" inspection collectorsDmitry Tantsur2015-10-014-0/+146
| | | | | | | | | | | | | | | | | | 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
* | Use doc8 style checkerRuby Loo2015-10-013-10/+13
|/ | | | | | | | | | | 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
* Merge "Make the erase_devices clean step abortable"Jenkins2015-09-261-1/+8
|\
| * Make the erase_devices clean step abortableLucas Alvares Gomes2015-09-261-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Merge "Fixes config drive creation failure in UEFI boot mode"Jenkins2015-09-242-7/+37
|\ \
| * | Fixes config drive creation failure in UEFI boot modeShivanand Tendulker2015-09-232-7/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Fix default ironic api port numberNaohiro Tamura2015-09-231-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Merge "Add more info to checksum exception"Jenkins2015-09-224-14/+23
|\ \
| * | Add more info to checksum exceptionJosh Gachnang2015-09-214-14/+23
| | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Expose serial, wwn and vendor on the BlockDevice objectDmitry Tantsur2015-09-213-98/+181
| | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Create a SerializableComparable class"Jenkins2015-09-184-5/+85
|\ \ \
| * | | Create a SerializableComparable classJohn L. Villalovos2015-09-114-5/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Updated from global requirementsOpenStack Proposal Bot2015-09-172-3/+3
| | | | | | | | | | | | | | | | Change-Id: I9ade8873a2aa239f7605040385171189af86f01b
* | | | Merge "inspection: prepare for future deprecations"Jenkins2015-09-172-30/+66
|\ \ \ \ | |_|/ / |/| | |
| * | | inspection: prepare for future deprecationsDmitry Tantsur2015-09-162-30/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Merge "Added home-page value in setup.cfg file"Jenkins2015-09-161-0/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Added home-page value in setup.cfg filevenkatamahesh2015-09-161-0/+1
| | | | | | | | | | | | | | | | Change-Id: Ie7d171e891f57fc7f8b92de1761ff7ef6ec4aaa9
* | | | Merge "Follow-up to inspection patch 096830414b"Jenkins2015-09-164-4/+32
|\ \ \ \
| * | | | Follow-up to inspection patch 096830414bDmitry Tantsur2015-09-164-4/+32
| | |/ / | |/| | | | | | | | | | Change-Id: I7ec05e501ec40802efa14cabe14752972919c7a9
* | | | Fixes incorrect description URLMario Villaplana2015-09-151-3/+3
| |/ / |/| | | | | | | | | | | | | | | | | | | | 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
* | | Adds project resources links to Readmelekhajee2015-09-141-0/+11
|/ / | | | | | | | | | | | | Includes links for launchpad, wiki and dev docs. Closes-Bug:1495668 Change-Id: Ic0559d50425b3ab68fe34d9e88e78c599c29100a
* | Install "dmidecode" and "ipmitool" by defaultLucas Alvares Gomes2015-09-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | Merge "Add support for inspection using ironic-inspector"Jenkins2015-09-1011-15/+735
|\ \
View Lorry Controller Status