#116 add support of QUEUED and RUNNING outcomes
Opened 2 months ago by bgoncalv. Modified 2 months ago
fedora-infra/ bgoncalv/ansible additional-outcomes  into  master

@@ -22,5 +22,6 @@ 

  

  

  # Extend the list of allowed outcomes

- # See: https://pagure.io/task-abicheck/issue/19

- ADDITIONAL_RESULT_OUTCOMES = ('CRASHED', )

+ # For CRASHED see: https://pagure.io/task-abicheck/issue/19

+ # For QUEUED and RUNNING see: https://pagure.io/fedora-infrastructure/issue/8989

+ ADDITIONAL_RESULT_OUTCOMES = ('CRASHED', 'QUEUED', 'RUNNING')

Alright, so while the change itself looks pythonic enough to me, I have two issues with the PR:

For these reasons, I would feel better if we could wait for after staging is back to merge this PR so we have a place where we can at least check that nothing is imploding before it goes to production :)

@pingou thanks, we have been without these outcomes for long time already, we can wait a bit more :) just ccing @bookwar and @mvadkert