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

minor: fix CSM o11y with injection test filename #144

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

sergiitk
Copy link
Member

@sergiitk sergiitk commented Dec 11, 2024

Rename csm_observability_test_with_injection.py to csm_observability_with_injection_test.py: _test should be the suffix, not the infix.

Related cl/705131445.

`_test` should be the suffix, not the infix.
@sergiitk sergiitk requested a review from a team as a code owner December 11, 2024 17:50
@sergiitk
Copy link
Member Author

cc @arvindbr8

@sergiitk
Copy link
Member Author

sergiitk commented Dec 11, 2024

@sergiitk sergiitk merged commit b1402dc into grpc:main Dec 11, 2024
6 checks passed
@sergiitk sergiitk deleted the fix-o11y-w-injection-name branch December 11, 2024 18:24
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.

2 participants