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

driver wants to use http, not https and cannot handle 307s #16

Open
yellowpattern opened this issue Mar 20, 2020 · 5 comments
Open

driver wants to use http, not https and cannot handle 307s #16

yellowpattern opened this issue Mar 20, 2020 · 5 comments
Assignees

Comments

@yellowpattern
Copy link

On newer FreeNAS systems, port 80 is but this:

    server {
        listen 10.1.1.1:80;
        listen [::]:80;
        server_name localhost;
        return 307 https://$host:443$request_uri;
    }

However this plugin wants to use port 80 and can't handle the 307:

...
ERROR oslo_messaging.rpc.server   File "/usr/lib/python2.7/site-packages/cinder/volume/flows/manager/create_volume.py", line 985, in _create_raw_volume
ERROR oslo_messaging.rpc.server     ret = self.driver.create_volume(volume)
ERROR oslo_messaging.rpc.server   File "/usr/lib/python2.7/site-packages/cinder/volume/drivers/ixsystems/iscsi.py", line 93, in create_volume
ERROR oslo_messaging.rpc.server     self.common._create_volume(freenas_volume['name'], freenas_volume['size'])
ERROR oslo_messaging.rpc.server   File "/usr/lib/python2.7/site-packages/cinder/volume/drivers/ixsystems/common.py", line 89, in _create_volume
ERROR oslo_messaging.rpc.server     raise FreeNASApiError('Unexpected error', msg)
ERROR oslo_messaging.rpc.server FreeNASApiError: FREENAS api failed. Reason - Unexpected error:Error while creating volume: 307:Temporary Redirect
ERROR oslo_messaging.rpc.server
@yellowpattern
Copy link
Author

Might be worth changing the default for ixsystems_transport_type:

ixsystems_transport_type = https

@hbonath
Copy link
Contributor

hbonath commented Mar 20, 2020

Which git revision are you working with here?
I noticed that your error referenced python2.7.
The "1.0" tag is the last version that supported Python2.
The current master branch as of 8a09f57 focuses on Python3 for the Openstack Train release and beyond.

@yellowpattern
Copy link
Author

Not using Train so went with 1.0.

@yellowpattern
Copy link
Author

Maybe there should be a 1.1 release, at least with pullup #5

YiHuangDB added a commit to YiHuangDB/cindertruenas that referenced this issue Jan 24, 2023
william-gr added a commit that referenced this issue Jan 24, 2023
#13 #16 add apikey and https auth support
@YiHuangDB
Copy link
Contributor

YiHuangDB commented Jan 26, 2023

With PR #29 merged to master branch now you can add ixsystems_transport_type = https or http to choose which transportation protocol to use in /etc/cinder/cinder.conf config file. When your Openstack version is above Train (which drop python2 support and require python3), this issue no longer exists on master branch code.

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

No branches or pull requests

3 participants