Uploaded image for project: 'DC/OS'
  1. DC/OS
  2. DCOS_OSS-4877

Waiting for job run to be finished, but history for that job run is not available

    Details

    • Sprint:
      Tools-Infra Team Sprint 42
    • Story Points:
      1
    • Transition Due Date:

      Description

      PR https://github.com/dcos/dcos-test-utils/pull/62 attempted to fix a problem where a job finishes but the history is not yet ready (DCOS_OSS-4294).

      However, it does not work as described.

      The function is wrapped with a retry decorator and raises an exception if the above case occurs - https://github.com/dcos/dcos-test-utils/blob/74a1b6d184eff18002ff576b774ff59b589a0739/dcos_test_utils/jobs.py#L96-L102

      However, the function never retries on an exception - https://github.com/dcos/dcos-test-utils/blob/74a1b6d184eff18002ff576b774ff59b589a0739/dcos_test_utils/jobs.py#L82

      To make it retry, this function should either:

      1. return False when the job has finished but history is not available, or
      2. distinguish this exception from the other exceptions and change the retry_on_exception.

      This error also suggests that https://github.com/dcos/dcos-test-utils/pull/62/commits/8df3e98f311a3ec10c73a28d642823b1fd940568 isn't testing what it was created to test.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                justinbarrick Justin Barrick
                Reporter:
                jonathangiddy Jonathan Giddy
                Team:
                Dev Prod Team
                Watchers:
                Alex Rukletsov (Inactive), Anitha Muthu (Inactive), Branden Rolston, Carter Gawron, Charles Provencher (Inactive), Daniel Baker, Deepak Goel, Gastón Kleiman (Inactive), Gauri Powale, Gilbert Song, Greg Mann (Inactive), Gustav Paul (Inactive), Jan-Philip Gehrcke (Inactive), Jonathan Giddy, Julian Gieseke, Justin Barrick, Mergebot, Orlando Hohmeier (Inactive), Philip Norman (Inactive), Senthil Kumaran, Sergey Urbanovich (Inactive), Till Toenshoff, Tim Weidner (Inactive)
              • Watchers:
                23 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: