Skip to content

Conversation

bbasata
Copy link
Collaborator

@bbasata bbasata commented Aug 14, 2025

Related Issue

Backports #496 into release/1.13.x

cc: @BBBmau

Description

This change is intended to unlock an upgrade path from v1.5.1 to latest v1.13.1.

When "refresh after apply" is set to non-empty via an environment variable, a Config-mode test step will invoke a refresh before successful completion. This is intended as a compatibility measure for test cases that have different -- but semantically-equal -- state representations in their test steps. When comparing two states, the testing framework is not aware of semantic equality or set equality, as that would rely on provider logic.

A Config-mode test step that is followed by a Refresh-mode test step will not incur an extra refresh. This is a backward-compatible way for a provider to eventually remove the compatibility flag – existing tests can be updated to explicitly refresh, incrementally.

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

None

)

* experimental: refresh after config mode test step

* fixup! experimental: refresh after config mode test step

* Skip an unnecessary refresh

* Rename compatibility flag and extract to compatibility.go

* Add header

* Use an environment variable

* logging > fmt

* Add changelog entry
@bbasata bbasata marked this pull request as ready for review August 14, 2025 20:32
@bbasata bbasata requested a review from a team as a code owner August 14, 2025 20:32
@bbasata bbasata merged commit c569860 into release/1.13.x Aug 15, 2025
42 checks passed
@bbasata bbasata deleted the backport-refresh-after-config branch August 15, 2025 14:51
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants