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

Fix TypeError sending message to statsd socked #948

Merged
merged 1 commit into from
Dec 18, 2015
Merged

Fix TypeError sending message to statsd socked #948

merged 1 commit into from
Dec 18, 2015

Conversation

gbarba
Copy link

@gbarba gbarba commented Dec 18, 2015

To fix #947

@k4nar
Copy link
Contributor

k4nar commented Dec 18, 2015

Indeed, socket.sendto should receive bytes in Python 3.

Unfortunately something is wrong with Travis, so the tests couldn't run. However I tested locally with both Python 2 and 3 and it passed, so I'm ok with merging it.

Thank you very much!

k4nar added a commit that referenced this pull request Dec 18, 2015
Fix TypeError sending message to statsd socked
@k4nar k4nar merged commit cb135a3 into circus-tent:master Dec 18, 2015
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.

FullStats failts on python 3
2 participants