Skip to content

Commit

Permalink
fix priority type of wasmplugin (#881)
Browse files Browse the repository at this point in the history
  • Loading branch information
johnlanni committed Mar 26, 2024
1 parent 97cf58e commit a45748b
Show file tree
Hide file tree
Showing 3 changed files with 45 additions and 45 deletions.
86 changes: 43 additions & 43 deletions api/extensions/v1alpha1/wasm.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion api/extensions/v1alpha1/wasm.proto
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ message WasmPlugin {
// If `priority` is not set, or two `WasmPlugins` exist with the same
// value, the ordering will be deterministically derived from name and
// namespace of the `WasmPlugins`. Defaults to `0`.
google.protobuf.Int64Value priority = 10;
google.protobuf.Int32Value priority = 10;

// Extended by Higress, the default configuration takes effect globally
google.protobuf.Struct default_config = 101;
Expand Down
2 changes: 1 addition & 1 deletion pkg/ingress/config/ingress_config.go
Original file line number Diff line number Diff line change
Expand Up @@ -798,7 +798,7 @@ func (m *IngressConfig) convertIstioWasmPlugin(obj *higressext.WasmPlugin) (*ext
PluginConfig: obj.PluginConfig,
PluginName: obj.PluginName,
Phase: extensions.PluginPhase(obj.Phase),
Priority: obj.Priority,
Priority: &types.Int64Value{Value: int64(obj.GetPriority().Value)},
}
if result.PluginConfig != nil {
return result, nil
Expand Down

0 comments on commit a45748b

Please sign in to comment.