2960f71c64a52e9f4daf9bbb8d4fd06e923b213c
453 Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
|
Jay Faulkner
|
2960f71c64 |
Minor fixes to image build
- Correct directory to match the relative path correctly - Include additional requirements.txt for coreos image injector Change-Id: Ie68f86220f2db786e3e97e72fed288b92704d028 |
||
|
Jenkins
|
304be37239 | Merge "Use configurable driver name in lookup URL" | ||
|
Jim Rollenhagen
|
46e9776d42 |
Use configurable driver name in lookup URL
Allow configuration via command line arguments or kernel parameters. Default to agent_ipmitool, the reference driver. Depends on https://review.openstack.org/#/c/84795/12 Change-Id: I55c4a8713308d038002a6567471cd862bf89ec76 |
||
|
Jenkins
|
00a299cc80 | Merge "Add FlowExtension" | ||
|
Jenkins
|
952ade37f6 | Merge "Kill the build_agent function" | ||
|
Russell Haering
|
8422118fa4 |
Kill the build_agent function
This used to actually do things, now it just wraps the constructor. Lets kill it. Change-Id: I320684f8f52d032a1755460eacc70f5aeee92d7a |
||
|
Alexander Gordeev
|
18aa51f1f1 |
Add FlowExtension
It makes possible of running data-driven flow which's the list of the other existent extensions to be called. Change-Id: Ib73ea4da92f291c872b7ae51e46ecc7fdd45ee16 |
||
|
Alexander Gordeev
|
bd19a03f92 |
Add self to validator method
This patchset allows access to extension class instance for more advanced command validation. Change-Id: I9d6df1ab44ab06ef2e663ad777ad3560d9ab7565 |
||
|
Alexander Gordeev
|
5bc793bcb0 |
Resolve circular import, introduce ExecuteCommandMixin
Adding new methods to utils won't work as the circular import appears between errors.py and utils.py. Introduce ExecuteCommandMixin and use it for IronicPythonAgent and FlowExtension (in future patch). Also add tests for its. Change-Id: Id95b31349292a7967d2ee66ec82c1662d8e5de94 |
||
|
Jenkins
|
88f06e2831 | Merge "Use # instead of """ for copyright blocks" | ||
|
Jenkins
|
7d4621eb13 | Merge "Implement version 2 payload with more hw info" | ||
|
Jenkins
|
40d2982b2d | Merge "Use a released version of Pecan" | ||
|
Jay Faulkner
|
1384d79d2c |
Implement version 2 payload with more hw info
Bumping the payload version to two, and adding CPU, Memory, and Block Device information to the inventory. Added dependency on psutil for available memory and cpu_count. Requirements line copied from global requirements Change-Id: Ia39c85c91b1d60468667787a7978020084dc6c2a |
||
|
Jenkins
|
bbe50749fc | Merge "Add BackOffLoopingCall with jitter" | ||
|
Jim Rollenhagen
|
05ff7b802d |
Use a released version of Pecan
Patch was upstreamed, now we can use 0.5.0. Yay. Change-Id: I6810aa1c0d967347c6ea8a247ca36c12d5ced492 |
||
|
Jay Faulkner
|
6e619f0261 |
Deprecate hardware._cmd in favor of utils.execute
This removes the custom function that was in place before our move to Openstack in favor of the utils method integrated with oslo. Change-Id: If668e53ccc97743b00c4beae39d153ee63ed015a |
||
|
Jenkins
|
dd79f5acb5 | Merge "Properly mock _cmd so blockdev does not run" | ||
|
Jay Faulkner
|
1653c558ef |
Close file handle after use
The file handle for addr_info was left open, meaning we were leaking file handles when sending system information. This ensures the handle is closed. Change-Id: I4b117d4f9ec32ad3f16f43632ffe9dc96da5ab7f |
||
|
Jay Faulkner
|
b43261c815 |
Properly mock _cmd so blockdev does not run
blockdev --report was being run with the previous way this test was mocked. This fix causes it to not be run. Change-Id: I7ab5dcca0a3a89ef4c62a2f40981958de73cd6f1 |
||
|
Jim Rollenhagen
|
3c1d52cbb1 |
Use # instead of """ for copyright blocks
Reformats copyright messages to be comments rather than docstring-style blocks. Change-Id: I4d863f53b67bb49d03bda0952b9e6179b6d23c59 |
||
|
Jim Rollenhagen
|
b826a72a7b |
Version node lookup payload
This payload may change over time and should be versioned. Corresponds to https://review.openstack.org/#/c/85228/ Change-Id: I58b1d69a2c41ff105038178632e289cb4b12c971 |
||
|
Jenkins
|
7554b78d76 | Merge "Move split_command to utils" | ||
|
Jay Faulkner
|
c121bef9f0 |
Compatibility fixes for Python 3.3
1) Added a py33 environment to tox 2) Updated tests to mock the correctly named builtins.open based on python version 3) Other minor compatibility fixes Tests for Python 3.3 will not pass due to this bug: https://github.com/eventlet/eventlet/issues/83 among possibly others in eventlet. Change-Id: Ie4b512a926fa690ee77a71a89851c871ea1f6be0 |
||
|
Josh Gachnang
|
bd25174338 |
Add BackOffLoopingCall with jitter
Using DynamicLoopingCall involved a few hacks to make it work properly. This new BackOffLoopingCall will start an exponential backoff (with a configurable jitter) when there is a failure. The backoff will continue until the given function returns True or timeout is about to be exceeded. The function will run indefinitely until either an exception is raised or timeout is reached. I plan to merge this into oslo loopingcall and switch the heartbeat to this. Change-Id: I1482348e98c6b68c34b3003645029e08135b1341 |
||
|
Alexander Gordeev
|
9171ce659d |
Move split_command to utils
This patch moves _split_command from IronicPythoAgent class to utils module. Also fixes import looping. Change-Id: Ibf2b0b3885286b9ad78db64cf9e195de4ad627ad |
||
|
Jenkins
|
cae81837ce | Merge "Added execute util" | ||
|
Jenkins
|
7164b4220b | Merge "Add timeout param for execution_thread.join" | ||
|
Vladimir Kozhukalov
|
7736de66ca |
Added execute util
It is needed to run OS commands. It uses oslo processutils. It is just a copy of the same method in openstack/ironic except using rootwrap was removed. Change-Id: I2efede22b1fa25febe91879c0fefcdfc7f3d1dd5 |
||
|
Alexander Gordeev
|
160c0775f7 |
Add timeout param for execution_thread.join
Having timeout is necessary for the FlowExtension. Change-Id: I0b8b937595cd6daa64f6a69dc6b1042bba2724c8 |
||
|
Josh Gachnang
|
1a87ebfcce |
Adding oslotest to improve debugging
oslotest includes things like showing log messages/stdout on test errors, which makes debugging much easier. Also moved mock to second group in imports, as it's a 3rd party library. Change-Id: I016ae0a376d42dec28085687ea7194df4cd44eda |
||
|
Vladimir Kozhukalov
|
74ba66568e |
Added some infrastructure tools
Copied generate_sample.sh and with_venv.sh scripts from openstack/ironic. Generated sample config. Change-Id: I1beab119499156c711d838a3a1ecc3316b655900 |
||
|
Vladimir Kozhukalov
|
51bb147e47 |
Added some oslo modules
Added some oslo modules which are needed for generating config samples and for running os commands. Re-synced oslo from b7ad6ddab8b1d61bf4f52ccaa461a9d68809747b which is master. Change-Id: I6e35ba1f0df007876f4dc25e7e2bd553986bfa8d |
||
|
Vladimir Kozhukalov
|
826edb688f |
Synced oslo modules
Re-synced oslo from b7ad6ddab8b1d61bf4f52ccaa461a9d68809747b which is current master. Change-Id: I9a8c4a71541381d00e90011cd6d95645546764b7 |
||
|
Jim Rollenhagen
|
9cfa63d2d0 |
Add kernel parameter support
Allow the agent to read arguments from the kernel command line. Priority is: agent command line, kernel command line, defaults. Change-Id: Idfd43a8b7fdf6c368cf55d45b32cb7bcfbb56212 |
||
|
OpenStack Jenkins
|
32fe6e14c9 |
Updated from global requirements
Change-Id: I835e8f7438bdaab718e0a9910880819cab961936 |
||
|
Jenkins
|
517616938e | Merge "Adding DynamicLoopingCall around lookup" | ||
|
Josh Gachnang
|
3bc4866a21 |
Adding DynamicLoopingCall around lookup
Currently, if a single lookup call to the Ironic API fails, the entire agent errors out and restarts. This allows the agent to retry for a set amount of time before throwing an uncaught exception forcing a restart. Change-Id: I39752fb3f42ad3e4f15a49194f1554e1d3463cf8 Closes-Bug: 1297019 |
||
|
Jenkins
|
30b6d8ed5a | Merge "Remove advertise-host from service file" | ||
|
Jim Rollenhagen
|
2b6f531052 |
Remove advertise-host from service file
The regex used is environment-specific, let's just not pass this argument for now (it's optional). Change-Id: Ia5194ecbb7ec5425ee902338631559f8415308f2 |
||
|
Alex Gaynor
|
a3cab12393 |
Added PyPy env to tox.ini
Also fixes tests that were broken on PyPy. Co-Authored-By: Jim Rollenhagen <jim@jimrollenhagen.com> Change-Id: I2554aa844b040c4c74cbf287dccb06d816a39dbe |
||
|
Jim Rollenhagen
|
6329ae46db |
Fix URL for node lookup
Change-Id: I5b128a74bf0442f61d722978308c03c6ec45f8d8 |
||
|
Alexander Gordeev
|
aee1555156 |
Get rid of modes. Introduce pluggable extensions
Allow multiple extensions to be loaded by switching to ExtensionManager from stevedore. Remove any reference to modes. Change-Id: Ic160478625226b4dd17bd68b3d37f3b05823e519 |
||
|
Josh Gachnang
|
55ea7b8edd |
Merge pull request #73 from rackerlabs/JoshNang/heartbeat
Getting the heartbeat from Ironic |
||
|
Josh Gachnang
|
d8c0f1b794 | Actual exception message, stylistic changes | ||
|
Josh Gachnang
|
234d209729 | Removing commented out code | ||
|
Josh Gachnang
|
6dc09f3148 | Getting the heartbeat from Ironic instead | ||
|
Josh Gachnang
|
0e6776f6c0 |
Merge pull request #72 from rackerlabs/JoshNang/ipa
Rename to Ironic Python Agent |
||
|
Josh Gachnang
|
fc043dd1d3 | Fixing import order | ||
|
Josh Gachnang
|
ececa3bce2 | Doing replacement on imagebuild | ||
|
Josh Gachnang
|
63fa74582d | Renaming things in Dockerfile |