Skip to content

Extracting refresh_jid_expiration to base class

Max Fan requested to merge 427514-fix-refresh-import-state into master

What does this MR do and why?

Currently there are gaps where stages aren't calling project.import_state.refresh_jid_expiration. This will mess up the import if it's a long running one.

So, instead of calling this in each stage, we can call it before each stage in the base class. This will ensure future stages will already have this incorporated.

Screenshots or screen recordings

n/a

How to set up and validate locally

  1. Github should still work

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #427514 (closed)

Edited by Max Fan

Merge request reports