Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improvement: OSIS-76-make-s3-capabilities-file-path-configurable #100

Merged

Conversation

anurag4DSB
Copy link
Contributor

@anurag4DSB anurag4DSB commented Aug 18, 2022

standardizes s3 capabilities with minimal changes. We need to refactor the whole code base and the format of the files will change to JSON/YAML for all resources. Hence this is the most minimal change with code we are going to refactor.

@bert-e
Copy link
Contributor

bert-e commented Aug 18, 2022

Hello anurag4dsb,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Status report is not available.

@bert-e
Copy link
Contributor

bert-e commented Aug 18, 2022

Waiting for approval

The following approvals are needed before I can proceed with the merge:

  • the author

  • 2 peers

@anurag4DSB anurag4DSB force-pushed the improvement/OSIS-76-standerdize-and-update-s3capabilities branch from 1878f20 to a77a563 Compare August 18, 2022 12:52
@codecov
Copy link

codecov bot commented Aug 18, 2022

Codecov Report

Merging #100 (cbefecc) into main (0313544) will increase coverage by 0.01%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##               main     #100      +/-   ##
============================================
+ Coverage     63.66%   63.68%   +0.01%     
  Complexity      404      404              
============================================
  Files            71       71              
  Lines          2323     2324       +1     
  Branches        160      160              
============================================
+ Hits           1479     1480       +1     
  Misses          772      772              
  Partials         72       72              
Impacted Files Coverage Δ
...lity/osis/service/impl/ScalityOsisServiceImpl.java 87.94% <66.66%> (+0.01%) ⬆️

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

@anurag4DSB
Copy link
Contributor Author

@anurag4DSB anurag4DSB changed the base branch from development/1.1 to main August 22, 2022 14:38
@anurag4DSB anurag4DSB force-pushed the improvement/OSIS-76-standerdize-and-update-s3capabilities branch from 183d8ca to d70df80 Compare August 24, 2022 11:25
@anurag4DSB anurag4DSB marked this pull request as ready for review August 24, 2022 11:27
@anurag4DSB anurag4DSB requested review from a team and XinLi0207 as code owners August 24, 2022 11:27
@scality scality deleted a comment from bert-e Aug 24, 2022
@anurag4DSB anurag4DSB force-pushed the improvement/OSIS-76-standerdize-and-update-s3capabilities branch 2 times, most recently from 18b2e33 to 0b52a5c Compare August 26, 2022 09:26
- not having any exlusions will give us a heads up about upcoming issues
- actual version specific s3capabilities will be moved to federation
Current set of capabilities in S3C v9.1:

```json
{
    "s3_DeleteBucketAnalyticsConfiguration": {},
    "s3_DeleteBucketIntelligentTieringConfiguration": {},
    "s3_DeleteBucketInventoryConfiguration": {},
    "s3_DeleteBucketMetricsConfiguration": {},
    "s3_DeleteBucketOwnershipControls": {},
    "s3_DeletePublicAccessBlock": {},
    "s3_delete_bucket_lifecycle": {},
    "s3_GetBucketAccelerateConfiguration": {},
    "s3_GetBucketAnalyticsConfiguration": {},
    "s3_GetBucketIntelligentTieringConfiguration": {},
    "s3_GetBucketInventoryConfiguration": {},
    "s3_GetBucketLifecycle": {},
    "s3_GetBucketLogging": {},
    "s3_GetBucketMetricsConfiguration": {},
    "s3_GetBucketPolicyStatus": {},
    "s3_GetBucketRequestPayment": {},
    "s3_GetObjectAttributes": {},
    "s3_GetObjectTorrent": {},
    "s3_GetPublicAccessBlock": {},
    "s3_ListBucketAnalyticsConfigurations": {},
    "s3_ListBucketIntelligentTieringConfigurations": {},
    "s3_ListBucketInventoryConfigurations": {},
    "s3_ListBucketMetricsConfigurations": {},
    "s3_PutBucketAccelerateConfiguration": {},
    "s3_PutBucketAnalyticsConfiguration": {},
    "s3_PutBucketIntelligentTieringConfiguration": {},
    "s3_PutBucketInventoryConfiguration": {},
    "s3_PutBucketLifecycle": {},
    "s3_PutBucketLogging": {},
    "s3_PutBucketMetricsConfiguration": {},
    "s3_PutBucketRequestPayment": {},
    "s3_PutPublicAccessBlock": {},
    "s3_RestoreObject": {},
    "s3_SelectObjectContent": {},
    "s3_WriteGetObjectResponse": {}
}
```
@anurag4DSB anurag4DSB force-pushed the improvement/OSIS-76-standerdize-and-update-s3capabilities branch from 0b52a5c to cbefecc Compare August 26, 2022 11:12
@anurag4DSB anurag4DSB merged commit 1da09c8 into main Aug 26, 2022
@anurag4DSB anurag4DSB deleted the improvement/OSIS-76-standerdize-and-update-s3capabilities branch August 26, 2022 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants