Skip to content

Commit

Permalink
fix: use JSON patch instead of expecting the whole object (#15)
Browse files Browse the repository at this point in the history
Signed-off-by: Leonardo Cecchi <[email protected]>
  • Loading branch information
leonardoce authored Jan 22, 2024
1 parent e99be6d commit 60e6b0f
Showing 1 changed file with 6 additions and 9 deletions.
15 changes: 6 additions & 9 deletions proto/operator.proto
Original file line number Diff line number Diff line change
Expand Up @@ -33,11 +33,9 @@ message OperatorMutatePodRequest {
}

message OperatorMutatePodResult {
// This field is OPTIONAL. Value of this field is the JSON
// serialization of the Pod as modified by the plugin.
// If not present, the operator will assume that no changes
// have been requested by the plugins
bytes definition = 1;
// This field is OPTIONAL. Value of this field is a JSONPatch
// to be applied on the passed Pod definition
bytes json_patch = 1;
}

message OperatorCapabilitiesRequest {
Expand Down Expand Up @@ -86,10 +84,9 @@ message OperatorMutateClusterRequest {
}

message OperatorMutateClusterResult {
// This field is REQUIRED. Value of this field is the JSON
// serialization of the Cluster that have received the
// default values
bytes definition = 1;
// This field is OPTIONAL. Value of this field is a JSONPatch
// to be applied on the passed Cluster definition
bytes json_patch = 1;
}

message ValidationError {
Expand Down

0 comments on commit 60e6b0f

Please sign in to comment.