Plugin occasionally fails to sync backwards

[improbable-mattchurch]

Agent bless used: https://buildkite.com/improbable/midwinter-bless/builds/92, note CL 68063

Specified CL 68005 for build, with a patch of 68241

Note errors:
Asset 'Cascade.umap' has been saved with engine version newer than current and therefore can't be loaded. CurrEngineVersion: 4.24.3-67998+++midwinter+main AssetEngineVersion: 4.24.3-68054+++midwinter+main

(CL 67998 was editor version compatible with CL 68005)

This would only be present if Cascade.umap wasn't synched to CL 68005. It turns out that the agent running the step was previously synced to CL 68054 from the last build it ran.

https://buildkite.com/improbable/midwinter-build/builds/2697#3a9cc5c2-d324-42b3-8d17-21d32316d055/6-7

Plugin version: perforce-buildkite-plugin#v4.3.2

[improbable-mattchurch]

On a more interesting note: I made two builds with the exact same parameters as above, and only one of them in failing with this issue:

see: https://buildkite.com/improbable/midwinter-build/builds/2698 vs https://buildkite.com/improbable/midwinter-build/builds/2697

[improbable-mattchurch]

It appears that the failed step was previously synced to CL 68054 in a different build:

The step that succeeded was done on a fresh agent

[improbable-mattchurch]

PEBCAK, cascade.umap was being modified in a shelf and wasn't being marked for edit so future jobs were spoiled