layotto icon indicating copy to clipboard operation
layotto copied to clipboard

feat(oss api): support ceph oss

Open wlwilliamx opened this issue 3 years ago • 3 comments

What this PR does: Support ceph object gateway, so that we can use Layotto to store object into CEPH.

Which issue(s) this PR fixes:

Fixes #775

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


wlwilliamx avatar Aug 30 '22 06:08 wlwilliamx

Codecov Report

Base: 60.39% // Head: 60.50% // Increases project coverage by +0.11% :tada:

Coverage data is based on head (fa62804) compared to base (15a9192). Patch coverage: 58.29% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #775      +/-   ##
==========================================
+ Coverage   60.39%   60.50%   +0.11%     
==========================================
  Files         133      135       +2     
  Lines        8398     8807     +409     
==========================================
+ Hits         5072     5329     +257     
- Misses       2738     2825      +87     
- Partials      588      653      +65     
Impacted Files Coverage Δ
components/oss/aws/oss.go 59.93% <8.33%> (+7.59%) :arrow_up:
components/oss/option.go 58.87% <58.87%> (ø)
components/oss/ceph/oss.go 59.88% <59.88%> (ø)
components/oss/registry.go 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Aug 30 '22 15:08 codecov[bot]

please add ut case to fix the ut coverage

wenxuwan avatar Sep 21 '22 14:09 wenxuwan

please add ut case to fix the ut coverage

ok

wlwilliamx avatar Sep 22 '22 12:09 wlwilliamx

@wlwilliamx Hi, could u add docs in some other pull request? We need:

  • quickstart documentation. You can take https://mosn.io/layotto/#/zh/start/oss/oss as an example. Layotto can "run" your quickstart doc in the CI.
  • component doc

Reference: 文档贡献指南 使用工具自动测试 Quickstart 文档

seeflood avatar Oct 13 '22 09:10 seeflood

@wlwilliamx Hi, could u add docs in some other pull request? We need:

  • quickstart documentation. You can take https://mosn.io/layotto/#/zh/start/oss/oss as an example. Layotto can "run" your quickstart doc in the CI.
  • component doc

Reference: 文档贡献指南 使用工具自动测试 Quickstart 文档

ok!

wlwilliamx avatar Oct 13 '22 09:10 wlwilliamx

@wenxuwan @seeflood thanks !

wlwilliamx avatar Oct 13 '22 09:10 wlwilliamx