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

Manila: add share resource #525

Closed

Conversation

kayrus
Copy link
Collaborator

@kayrus kayrus commented Dec 14, 2018

Part of #502

@kayrus
Copy link
Collaborator Author

kayrus commented Dec 14, 2018

I'm afraid that functional tests won't work on OpenLab. They passed locally on rocky.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 14, 2018

Build failed.

@kayrus
Copy link
Collaborator Author

kayrus commented Dec 14, 2018

As I expected (from the gophercloud PR, where I added explicitly skip of manila tests), shares are not created:

openstack_sharedfilesystem_share_v2.share_1: Error waiting for share a24fb1e6-1e88-4777-a301-0ed864b12b6b to become available: unexpected state 'error', wanted target 'available'. last error: %!s(<nil>)

@jtopjian did you get an update from OpenLab?

@kayrus
Copy link
Collaborator Author

kayrus commented Dec 14, 2018

Relates to gophercloud/gophercloud#1345

@jtopjian
Copy link
Contributor

I've opened an issue here: theopenlab/openlab#142

This might delay this PR from being merged, but I'd like to leave it open for a few days for the OpenLab team to look into it. We can proceed with reviewing and making changes, but I want to keep Manila enabled so a recheck can reproduce the problem.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 17, 2018

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 17, 2018

Build failed.

@huangtianhua
Copy link

recheck

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 22, 2018

Build failed.

@huangtianhua
Copy link

recheck

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 22, 2018

Build failed.

@huangtianhua
Copy link

please review theopenlab/openlab-zuul-jobs#398, we decide to set up devstack with manila enabling the LVM driver which operates in driver_handles_share_services=False mode, and please to modify your acctest to create share only with protocol, size and name, like "manila create NFS 1 --name test_share", no need to create network resources anymore.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 30, 2018

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 30, 2018

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 30, 2018

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Dec 31, 2018

Build failed.

@kayrus
Copy link
Collaborator Author

kayrus commented Dec 31, 2018

@h00130372, share creation and extending work, but shrinking fails.

@huangtianhua
Copy link

@kayrus, I think it's due to not all drivers support shrink feature, the mappings see:
https://github.com/openstack/manila/blob/master/doc/source/admin/share_back_ends_feature_support_mapping.rst
and now we enable lvm driver when setting up manila, lvm driver does not support shrink.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 2, 2019

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 2, 2019

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 2, 2019

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 2, 2019

Build failed.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 2, 2019

Build succeeded.

@kayrus
Copy link
Collaborator Author

kayrus commented Jan 2, 2019

finally
@jtopjian ready for review

Copy link
Contributor

@jtopjian jtopjian left a comment

Choose a reason for hiding this comment

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

@kayrus Thank you for working on this. I apologize for the slow reviews - I'm back to work after a few weeks of vacation.

I've left a few comments. Please let me know if you have any questions or if I made a mistake.

openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
openstack/resource_openstack_sharedfilesystem_share_v2.go Outdated Show resolved Hide resolved
website/docs/r/sharedfilesystem_share_v2.html.markdown Outdated Show resolved Hide resolved
@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 4, 2019

Build succeeded.

@kayrus
Copy link
Collaborator Author

kayrus commented Jan 4, 2019

Added detailed error messages. Now if the share source snapshot doesn't exists, terraform will print 404 detailed error message:

Error: Error applying plan:

1 error(s) occurred:

* openstack_sharedfilesystem_share_v2.share_2: 1 error(s) occurred:

* openstack_sharedfilesystem_share_v2.share_2: Error creating share: itemNotFound (404): ShareSnapshotNotFound: Snapshot 70bfbebc-d3ff-4528-8bbb-58422daa280b could not be found.

@theopenlab-ci
Copy link

theopenlab-ci bot commented Jan 4, 2019

Build succeeded.

@jtopjian
Copy link
Contributor

jtopjian commented Jan 5, 2019

Implemented as part of #526

@jtopjian jtopjian closed this Jan 5, 2019
@kayrus kayrus deleted the manila-share branch January 5, 2019 07:47
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.

None yet

3 participants