Skip to content

Swapped Minio for RustFS - #209

Merged
AndyRae merged 1 commit into
mainfrom
rustfs-integration
Oct 26, 2026
Merged

AndyRae merged 1 commit into
mainfrom
rustfs-integration

Conversation

@Thomas-Rowlands

Copy link
Copy Markdown
Contributor
♻️ Refactor

PR Description

Swapped the image from Minio to RustFS and made it an S3 configurable string, along with required parameters such as credentials.

Related Issues or other material

Related #
Closes #208

Screenshots, example outputs/behaviour etc.

✅ Added/updated tests?

  • [] This PR contains relevant tests / Or doesn't need to per the below explanation
    Existing tests used.

[optional] What gif best describes this PR or how it makes you feel?

alt_text

@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
carrottransform
   __init__.py201525%16–17, 20, 22, 25, 28–30, 33–37, 39, 41
   _version.py5260%5–6
carrottransform/cli
   __init__.py00100% 
carrottransform/cli/subcommands
   __init__.py00100% 
   run.py1591789%92–93, 105–107, 132, 165–167, 291, 299, 334–337, 383, 393
   run_v2.py20480%55, 57–59
carrottransform/tools
   __init__.py40100% 
   args.py1462881%96–97, 106–109, 133, 150, 154, 163, 168, 178, 181, 186, 197–199, 206, 212, 214, 222, 230, 260, 273, 283, 288, 331, 335
   at_path.py80100% 
   concept_helpers.py21195%27
   core.py69297%118, 126
   date_helpers.py45589%24–25, 28, 65, 70
   db.py210100% 
   file_helpers.py532749%18–20, 35, 37–41, 50–52, 57–61, 76, 78–80, 83–85, 87, 91, 96
   logger.py120100% 
   mapping_types.py200100% 
   mappingrules.py238598%120, 377, 380–382
   metrics.py1394667%129–134, 136, 143–156, 159, 168, 174–179, 262, 264–270, 286–288, 291, 298, 300, 303, 349–350
   omopcdm.py1531392%45–47, 131, 155, 160, 172–175, 181, 199, 205
   orchestrator.py1751094%87–88, 98, 101, 118–120, 205, 370, 372
   outputs.py1766364%50, 73, 125, 151–152, 169–171, 174–177, 180–183, 186, 189–190, 192–196, 198, 201, 204, 207, 211–212, 215, 217, 219, 221–222, 225–227, 230–232, 241–242, 251–253, 267, 281, 283, 290, 292–295, 297, 309–310, 315–316, 318–320, 331
   person_helpers.py872374%17–18, 20–21, 23–24, 85, 98, 102, 133–134, 147, 160–163, 166–168, 170–171, 174–175
   record_builder.py1822288%26, 56, 71, 76, 79, 88, 100, 108–109, 183, 210, 259, 300–301, 315, 323, 327, 341, 348, 365, 392–393
   sources.py1555266%27, 47, 50–51, 54, 64, 114–115, 184–188, 190–191, 193–194, 196, 199–200, 203–205, 207–212, 214, 218–220, 222, 228, 230–231, 233–234, 236, 239–240, 243–245, 247–252, 254
   stream_helpers.py340100% 
   types.py640100% 
   validation.py30197%51
TOTAL203633683% 

Tests Skipped Failures Errors Time
87 1 💤 0 ❌ 0 🔥 16.226s ⏱️

@Thomas-Rowlands Thomas-Rowlands self-assigned this Oct 20, 2026
@Thomas-Rowlands Thomas-Rowlands added the dependencies Pull requests that update a dependency file label Oct 20, 2026
@github-project-automation github-project-automation Bot moved this to MoreInfo in Carrot Backlog Oct 20, 2026
@AndyRae
AndyRae merged commit 3dbf91a into main Oct 26, 2026
28 checks passed
@AndyRae
AndyRae deleted the rustfs-integration branch October 26, 2026 06:49
@github-project-automation github-project-automation Bot moved this from MoreInfo to Done in Carrot Backlog Oct 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

conftest uses Minio

2 participants