Skip to content

[test] [draft] Test Postgres (s3 lib v0) with S3 Integrator (object_storage lib v1) -- DO NOT MERGE#1242

Closed
theoctober19th wants to merge 4 commits into
16/edgefrom
test-s3/req0-prov1
Closed

[test] [draft] Test Postgres (s3 lib v0) with S3 Integrator (object_storage lib v1) -- DO NOT MERGE#1242
theoctober19th wants to merge 4 commits into
16/edgefrom
test-s3/req0-prov1

Conversation

@theoctober19th

Copy link
Copy Markdown
Member

Testing the behavior of postgres charm with s3-integrator from 2/edge channel.

@theoctober19th theoctober19th changed the base branch from main to 16/edge February 9, 2026 16:54
@theoctober19th theoctober19th added the not bug or enhancement PR is not 'bug' or 'enhancement'. For release notes label Feb 9, 2026
@github-actions github-actions Bot added the Libraries: Out of sync The charm libs used are out-of-sync label Feb 10, 2026
@codecov

codecov Bot commented Feb 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.91%. Comparing base (928740e) to head (2b087c2).
⚠️ Report is 106 commits behind head on 16/edge.

Additional details and impacted files
@@             Coverage Diff             @@
##           16/edge    #1242      +/-   ##
===========================================
- Coverage    69.01%   68.91%   -0.11%     
===========================================
  Files           16       16              
  Lines         3805     3812       +7     
  Branches       574      575       +1     
===========================================
+ Hits          2626     2627       +1     
- Misses         978      983       +5     
- Partials       201      202       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@welpaolo welpaolo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM. I am looking forward to review the next stages of this transition!

@theoctober19th theoctober19th changed the title [test] [draft] Test existing postgres with S3 Integrator from 2/edge -- DO NOT MERGE [test] [draft] Test Postgres (s3 lib v0) with S3 Integrator (object_storage lib v1) -- DO NOT MERGE Feb 11, 2026
@taurus-forever

Copy link
Copy Markdown
Contributor

@theoctober19th abandon?

@theoctober19th

Copy link
Copy Markdown
Member Author

Since we completed the tests, and also for the CI test purposes, we have another PR #1280 which we'll keep intact. Therefore, I'm closing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Libraries: Out of sync The charm libs used are out-of-sync not bug or enhancement PR is not 'bug' or 'enhancement'. For release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants