Project

General

Profile

Actions

Bug #19594

closed

arvados-user-activity tool crash

Added by Lucas Di Pentima 4 months ago. Updated about 2 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
10/06/2022
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

We got the following report from one of our clusters:

Traceback (most recent call last):
  File "/usr/bin/arv-user-activity", line 8, in <module>
    sys.exit(main())
  File "/usr/share/python3/dist/python3-arvados-user-activity/lib/python3.7/site-packages/arvados_user_activity/main.py", line 219, in main
    getCollectionName(arv, e["properties"].get("collection_uuid"), e["properties"].get("portable_data_hash")),
  File "/usr/share/python3/dist/python3-arvados-user-activity/lib/python3.7/site-packages/arvados_user_activity/main.py", line 101, in getCollectionName
    if len(uuid) != 27:
TypeError: object of type 'NoneType' has no len()

Subtasks 1 (0 open1 closed)

Task #19595: Review 19594-user-activity-fixResolvedPeter Amstutz10/06/2022

Actions
Actions #1

Updated by Lucas Di Pentima 4 months ago

Updates at 88f14b140 - branch 19594-user-activity-fix
Test run: developer-run-tests: #3310

  • Checks for uuid to not be None before attempting to get its length.
  • Improves some code formatting for better readability.

Note for the reviewer: The issue was happening on pirca

Actions #2

Updated by Peter Amstutz 4 months ago

Lucas Di Pentima wrote in #note-1:

Updates at 88f14b140 - branch 19594-user-activity-fix
Test run: developer-run-tests: #3310

  • Checks for uuid to not be None before attempting to get its length.
  • Improves some code formatting for better readability.

Note for the reviewer: The issue was happening on pirca

This LGTM, thanks for fixing!

Actions #3

Updated by Lucas Di Pentima 4 months ago

  • Status changed from In Progress to Resolved
Actions #4

Updated by Peter Amstutz about 2 months ago

  • Release set to 47
Actions

Also available in: Atom PDF