cinder quota-defaults with wrong project name should return error

Bug #1638804 reported by sandeep nandal
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
New
Undecided
Unassigned

Bug Description

command "cinder quota-defaults" with wrong project name should return error instead of showing default-quota values. But when we execute command "cinder quota-defaults wrong_project_name", it returns default quota values instead of error message.

Changed in cinder:
assignee: nobody → Pranali Deore (pranali-deore)
Revision history for this message
Sean McGinnis (sean-mcginnis) wrote : Bug Assignee Expired

Unassigning due to no activity for > 6 months.

Changed in cinder:
assignee: Pranali Deore (pranali-deore) → nobody
Revision history for this message
wangxiyuan (wangxiyuan) wrote :

I think we should not check the project id and return 404 to users, it's not safe enough.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to cinder (master)

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

commit dcf66a96a43dd1383be1e8134ef4941e5bf43c9e
Author: Rajat Dhasmana <email address hidden>
Date: Thu Sep 17 10:16:50 2020 +0000

    Modify/Move project validation methods to api_utils

    We currently only validate the existence of a project on
    the default types code so that we cannot set it for non
    existing projects, but there are other places where we would
    benefit from having this validation, such as volume type access,
    setting quotas.
    To achieve this we are moving methods from quota_utils and default_types
    to api_utils.
    This patch also modifies the method get_project_hierarchy to get_project
    and removes related hierarchy tests since with the removal of nested quota
    driver we don't require the hierarchy properties anymore.

    Related-Bug: #1638804

    Change-Id: I20f8e500f583eb85f24a4c36f344c11c7face06c

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.