Skip to content

Fixes #29768: refactor(ingestion): migrate S3 test-connection to declarative framework#29770

Open
IceS2 wants to merge 5 commits into
mainfrom
migrate-s3-testconnection
Open

Fixes #29768: refactor(ingestion): migrate S3 test-connection to declarative framework#29770
IceS2 wants to merge 5 commits into
mainfrom
migrate-s3-testconnection

list_buckets: cap count locally instead of MaxBuckets, for S3-compati…

0391524
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ingestion] SonarCloud Code Analysis succeeded Jul 6, 2026 in 2m 12s

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
97.3% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 99 in ingestion/src/metadata/core/connections/test_connection/checks/storage.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ingestion] SonarCloud Code Analysis

Add the 'ExpectedBucketOwner' parameter to verify S3 bucket ownership. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ingestion&issues=AZ84IZgwF2Yd8keULkmI&open=AZ84IZgwF2Yd8keULkmI&pullRequest=29770
View more details on SonarQubeCloud