Skip to content

Fix variable name in log statement#26196

Closed
ericgribkoff wants to merge 1 commit intogrpc:masterfrom
ericgribkoff:fix_variable_name
Closed

Fix variable name in log statement#26196
ericgribkoff wants to merge 1 commit intogrpc:masterfrom
ericgribkoff:fix_variable_name

Conversation

@ericgribkoff
Copy link
Copy Markdown
Contributor

@ericgribkoff ericgribkoff commented May 7, 2021

Fixes a typo from #26090

Currently test is failing with:

  File "grpc/tools/run_tests/run_xds_tests.py", line 2134, in test_fault_injection
    logger.info('%s: attempt %d', testcase_name, i)
NameError: name 'i' is not defined

@ericgribkoff ericgribkoff added the release notes: no Indicates if PR should not be in release notes label May 7, 2021
@ericgribkoff ericgribkoff requested a review from lidizheng May 7, 2021 11:59
@ericgribkoff ericgribkoff mentioned this pull request May 7, 2021
Merged
@ericgribkoff
Copy link
Copy Markdown
Contributor Author

ericgribkoff commented May 7, 2021

I believe this should be closed and the original PR reverted, which is done in #26197.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lidizheng lidizheng Awaiting requested review from lidizheng

Assignees

@lidizheng lidizheng

Labels

release notes: no Indicates if PR should not be in release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ericgribkoff @lidizheng