Skip to content

[GRPC] Fix compilation errors from core protobuf version bump to 0.23.0#5763

Merged
cwperks merged 2 commits intoopensearch-project:mainfrom
lucy66hw:0.23.0
Oct 31, 2025
Merged

[GRPC] Fix compilation errors from core protobuf version bump to 0.23.0#5763
cwperks merged 2 commits intoopensearch-project:mainfrom
lucy66hw:0.23.0

Conversation

@lucy66hw
Copy link
Contributor

@lucy66hw lucy66hw commented Oct 31, 2025

Description

[GRPC] Fix compilation errors from core protobuf version bump to 0.23.0
Core OpenSearch bumped the protobuf version to 0.23.0, which might break compilation in the security plugin's gRPC integration tests

Issues Resolved

[List any issues this PR will resolve]

Is this a backport? If so, please add backport PR # and/or commits #, and remove backport-failed label from the original PR.

Do these changes introduce new permission(s) to be displayed in the static dropdown on the front-end? If so, please open a draft PR in the security dashboards plugin and link the draft PR here

Testing

[Please provide details of testing done: unit testing, integration testing and manual testing]

Check List

  • New functionality includes testing
  • New functionality has been documented
  • New Roles/Permissions have a corresponding security dashboards plugin PR
  • API changes companion pull request created
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@lucy66hw lucy66hw changed the title Fix compilation errors for protobuf version 0.23.0 [GRPC] Fix compilation errors from core protobuf version bump to 0.23.0 Oct 31, 2025
@karenyrx
Copy link
Contributor

cc @cwperks @finnegancarroll

@finnegancarroll
Copy link
Contributor

Thanks for updating these tests @lucy66hw. I have had limited bandwidth recently but will make a point to address these breaking integration tests for 3.4 as these have been noted to be disruptive to security plugin and need addressing.

@cwperks
Copy link
Member

cwperks commented Oct 31, 2025

Will merge this if the checks pass as its fixing a compilation issue due to breaking change from the core.

@cwperks cwperks merged commit bd6bb31 into opensearch-project:main Oct 31, 2025
64 of 65 checks passed
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.

4 participants