Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add link to related project https://github.com/haveachin/infrared #20

Merged
merged 1 commit into from
Nov 8, 2021

Conversation

vorburger
Copy link
Contributor

@vorburger vorburger commented Nov 8, 2021

I don't know if you would consider linking to a project by @haveachin in roughly the same space as this? Even if you don't seem to serve exactly the same use case, it might still be of interest to other folks looking at this. (Personally I'm considering to explore perhaps using BOTH of these projects together - one single mc-router as front-end entry point to dispatch, and multiple N infrared back-end "wrapping" instances for on-demand starting of Minecraft server containers via their respective StatefulSets. FYI @edewit.)

…README

I don't know if you would consider linking to a project by @haveachin in roughly the same space as this? Even if you don't seem to serve exactly the same use case, it might still be of interest to other folks looking at this. (Personally I'm considering to explore perhaps using BOTH of these projects together - one single `mc-router` as front-end entry point to dispatch, and multiple N `infrared` back-end "wrapping" instances for on-demand starting of Minecraft server containers via their respective StatefulSets. FYI @edewit.)
@vorburger vorburger changed the title add link to related project https://github.com/haveachin/infrared to … add link to related project https://github.com/haveachin/infrared Nov 8, 2021
@itzg
Copy link
Owner

itzg commented Nov 8, 2021

Wow, 1.7 million pulls in 7 months. It seems to be vastly more popular than mine. It looks like it covers mc-router's use cases, so I'm not sure why you'd need to include mc-router.

@haveachin
Copy link

Thanks, but I'm actually not sure how it got so many Docker pulls. As for this PR, I think it would be cool to link similar projects even if one seems like a superset of the other.

Copy link
Owner

@itzg itzg left a comment

Choose a reason for hiding this comment

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

If the link from infrared is merged in then I'm good with merging this one.

@haveachin
Copy link

Merged :) haveachin/infrared#105

@itzg itzg merged commit 8b54fdf into itzg:master Nov 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants