Block Storage API V3 (CURRENT) in cinder - wrong URL for backup-detail

Bug #1930526 reported by Kim Je Hee
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Stephan Pampel

Bug Description

This bug tracker is for errors with the documentation, use the following as a template and remove or add fields as you see fit. Convert [ ] into [x] to check boxes:

- [x] This doc is inaccurate in this way:

API URL of Import a backup is wrong (https://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded=import-a-backup-detail)

/v3/{project_id}/backups/{backup_id}/import_record

URL should be fixed to /v3/{project_id}/backups/import_record

-----------------------------------
Release: on 2021-05-25 19:08:09
SHA: da2fbff247f23c7b82e77db2c1e7772cb22801ac
Source: https://opendev.org/openstack/cinder/src/api-ref/source/v3/index.rst
URL: https://docs.openstack.org/api-ref/block-storage/v3/index.html?expanded=import-a-backup-detail

tags: added: backup-service documentation
summary: - Block Storage API V3 (CURRENT) in cinder
+ Block Storage API V3 (CURRENT) in cinder - wrong URL for backup-detail
Changed in cinder:
importance: Undecided → Low
tags: added: low-hanging-fruit
Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Looks like this was a bad copy/paste from export_record. The bug seems correct if you look at the URLs in cinder/tests/unit/api/contrib/test_backups.py

Cinder doesn't support importing to an existing backup.

Changed in cinder:
assignee: nobody → Sofia Enriquez (lsofia-enriquez)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/cinder/+/803896

Changed in cinder:
status: New → In Progress
Changed in cinder:
assignee: Sofia Enriquez (lsofia-enriquez) → Stephan Pampel (stephanpampel)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/803896
Committed: https://opendev.org/openstack/cinder/commit/7b9143fae980f118c6e439a7f09184b571d9567f
Submitter: "Zuul (22348)"
Branch: master

commit 7b9143fae980f118c6e439a7f09184b571d9567f
Author: Stephan Pampel <email address hidden>
Date: Mon Aug 9 12:03:39 2021 +0200

    Doc: Remove backup_id from backup import url and parameter

    In the documentation for importing backups the backup_id is removed
    because the backup_id is generated after the upload and the user
    does not need to provide it. Therefore the request parameter
    backup_id has also been removed.

    Closes-Bug: #1930526
    Change-Id: I49e0adea8fbd97d49f4386b8240320ea2baf3661

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 19.0.0.0rc1

This issue was fixed in the openstack/cinder 19.0.0.0rc1 release candidate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.