Skip to content

Conversation

@EdSchouten
Copy link
Member

@EdSchouten EdSchouten commented Nov 7, 2025

In the early days of REv2, we made some backward incompatible code changes:

  • REv2.1 changes the way output paths are declared in Command.
  • REv2.2 moves the platform properties from Command to Action.

Up until this point we provided support for both the old and new schemes. Support for the old schemes is gated by configuration options on the scheduler and worker. Given that the new schemes were added to the protocol on 2019-11-19 and 2020-10-17, I think it's time to go ahead and remove support for the old schemes. Maintainers of clients have had more than enough time to implement them.

On 2019-11-19, REv2 changed the way expected output paths are declared
in the Command message. It used to be the case that files and
directories were declared separately, but this prevents you from writing
an action that captures either of those, depending on what the action
generated.

Given that this feature was added 6.5 years ago, let's go ahead and
remove support for the old scheme. Clients have had more than enough
time to implement the new scheme.
REv2.2 promoted the platform properties from the Command message to the
Action message. This has the advantage that the scheduler no longer
needs to process any Command messages. This saves one round trip to the
CAS. Command messages can in some cases also become very large, meaning
it adds unnecessary memory pressure on the scheduler.

Given that this feature was added on 2020-10-17 and clients were able to
adopt quickly. it's time to remove support for the old scheme.
With support for REv2.0 and REv2.1 specific features removed, we should
go ahead and bump the deprecated API version to REv2.2. This will force
clients to stop using these old features, or simply stop working
altogether.
@aspect-workflows
Copy link

aspect-workflows bot commented Nov 7, 2025

Test

Buildkite build #57 is running...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants