Skip to content

Refactor object storage API to namespace-based#2258

Merged
cb-github-robot merged 1 commit intocloud-barista:mainfrom
yunkon-kim:260107-14
Jan 7, 2026
Merged

Refactor object storage API to namespace-based#2258
cb-github-robot merged 1 commit intocloud-barista:mainfrom
yunkon-kim:260107-14

Conversation

@yunkon-kim
Copy link
Copy Markdown
Member

  • Preserve proxy-based API as legacy for backward compatibility
  • Add native Object Storage CRUD via CB-Tumblebug's own concept/logic
  • Support bucket operations: create, get, list, delete, check existence, and location
  • Store Object Storage info in CB-Tumblebug metadata DB
  • Migrate APIs from /resources/objectStorage to /ns/{nsId}/resources/objectStorage
  • Update ExecuteHttpRequest to return resty response

- Preserve proxy-based API as legacy for backward compatibility
- Add native Object Storage CRUD via CB-Tumblebug's own concept/logic
- Support bucket operations: create, get, list, delete, check existence, and location
- Store Object Storage info in CB-Tumblebug metadata DB
- Migrate APIs from `/resources/objectStorage` to `/ns/{nsId}/resources/objectStorage`
- Update ExecuteHttpRequest to return resty response
@yunkon-kim yunkon-kim requested a review from seokho-son as a code owner January 7, 2026 12:19
@yunkon-kim
Copy link
Copy Markdown
Member Author

@seokho-son @hanizang77

Please note that the ExecuteHttpRequest function and related code have been updated to use the resty Response.

@yunkon-kim
Copy link
Copy Markdown
Member Author

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Jan 7, 2026
@cb-github-robot cb-github-robot merged commit bf3be2e into cloud-barista:main Jan 7, 2026
4 checks passed
objectStorageGroup.PUT("/resources/objectStorage/:objectStorageName/versioning", rest_resource.SetObjectStorageVersioning)
objectStorageGroup.GET("/resources/objectStorage/:objectStorageName/versions", rest_resource.ListObjectVersions)
objectStorageGroup.DELETE("/resources/objectStorage/:objectStorageName/versions/:objectKey", rest_resource.DeleteVersionedObject)
objectStorageGroup.GET("/resources/objectStorage/:objectStorageName/versioning", rest_resource.GetObjectStorageVersioningLagacy)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yunkon-kim
Lagacy 를 남겨둘 필요가 있을까요? 특별한 의도가 있지 않다면, 완성 단계에서 오래 사용한 기능이 아니므로, deprecate / remove 예정이라면 현재 단계에서 삭제를 해버리고 공지하는 것이 더 바람직할 것 같습니다.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@seokho-son Backward-compatibility 지원을 위해 당분간 유지하고자 했습니다.

상위 서브시스템에서 신규 API 적용을 완료 후 삭제할 예정입니다.

(cc. @cloud-barista/mc-data-manager-maintainer)

@yunkon-kim yunkon-kim deleted the 260107-14 branch January 9, 2026 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved This PR is approved and will be merged soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants