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

readme.md: Document how can I get the DSM IP address if I don't set up macvlan #678

Closed
wants to merge 1 commit into from

Conversation

pmorch
Copy link

@pmorch pmorch commented Apr 7, 2024

readme.md: Document how can I get the DSM IP address if I don't set up macvlan

@pmorch
Copy link
Author

pmorch commented Apr 7, 2024

And thanks for this project. I needed it to experiment with setting up the Nix package manager, and I didn't want to risk my existing setup. This project worked 100% as hoped (and way better than expected). No small feat. Kudos!

@pmorch pmorch changed the title readme.md: Document how can I get the DSM IP address if I don't set u… readme.md: Document how can I get the DSM IP address if I don't set up macvlan Apr 7, 2024
@kroese
Copy link
Collaborator

kroese commented Apr 7, 2024

Thanks!! I am not sure if how you do it is the easiest way to be honest..

For example, you can just run docker inspect dsm and it will also output the IP.

Or even better, you can add hostname: dsm to the compose file, and you do not even need the IP anymore because you can connect by name.

A third method is to just add the ports to the compose file, and you also wouldnt need the IP, because you can connect to localhost.

@pmorch pmorch marked this pull request as draft April 8, 2024 11:47
@pmorch
Copy link
Author

pmorch commented Apr 8, 2024

I totally agree. This PR is not currently suited for merging. I did spend some time finding out how to connect to the DSM over ssh. So I'd like to help out the next sorry person.

Adding ports 22222:22 allowed me to ssh -p 22222 user@localhost. But for NFS, things are a little less simple. And connecting to NFS on different ports is a pain, so instead:

  • Suggest adding ports as the primary approach, using ssh/22 as an example
  • Mention the docker inspect dsm as a way to find the IPAddress for NFS so standard ports can be used.

Sounds good/better? If we agree, I'll modify my commit as described and force push.

@kroese kroese closed this May 13, 2024
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.

None yet

2 participants