netflix / pygenie Goto Github PK

It errors out

Are there plans to add python 3 support now that python2 is EOL?
If it is already usable in python 3 then it would be a good idea to update the package config here.

The "offending" line is https://github.com/Netflix/pygenie/blob/master/pygenie/utils.py#L34-L40

Even when importing unicode literals, it seems like socket.getfqdn() returns a str instead of a unicode, and the concatenation fails if the hostname contains a unicode character.

With newer Mac OSX, it seems the hostname is set with a unicode apostrophe by default e.g. "Arthur’s".

steps to reproduce on Mac OSX:

oldhost=$(scutil --get HostName)  # To save your current hostname
scutil --set HostName Arthur’s . # for instance
# Running anything importing pygenie under python 2
python2 -c "import pygenie"

It's more of an FYI, since there is an easy workaround (no unicode in the hostname)

just reporting a problem I am facing while installing PyGenie on a MacOS :

Processing dependencies for nflx-genie-client==3.6.16
Searching for pyconfigurator
Reading https://pypi.org/simple/pyconfigurator/
Downloading https://files.pythonhosted.org/packages/4b/e0/22d25b713e84cb6c0a275802ed08fb906b841d15f646491c8c23f0f08799/pyconfigurator-0.4.14.tar.gz#sha256=fcb4607b8370a56f0836c00230e2f5c302be05c42199dadc3f66c74c32561224
Best match: pyconfigurator 0.4.14
Processing pyconfigurator-0.4.14.tar.gz
Writing /var/folders/6t/mj5ndcyx5hn_xx52ncf2qjs40000gp/T/easy_install-5ld3xqc2/pyconfigurator-0.4.14/setup.cfg
Running pyconfigurator-0.4.14/setup.py -q bdist_egg --dist-dir /var/folders/6t/mj5ndcyx5hn_xx52ncf2qjs40000gp/T/easy_install-5ld3xqc2/pyconfigurator-0.4.14/egg-dist-tmp-j02_n4a_
error: Setup script exited with error: SandboxViolation: mkdir('/Users/mdesavera/Library/Caches/com.apple.python/private/var/folders/6t/mj5ndcyx5hn_xx52ncf2qjs40000gp/T/easy_install-5ld3xqc2', 511) {}

HI Netflix,

noticed that PyGenie is not compatible in some places with version 4. For example, when using RunningJob::genie_log. In version 3 the source of the log was genie.log but in new version you have 3 log files agent.log and some other files. Also there is only genie adapter for 2x and 3x but no for 3x. Please let us know your priorities, otherwise we can run a PR on our end.

Travis are now recommending removing the sudo tag.

"If you currently specify sudo: false in your .travis.yml, we recommend removing that configuration"