processone / tsung

Tsung is a high-performance benchmark framework for various protocols including HTTP, XMPP, LDAP, etc.
http://www.process-one.net/en/tsung/
GNU General Public License v2.0
2.52k stars 404 forks source link

tsung-plotter is not ready for python 3 (python2 EOL) #351

Open didier13150 opened 5 years ago

didier13150 commented 5 years ago

Hi, I'm the fedora packager for tsung. As you known, python 2 will be officially end of life at January the 1st 2020 and tsung-plotter is not ready for python 3. Fedora plains to remove python2 entirely before this date.

tisba commented 5 years ago

I'm not familiar with tsung-plotter nor python. But it looks like there is not much Python LOC in tsung (https://github.com/processone/tsung/search?q=language%3APython&unscoped_q=language%3APython).

What would you suggest? Supporting both versions? Drop Python 2 support?

didier13150 commented 5 years ago

tsung-python3.patch.txt Support both versions is a good idea because upgrading to python3 doesn't break python2 running (just add parenthesis to each print function). But the patch is not hardened tested.

tisba commented 5 years ago

Thank you for the patch. It would greatly be appreciated if you could open up a Pull Request. This makes it a lot easier to be reviewed, discussed and merged eventually. Thank you!