Skip to content

Introduce `UpdateJobResult` and `PatchState`

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 (Back 2025-01-01)

Merge request reports

Loading