Subtask #4990
closedFeature #4994: Teuthology test suite failure to assign pass/fail needs to be recorded
Logging of Teuthology runs passing and failing should be more user friendly
0%
Description
Methods to update database entries from the teuthology log information need to be implemented
Updated by Anonymous almost 11 years ago
- Target version deleted (
v0.67 - Dumpling)
Updated by Anonymous almost 11 years ago
I believe 3792 is an implementation detail of this problem.
Updated by Anonymous almost 11 years ago
3792 has been closed as a detail. Xstudio support should be considered when implementiong this fix.
Updated by Anonymous over 10 years ago
The changes proposed in 4994 may take care of this issue. (see my update done at roughly the same time as this message).
Updated by Anonymous over 10 years ago
- Estimated time set to 0:00 h
A copy of the changes that I have made so far are in origin/wip-teuthupddb-wusui. This can be reviewed/scanned but this is not ready for primetime yet. The writing of entries is running into some errors (I think that I need to set the size of my output data differently) and the pretty print operation is still quite ugly.
Updated by Anonymous over 10 years ago
- Status changed from New to Fix Under Review
- Assignee changed from Anonymous to Alfredo Deza
- Start date set to 08/28/2013
Updated by Ian Colle over 10 years ago
- Assignee changed from Alfredo Deza to Zack Cerza
Updated by Zack Cerza over 10 years ago
In the future, when we assign a ticket to someone for the purposes of review, let's be polite and link to the pull request or branch:
https://github.com/ceph/teuthology/compare/wip-teuthupddb-wusui
Updated by Zack Cerza over 10 years ago
Oh. This is something both Alfredo and I have been reviewing:
https://github.com/ceph/teuthology/pull/56
Our comments don't seem to be visible outside the pull request, though. Warren, I think it's because you're squashing commits every time you push. That's making it really hard to follow your changes.
Updated by Anonymous over 10 years ago
Okay. This is a brand new branch with this change implemented. This does not use the http pushing and posting mechanism that teuthology_lock did, and so I believe that it is much cleaner. It also does not have the issues where sys.path was being modified.
The wip branch that I just pushed is wip-db_results-wusui
Updated by Anonymous over 10 years ago
https://github.com/ceph/teuthology/pull/86 is the pull request
Updated by Anonymous over 10 years ago
https://github.com/ceph/teuthology/compare/wip-db_results-wusui is the wip comparison
Updated by Anonymous over 10 years ago
A new and improved version has been pushed.
Updated by Zack Cerza over 10 years ago
- Status changed from Fix Under Review to Resolved
I reimplemented this - it was merged here: https://github.com/ceph/teuthology/commit/f28643c39758fab98cf3149e18bedaeb006a431f/