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 make check for python 3 #15226

Merged

Conversation

mikemadden42
Copy link
Contributor

make check works fine on Ubuntu 18.04 after this fix.

@kvch
Copy link
Contributor

kvch commented Dec 19, 2019

It does not seem to fix the build. :(

@mikemadden42
Copy link
Contributor Author

Thanks @kvch . I'm guessing you're referring to the travis-ci jobs?

I wonder if this is an issue with an older Python in Travis CI? @jsoriano Is there a minimum python 3.x that we're targeting?

DEPRECATION: Python 3.4 support has been deprecated. pip 19.1 will be the last one supporting it. Please upgrade your Python as Python 3.4 won't be maintained after March 2019 (cf PEP 429).

@mikemadden42
Copy link
Contributor Author

I wonder if we need to update the version of Ubuntu used for our travis-ci jobs. We're using Ubuntu 14.04 (Trusty) which looks like it ships with Python 3.4. We may need to switch to Ubuntu 16.04 (Xenial) or 18.04 (Bionic). I've been doing my testing on Ubuntu 18.04.

$ grep ^dist .travis.yml
dist: trusty

@mikemadden42 mikemadden42 changed the title Run autopep on script/update_golang_x.py to fix make check Fix make check for python 3 Dec 19, 2019
@mikemadden42
Copy link
Contributor Author

@kvch @jsoriano make check is passing in the latest travis build for this PR.

https://travis-ci.org/elastic/beats/builds/627459709

Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! If travis likes it, I like it!

I wonder if this is an issue with an older Python in Travis CI? @jsoriano Is there a minimum python 3.x that we're targeting?

We haven't decided a minimum version to support, but in any case we shouldn't care about deprecated or unmaintained versions, so no worries if this doesn't work with Python 3.4.

There was a point about upgrading python, or the base image in Travis in #14798, I have added this PR as fix for that.

.travis.yml Show resolved Hide resolved
.travis.yml Show resolved Hide resolved
.travis.yml Show resolved Hide resolved
@mikemadden42
Copy link
Contributor Author

There are around 11 failures unrelated to make check in the last travis build. It would be nice to investigate these.

https://travis-ci.org/elastic/beats/builds/627459709

@mikemadden42 mikemadden42 merged commit b9629f9 into elastic:feature/python3 Dec 20, 2019
@mikemadden42 mikemadden42 deleted the autopep-update_golang_x branch December 20, 2019 14:05
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* Run autopep on script/update_golang_x.py to fix make check
* Update travis to use Ubuntu Bionic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants