podman-compose icon indicating copy to clipboard operation
podman-compose copied to clipboard

podman-compose PRs are not merged - shortterm version available

Open maurerle opened this issue 1 year ago • 2 comments

Is your feature request related to a problem? Please describe. Somehow it is frustrating, that a lot Pull-Requests of podman-compose are not merged without comments, even though they provide additional value. I do know that open-source can be hard, peoples life changes or people lose interest in things they used to maintain. So we should always be respectful with that!

Describe the solution you'd like I would like to have a version in the containers org which addresses the fixes provided by the community or gives responses what has to be changed for them to be integrated.

Describe alternatives you've considered Until this happens, I merged pull requests which are not deprecated or break the single file installation into my devel branch: https://github.com/maurerle/podman-compose

I would not like to keep maintaining this, but for now, this is a working solution which might be interesting for others in need as well. To install, run: curl -o /usr/local/bin/podman-compose https://raw.githubusercontent.com/maurerle/podman-compose/devel/podman_compose.py && chmod +x /usr/local/bin/podman-compose

Additional context @muayyad-alsadi Thank you for this tool, which makes migrating to podman quite easy on typical development installations!

maurerle avatar Aug 07 '23 20:08 maurerle

it's not that I lost interest some times the PR is supposed to solve something small but instead of fixing that simple thing they change every aspect of the file (ex. split the single .py into many files or change shabang or ...)

some times the change is big and the affected area is large it fixes a case and break others (to this day we are suffering consequences of merging coloring output using sed, which is not available on windows and swallows last line of output if it does not have newline)

another example, some people like putting all containers in a pod, others hate it. sometimes it will work for most cases but breaks less common but standard cases (ex. when putting the entire stack in a single pod, having a custom different hostname for each container is not allowed by default ...etc)

sometimes the fix uses threads ..etc. sometimes it makes podman-compose active in the background (my vision to podman-compose, just exec and fork podman), that's why I suggested some changes to podman

how to go from here? maybe podman team suggest names to become co-maintainers as I'm very busy with my full-time job.

muayyad-alsadi avatar Aug 08 '23 15:08 muayyad-alsadi

Hello @muayyad-alsadi - I understand, that PRs which are against the philosophy of this project (single process, single file) do not get merged. However, improvements, adjustments to podman/docker features are needed and are often just small fixes. The situation somehow did not improve at all. I updated my devel branch to include useful fixes from latest PRs: https://github.com/containers/podman-compose/compare/devel...maurerle:podman-compose:devel (mostly using wget to download a patchset and apply it using git am --3way file.patch)

Issues and PRs are created for the same things, as some small issues do not get merged (see #771 and #761).

And the PR to reduce logging output #741 did not get any feedback yet.

I really would not like if it is needed to create a maintained fork of this project.

maurerle avatar Oct 04 '23 05:10 maurerle

Pull requests are now merged to podman-compose, we can close this issue.

p12tic avatar Jun 27 '24 13:06 p12tic

Thank you very much for taking care of this @p12tic !

maurerle avatar Jun 27 '24 14:06 maurerle