Skip to content

Commit

Permalink
stacks: remove deferred actions for UpgradeResourceState
Browse files Browse the repository at this point in the history
UpgradeResourceState is offline; it does not need a configured provider

therefore it can not be deferred
  • Loading branch information
DanielMSchmidt committed Mar 26, 2024
1 parent 71873ec commit 45e9530
Show file tree
Hide file tree
Showing 4 changed files with 979 additions and 1,049 deletions.
8 changes: 0 additions & 8 deletions docs/plugin-protocol/tfplugin5.6.proto
Original file line number Diff line number Diff line change
Expand Up @@ -344,10 +344,6 @@ message UpgradeResourceState {
// appropriate older schema. The raw_state will be the json encoded
// state, or a legacy flat-mapped format.
RawState raw_state = 3;

// deferral_allowed signals that the provider is allowed to defer the
// changes. If set the caller needs to handle the deferred response.
bool deferral_allowed = 4;
}
message Response {
// new_state is a msgpack-encoded data structure that, when interpreted with
Expand All @@ -359,10 +355,6 @@ message UpgradeResourceState {
// be safely resolved, and warnings about any possibly-risky assumptions made
// in the upgrade process.
repeated Diagnostic diagnostics = 2;

// deferred is set if the provider is deferring the change. If set the caller
// needs to handle the deferral.
Deferred deferred = 3;
}
}

Expand Down
8 changes: 0 additions & 8 deletions docs/plugin-protocol/tfplugin6.6.proto
Original file line number Diff line number Diff line change
Expand Up @@ -362,10 +362,6 @@ message UpgradeResourceState {
// appropriate older schema. The raw_state will be the json encoded
// state, or a legacy flat-mapped format.
RawState raw_state = 3;

// deferral_allowed signals that the provider is allowed to defer the
// changes. If set the caller needs to handle the deferred response.
bool deferral_allowed = 4;
}
message Response {
// new_state is a msgpack-encoded data structure that, when interpreted with
Expand All @@ -377,10 +373,6 @@ message UpgradeResourceState {
// be safely resolved, and warnings about any possibly-risky assumptions made
// in the upgrade process.
repeated Diagnostic diagnostics = 2;

// deferred is set if the provider is deferring the change. If set the caller
// needs to handle the deferral.
Deferred deferred = 3;
}
}

Expand Down
Loading

0 comments on commit 45e9530

Please sign in to comment.