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

Downgrade our python-NetworkManager dependency #792

Merged
merged 1 commit into from Apr 6, 2017
Merged

Downgrade our python-NetworkManager dependency #792

merged 1 commit into from Apr 6, 2017

Conversation

bochecha
Copy link
Contributor

@bochecha bochecha commented Apr 6, 2017

The latest releases are giving us trouble. Version 2.0.0 is a full
rewrite, which seems to have broken compatibility with the old
NetworkManager on Debian Jessie.

Git master has a patch to restore compatibility:

https://github.com/seveas/python-networkmanager/commit/cad4dc79efd8d29e9fe1f1cd03f76e61534af808

However, even with that applied we still seem to encounter issues. So
for now, we've decided to stick with the older version.

@bochecha
Copy link
Contributor Author

bochecha commented Apr 6, 2017

What are the issues we encounter with the python-networkmanager patch ?

Something about an invalid DBus method signature.

The problem having been found on a Monday, I admit to not having searched for it in more details.

In any case, fixing the issue might take some time, and right now ~20 boxes have the broken python-NetworkManager 2.0.0. A revert to version 1.2.1 with a quick release of Ideascube would allow us to fix these boxes while we take our time to move towards python-NetworkManager 2.0.1.

The latest releases are giving us trouble. Version 2.0.0 is a full
rewrite, which seems to have broken compatibility with the old
NetworkManager on Debian Jessie.

Git master has a patch to restore compatibility:

    seveas/python-networkmanager@cad4dc7

However, even with that applied we still seem to encounter issues. So
for now, we've decided to stick with the older version.
@bochecha
Copy link
Contributor Author

bochecha commented Apr 6, 2017

We've talked about this, and we agree this is the short-term way to go.

Let's try and fix things later with #791 .

@bochecha bochecha merged commit 6c566f3 into master Apr 6, 2017
@bochecha bochecha deleted the nm-dep branch April 6, 2017 11:44
@mgautierfr
Copy link
Member

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants