Idea #11309
closed
[Crunch2] crunchstat-summary --container UUID should summarize container logs
Added by Tom Clegg about 7 years ago. Updated over 6 years ago.
Related issues
Updated by Tom Morris about 7 years ago
- Assigned To set to Tom Morris
- Target version set to Arvados Future Sprints
Updated by Tom Morris almost 7 years ago
- Tracker changed from Bug to Idea
- Target version changed from Arvados Future Sprints to 2017-07-05 sprint
Updated by Tom Morris almost 7 years ago
- Target version changed from 2017-07-05 sprint to 2017-07-19 sprint
Updated by Tom Morris almost 7 years ago
- Target version changed from 2017-07-19 sprint to 2017-07-05 sprint
Updated by Tom Morris almost 7 years ago
- Target version changed from 2017-07-05 sprint to 2017-07-19 sprint
Updated by Tom Morris almost 7 years ago
- Target version changed from 2017-07-19 sprint to 2017-08-02 sprint
Updated by Tom Morris almost 7 years ago
- Target version changed from 2017-08-02 sprint to 2017-08-16 sprint
Updated by Tom Clegg almost 7 years ago
- Assigned To changed from Tom Morris to Tom Clegg
Updated by Tom Clegg over 6 years ago
11309-crunchstat-summary-containers @ dc140f2c277a7571669f5bdf492bac6448a0fba0
I added "--container UUID" and "--container-request UUID" options to make it more obvious, but under the hood they're the same as "--job": they figure out the object type from the UUID.
I also dealt with #10359 because it was easy: added a --threads N option, default 8.
The report is unwieldy for a container with lots of "child" containers. Might be better to sort them by name (or creation time of container?) instead of UUID.
Updated by Tom Clegg over 6 years ago
11309-crunchstat-summary-containers @ f04ff34e47cc5ff1e625e9a5c1b1b0cbeb810d8c
Child containers are now sorted by creation time of container request.
Updated by Tom Clegg over 6 years ago
- Target version changed from 2017-08-16 sprint to 2017-08-30 Sprint
Updated by Lucas Di Pentima over 6 years ago
A couple comments:
- Failing test:
test_container (tests.test_examples.SummarizeContainer) - File
tools/crunchstat-summary/crunchstat_summary/summarizer.py- Line 437: Is there a case where
processcould be adict? If so, what happens with theuuidvariable?
- Line 437: Is there a case where
Updated by Tom Clegg over 6 years ago
Lucas Di Pentima wrote:
- Failing test:
test_container (tests.test_examples.SummarizeContainer)
Oops, updated fake API record to include created_at.
- File
tools/crunchstat-summary/crunchstat_summary/summarizer.py
- Line 437: Is there a case where
processcould be adict? If so, what happens with theuuidvariable?
This "pass already-retrieved object instead of just uuid" didn't end up getting used here but I was thinking it might end up being useful in #10472. But yes, it was broken; I've fixed it, and changed the arg to "process_or_uuid" to make it readable.
11309-crunchstat-summary-containers @ c77cb0091a8488eae52341920e8e32beb8fc47dc
Updated by Tom Clegg over 6 years ago
- Status changed from In Progress to Resolved