Skip to content

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

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

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

docstring: list the s3/cloudwatch permissions the test-connection ste…

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

Quality Gate passed

Issues
2 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check failure on line 106 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

Check warning on line 68 in ingestion/src/metadata/ingestion/source/storage/s3/connection.py

See this annotation in the file changed.

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

Replace this generic exception class with a more specific one. 

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