Arvados

To work around the CI build issue where it installs the published Python SDK, rather than the SDK from the branch being tested, I've split this into two branches, both of which are ready for review, but which need to be merged separately.

15554-python-sdk-constants @3c248f44a39ab0b792a8a91f875c4e09875bf54f just adds the three Python SDK constants
15554-crunchstat-summary-live-log @b3cf6fae4df6d9254a2dfa71a3ba555f4633127f has been rebased on top of it and force pushed

Tom Morris wrote:

15554-crunchstat-summary-live-log @b3cf6fae4df6d9254a2dfa71a3ba555f4633127f has been rebased on top of it and force pushed

It seems as though summarizer.py ln 109 passes ProcessSummarizer a string, but it doesn't look like it can handle it unless I am missing something. If I am correct, we should probably fix it and add test coverage for that case. Other than that it LGTM!

Eric Biagiotti wrote:

It seems as though summarizer.py ln 109 passes ProcessSummarizer a string, but it doesn't look like it can handle it unless I am missing something. If I am correct, we should probably fix it and add test coverage for that case. Other than that it LGTM!

Good catch! I've fixed the bug, but didn't create a new test because it's a Crunch 1 only issue.

I gave up on trying to get the Python SDK dependencies to work with Jenkins. I switched back to a hardcoded string. Not ideal, but I've wasted waaaay too much time on this already.

15554-crunchstat-summary-live-log-no-sd @0c3e58f58b9fbcbbea69c7856ce70f73cbfbd73a

tests pass locally and running on Jenkins here: https://ci.curoverse.com/view/Developer/job/developer-run-tests/1516/

Updates on the 15554-crunchstat-summary-live-log-no-sdk branch LGTM. Some Jenkins tests failed, but re-ran them locally without issues.

Merged @ b1f01918cfcd953e906d32691ddf76c0253a1948