Bug 479387
| Summary: | [RFE] store timestamps in journal | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Ales Zelinka <azelinka> |
| Component: | beakerlib | Assignee: | Petr Muller <pmuller> |
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | low | ||
| Version: | rawhide | CC: | jhutar, ohudlick, pmuller, psplicha, qa-errata-list |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | beakerlib-1.9-3.fc20 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2014-07-30 06:58:08 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 555302 | ||
| Bug Blocks: | 1116308, 1116317 | ||
|
Description
Ales Zelinka
2009-01-09 09:13:23 UTC
It's true that timestamps can be useful (especially when debugging multihost tests). I think the best solution could be to include timestamps both in journal and stdout but omitting them in rlJournalPrintText output. In this way we will keep the final report brief for quick glance and if we need some more details we can have a look into TESTOUT.log or the journal file. What about an option to the rlJournalPrintText, something like --with-timestamps? You don't need this in all tests, but you know this could be useful when you write one, so you could use this option to provide this sort of information to users of your test... > What about an option to the rlJournalPrintText, something like
> --with-timestamps?
Even better!
+1 for this
+1 I'm willing to prepare/review patch if Petr is not working on something already. Implemented in new journal. Driven by the environment variables: based on them, timestamps are either collected or not into the journal XML. When the underlying XML does have the timestamps collected, textual output can be created from it with or without the timestamps. The mechanism is intended to be consistent with other collecting/output use cases and wrapped with tools, so we wont bother users with such granularity. https://github.com/petr-muller/beakerlib/commit/8fbfa67d59ff25ceb2fb27c1ef556f3a87384eef https://github.com/petr-muller/beakerlib/commit/c5ab92976ed240145b7d4cd27234881c8ff51ff3 https://github.com/petr-muller/beakerlib/commit/d78cd1b76d7596153d0fbd0a8c98786fc8ece973 https://github.com/petr-muller/beakerlib/commit/256f725fd899ab1fc0a5eb200204b7cae8056192 https://github.com/petr-muller/beakerlib/commit/08823214c2a3490594fb782a585d68aab1e6327e *** Bug 563893 has been marked as a duplicate of this bug. *** beakerlib-1.9-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/beakerlib-1.9-1.fc20 Package beakerlib-1.9-1.fc20: * should fix your issue, * was pushed to the Fedora 20 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing beakerlib-1.9-1.fc20' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2014-7442/beakerlib-1.9-1.fc20 then log in and leave karma (feedback). beakerlib-1.9-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/beakerlib-1.9-2.fc20 Please consider the package fixing this bug available in Fedora stable repos once bz1116308 is closed and RHEL stable repos once bz1116317 is closed. Fixed in: beakerlib-1.9-2.fc19 beakerlib-1.9-2.fc20 beakerlib-1.9-2.fc21 beakerlib-1.9-3.el5 beakerlib-1.9-2.el6 beakerlib-1.9-2.el7 beakerlib-1.9-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/beakerlib-1.9-3.fc20 beakerlib-1.9-3.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report. |