Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added envoy test missing options for s390x #36915

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

surenderky
Copy link
Contributor

Commit Message: Enabling envoy test cases by adding missing options for s390x
Additional Description: envoy test cases were enabled for s390x, so included the missing options

Enabling envoy test cases by adding missing options for s390x

Signed-off-by: Surender Yadav <[email protected]>
Copy link

Hi @surenderky, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #36915 was opened by surenderky.

see: more, trace.

@surenderky surenderky changed the title Add envoy test missing options for s390x Added envoy test missing options for s390x Oct 30, 2024
@nezdolik nezdolik self-assigned this Nov 1, 2024
@nezdolik
Copy link
Member

nezdolik commented Nov 4, 2024

/assign @envoyproxy/senior-maintainers

Copy link

@envoyproxy/senior-maintainers cannot be assigned to this issue.

🐱

Caused by: a #36915 (comment) was created by @nezdolik.

see: more, trace.

@yanavlasov yanavlasov merged commit 9dd4bea into envoyproxy:main Nov 5, 2024
20 checks passed
@surenderky surenderky deleted the enable-envoy-test-s390x branch November 6, 2024 15:19
surenderky added a commit to surenderky/envoy that referenced this pull request Nov 6, 2024
With reference to PR envoyproxy#36915 by mistake added the .cc extension instead of the .h extension. So fixing the extension to options_impl_platform_linux.h.

Signed-off-by: Surender Yadav <[email protected]>
ggreenway pushed a commit that referenced this pull request Nov 6, 2024
With reference to PR #36915 by
mistake added the .cc extension instead of the .h extension. So fixing
the extension to (options_impl_platform_linux.h).

Signed-off-by: Surender Yadav <[email protected]>
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