| Summary: | cinder backup fails with local tgt | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat OpenStack | Reporter: | Yogev Rabl <yrabl> | ||||
| Component: | openstack-packstack | Assignee: | Ivan Chavero <ichavero> | ||||
| Status: | CLOSED DUPLICATE | QA Contact: | Yogev Rabl <yrabl> | ||||
| Severity: | urgent | Docs Contact: | |||||
| Priority: | unspecified | ||||||
| Version: | 4.0 | CC: | aortega, derekh, eharney, fvollero, hateya, ichavero, mmagr, yeylon | ||||
| Target Milestone: | async | Keywords: | Triaged | ||||
| Target Release: | 4.0 | Flags: | aortega:
needinfo+
|
||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | storage | ||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2013-12-11 18:35:19 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Bug Depends On: | 1021627 | ||||||
| Bug Blocks: | |||||||
| Attachments: |
|
||||||
|
Description
Yogev Rabl
2013-10-13 07:25:15 UTC
Please link upstream LP bug if filing upstream too. You need to configure at least these settings in /etc/cinder/cinder.conf: backup_swift_user=admin backup_swift_password=asdfasdf backup_swift_url=http://192.168.122.xxx:8080/v1/AUTH_ CCing mmagr since this is probably something we could have packstack do. Looks like this bug should be fixed within the Cinder puppet module. ichavero is currently working on a fix for it. Created attachment 828049 [details]
Add swift backup options to config file
Added swift backup options to cinder module.
The cinder puppet module lacks the backup options for swift i added the options to the module. Martin, could you check it out and give me some advice. Thanks! The best thing to is to submit your patch to upstream in case you need to modify puppet module. Patch looks OK to me, but it's better to have more eyes to check it. Martin, is it ok to submit the patch to our repos while is approved upstream so we can close this bug? The code on 1021627 is already merged and the cinder plugin uses cinder::backup::swift in it's manifests. Doing some tests, Martin, Francesco can you helo me confirm that it works?? *** This bug has been marked as a duplicate of bug 1021627 *** |