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

[Py3] Some salt-cloud drivers do not work using Python 3 #41720

Closed
rallytime opened this issue Jun 13, 2017 · 4 comments
Closed

[Py3] Some salt-cloud drivers do not work using Python 3 #41720

rallytime opened this issue Jun 13, 2017 · 4 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior P1 Priority 1 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. severity-critical top severity, seen by most users, serious issues severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around ZRELEASED - 2017.7.2
Milestone

Comments

@rallytime
Copy link
Contributor

Description of Issue/Question

The Joyent driver, as well as any driver using apache-libcloud, does not currently work when using Python 3.

Details for this issue can be found in this comment and this comment. This problem was initially found with the Joyent salt-cloud driver, as explained in issue saltstack/salt-ci-images#396, but this also affects gce, cloudstack, dimensiondata, and openstack.

This is happening on the Nitrogen branch when running salt-cloud with the aforementioned drivers when running with Python 3. Python 2 still works fine.

I am going to add a note to the Nitrogen release notes as a known issue for the time being, until we can get this fixed. I will link the issue when I have submitted the PR.

@tonybaloney This might be something you're interested in.

cc: @cachedout, @Ch3LL, @gtmanfred, and @techhat

@rallytime rallytime added Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around ZRELEASED - 2017.7.1 P1 Priority 1 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. labels Jun 13, 2017
@rallytime rallytime added this to the Approved milestone Jun 13, 2017
@cachedout
Copy link
Contributor

@rallytime I don't think a PR ever got linked here, fwiw.

@rallytime
Copy link
Contributor Author

@cachedout That's true. I am waiting for something to be added first, and then appending it to. I didn't state that here, though. I'll link it when I am able to submit the release note change. Thank you for the reminder!

@rallytime
Copy link
Contributor Author

The note about this issue was added to the 2017.7.0 release notes with PR #41852. (Thanks @Ch3LL!)

@Ch3LL Ch3LL added ZRELEASED - 2017.7.2 severity-critical top severity, seen by most users, serious issues and removed ZRELEASED - 2017.7.1 labels Jul 20, 2017
@gtmanfred
Copy link
Contributor

This is fixed by #42529

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior P1 Priority 1 RIoT Relates to integration with cloud providers, hypervisors, API-based services, etc. severity-critical top severity, seen by most users, serious issues severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around ZRELEASED - 2017.7.2
Projects
None yet
Development

No branches or pull requests

4 participants