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, Daniel Baker, Deepak Goel, Gastón Kleiman, Gauri Powale, Gilbert Song, Greg Mann, Gustav Paul, Jan-Philip Gehrcke, Jonathan Giddy, Julian Gieseke, Justin Barrick, Matthias Eichstedt, Mergebot, Orlando Hohmeier, Philip Norman, Senthil Kumaran, Sergey Urbanovich, Till Toenshoff, Tim Weidner
              • Watchers:
                24 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: