Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

CORTX-32589: [Bug fix] Fixed mapping in capacity response #889

Merged

Conversation

rohitkolapkar
Copy link
Contributor

@rohitkolapkar rohitkolapkar commented Aug 16, 2022

Signed-off-by: Rohit Kolapkar rohit.j.kolapkar@seagate.com

Pull Request

Problem Statement

Design

  • For Bug, Describe the fix here.
  • For Feature, Post the link for design

Coding

Checklist for Author

  • Coding conventions are followed and code is consistent

Testing

Checklist for Author

  • Unit and System Tests are added
  • Test Cases cover Happy Path, Non-Happy Path and Scalability
  • Testing was performed with RPM

Impact Analysis

Checklist for Author/Reviewer/GateKeeper

  • Interface change (if any) are documented
  • Side effects on other features (deployment/upgrade)
  • Dependencies on other component(s)

Review Checklist

Checklist for Author

  • JIRA number/GitHub Issue added to PR
  • PR is self reviewed
  • Jira and state/status is updated and JIRA is updated with PR link
  • Check if the description is clear and explained

Documentation

Checklist for Author

  • Changes done to WIKI / Confluence page / Quick Start Guide

Dev Testing

image

example 1:

request- GET https://hostname/api/v2/capacity/s3/user/t5$demouser1
response- 
{
    "capacity": {
        "s3": {
            "users": [
                { "tenant": "t5",                     

                  "id": "demouser1",

                   "used": 0,

                   "used_rounded": 0,

                    "objects": 0                 

                }

            ]
        }
    }
}

example 2:
 
request- GET https://hostname/api/v2/capacity/s3/user/user1
response- 
{
    "capacity": {
        "s3": {
            "users": [
              {

               "tenant": "",

                "id": "user1",

                "used": 0,

                "used_rounded": 0,

                 "objects": 0                 

               }

            ]
        }
    }
}

Signed-off-by: Rohit Kolapkar <rohit.j.kolapkar@seagate.com>
Signed-off-by: Rohit Kolapkar <rohit.j.kolapkar@seagate.com>
@cla-bot cla-bot bot added the cla-signed label Aug 16, 2022
Signed-off-by: Rohit Kolapkar <rohit.j.kolapkar@seagate.com>
@Pranali04796 Pranali04796 self-requested a review August 16, 2022 11:40
@pawankumarsrivastava pawankumarsrivastava merged commit f80f7d7 into Seagate:main Aug 16, 2022
@rohitkolapkar rohitkolapkar deleted the CORTX-32589-tenant-name branch August 24, 2022 12:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants