Skip to content

Introduce `UpdateJobResult` and `PatchState`

Kamil Trzciński requested to merge improve-update-and-patch-methods into master

What does this MR do?

Introduce UpdateJobResult and PatchState

Currently:

  • PatchTrace used UpdateState causing some confusion on handling of the different values
  • UpdateState as a return of UpdateJobResult is inflexible to append the result with additional data, like canceled or update-interval

This:

  • Introduces an explicit PatchState with a states directly supported by PatchTrace
  • Make UpdateJob to provide UpdateJobResult
  • It retains all current behavior for handling data

Why was this MR needed?

This is related to discussion about trace handling improvements for Runner: gitlab#232533 (comment 405974271).

This is needed to provide a better maintainability and explicitness of data processing.

Edited by Kamil Trzciński

Merge request reports