Skip to content

[rfc2737] Fix issue: expect redis pubsub data to be str type instead of bytes type#4

Closed
Junchao-Mellanox wants to merge 1 commit intomasterfrom
fix-bytes
Closed

[rfc2737] Fix issue: expect redis pubsub data to be str type instead of bytes type#4
Junchao-Mellanox wants to merge 1 commit intomasterfrom
fix-bytes

Conversation

@Junchao-Mellanox
Copy link
Copy Markdown
Owner

- What I did

Change redis pubsub expect data type from bytes to str. There was a snmpagent framework change which transfer redis pubsub data from bytes to str, after that change, there is no need for non-framework code to do this transfer. So now we need handle str data directly.

- How I did it

Change data expectation from bytes to str

- How to verify it

Run regression and manual test

- Description for the changelog

@Junchao-Mellanox
Copy link
Copy Markdown
Owner Author

sonic-net#196

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