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

rgw_sal_motr : [CORTX-32387] Fix null version-id issues in GET/HEAD object API #377

Merged
merged 1 commit into from
Aug 5, 2022
Merged

rgw_sal_motr : [CORTX-32387] Fix null version-id issues in GET/HEAD object API #377

merged 1 commit into from
Aug 5, 2022

Conversation

priyanka25081999
Copy link

Problem Statement:
GET and HEAD object with null version-id does not contain "VersionId" field in the response

Solution:
In GET/HEAD object with --version-id null parameter for simple and multipart object, set the ent.key.instance value as null to show the VersionId field in the response.

Signed-off-by: Priyanka Salunke priyanka.s.salunke@seagate.com

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

…bject API

Signed-off-by: Priyanka Salunke <priyanka.s.salunke@seagate.com>
@cdeshmukh cdeshmukh merged commit 7fa8943 into Seagate:main Aug 5, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants