Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ValueError: 'canceled' is not a valid TestingFarmResult #2339

Closed
sentry-io bot opened this issue Feb 11, 2024 · 1 comment · Fixed by #2340 or #2342
Closed

ValueError: 'canceled' is not a valid TestingFarmResult #2339

sentry-io bot opened this issue Feb 11, 2024 · 1 comment · Fixed by #2340 or #2342
Assignees
@mfocko
Labels
area/testing-farm Related to Testing Farm integration. gain/high This brings a lot of value to (not strictly a lot of) users. impact/high This issue impacts multiple/lot of users. kind/bug Something isn't working.

Comments

@sentry-io
Copy link

sentry-io bot commented Feb 11, 2024

Sentry Issue: PCKT-002-PACKIT-SERVICE-745

ValueError: 'canceled' is not a valid TestingFarmResult
(1 additional frame(s) were not displayed)
...
  File "packit_service/worker/tasks.py", line 556, in babysit_pending_tft_runs
    check_pending_testing_farm_runs()
  File "packit_service/worker/helpers/build/babysit.py", line 99, in check_pending_testing_farm_runs
    ) = Parser.parse_data_from_testing_farm(run, details)
  File "packit_service/worker/parser.py", line 1143, in parse_data_from_testing_farm
    result = TestingFarmResult(tf_state or "unknown")
  File "enum.py", line 384, in __call__
    return cls.__new__(cls, value)
  File "enum.py", line 702, in __new__
    raise ve_exc
@sentry-io sentry-io bot added area/testing-farm Related to Testing Farm integration. complexity/easy-fix gain/high This brings a lot of value to (not strictly a lot of) users. impact/high This issue impacts multiple/lot of users. kind/bug Something isn't working. labels Feb 11, 2024
mfocko added a commit to mfocko/packit-service that referenced this issue Feb 11, 2024
@mfocko
fix(tf): add the ‹canceled› state
887f56d 
Fixes packit#2339

Signed-off-by: Matej Focko <[email protected]>
@mfocko mfocko mentioned this issue Feb 11, 2024
Merged
@mfocko mfocko moved this from new to in-review in Packit Kanban Board Feb 11, 2024
softwarefactory-project-zuul bot added a commit that referenced this issue Feb 12, 2024
@softwarefactory-project-zuul
fix(tf): add the ‹canceled› state (#2340)
0678246 
fix(tf): add the ‹canceled› state

Fixes #2339

Reviewed-by: Laura Barcziová
@github-project-automation github-project-automation bot moved this from in-review to done in Packit Kanban Board Feb 12, 2024
@mfocko
Copy link
Member

mfocko commented Feb 12, 2024

Reporting should be also adjusted

@mfocko mfocko reopened this Feb 12, 2024
mfocko added a commit to mfocko/packit-service that referenced this issue Feb 12, 2024
@mfocko
fix(tf): add ‹canceled› as a neutral status
6adba48 
Fixes packit#2339

Signed-off-by: Matej Focko <[email protected]>
@mfocko mfocko mentioned this issue Feb 12, 2024
Merged
softwarefactory-project-zuul bot added a commit that referenced this issue Feb 12, 2024
@softwarefactory-project-zuul
fix(tf): add ‹canceled› as a neutral status (#2342)
38110e1 
fix(tf): add ‹canceled› as a neutral status

Fixes #2339

Reviewed-by: Maja Massarini
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mfocko mfocko

Labels
area/testing-farm Related to Testing Farm integration. gain/high This brings a lot of value to (not strictly a lot of) users. impact/high This issue impacts multiple/lot of users. kind/bug Something isn't working.
Projects
Packit Kanban Board
Archived in project
Milestone
No milestone
1 participant
@mfocko