Merge lp:~doanac/utah/bug1158743 into lp:utah/trunk
Proposed by
Andy Doan
Status: | Rejected |
---|---|
Rejected by: | Javier Collado |
Proposed branch: | lp:~doanac/utah/bug1158743 |
Merge into: | lp:utah/trunk |
Diff against target: |
135 lines (+80/-10) 4 files modified
debian/changelog (+3/-0) tests/test_process.py (+44/-0) utah/process.py (+25/-0) utah/provisioning/baremetal/inventory.py (+8/-10) |
To merge this branch: | bzr merge lp:~doanac/utah/bug1158743 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Javier Collado (community) | Approve | ||
Max Brustkern (community) | Approve | ||
Review via email: mp+155264@code.launchpad.net |
This proposal supersedes a proposal from 2013-03-22.
Description of the change
This should add the logging Javier was suggesting to understand what processes are actually using a machine.
Changed to proper branch
To post a comment you must log in.
Unmerged revisions
- 272. By Andy Doan
-
add changelog entry
- 271. By Andy Doan
-
fix pep257 issues
- 270. By Andy Doan
-
move pid _in_use to process.py
As per Javier's review comment this moves the function to become a
generic helper. I also added some unit-testing logic for it - 269. By Andy Doan
-
add logging for unavailable machines
this should help us understand who is using a machine in the event
one isn't available.
I see that you fixed the bug in a branch based on trunk and created the merge
for dev (which actually reveals an entry in debian/changelog that isn't in
dev). Is this the expected way to work with bugs?