operator-framework / catalogd

On-cluster FBC catalog content server
Apache License 2.0
16 stars 32 forks source link

🌱 feat: add support for Podman as a container runtime #464

Closed camilamacedo86 closed 1 week ago

camilamacedo86 commented 1 week ago

Since docker requires a license in some scenarios, we might have contributors who prefer to use Podman. This commit introduces the changes necessary to support both.

codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.23%. Comparing base (55689e8) to head (b1c5480). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #464 +/- ## ======================================= Coverage 38.23% 38.23% ======================================= Files 15 15 Lines 1224 1224 ======================================= Hits 468 468 Misses 706 706 Partials 50 50 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

joelanford commented 1 week ago

If you are looking for a docker-based system on MacOS, I've been using colima successfully for quite awhile. It does not have the same license problems as the official Docker Desktop application and was more stable for me when I was trying to decide between podman desktop and colima.

I think it is fine to align this repo with operator-controller, but just be forewarned: the stance in operator-controller is that we do not and likely will not have CI setup for multiple container runtimes. Right now, we use docker officially, and podman is best-effort.