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.

Sign up for GitHub

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

Jump to bottom

Issue with port forwarding and latest lima 1.0 #1181

Closed
1 of 5 tasks
matejsp opened this issue Nov 6, 2024 · 12 comments · Fixed by #1182
Closed
1 of 5 tasks

Issue with port forwarding and latest lima 1.0 #1181

matejsp opened this issue Nov 6, 2024 · 12 comments · Fixed by #1182

Comments

@matejsp
Copy link
Contributor

matejsp commented Nov 6, 2024

Description

When starting docker port forwarding does not work. I can run docker commands and start containers, however port forwarding is not working anymore.

Version

colima version && limactl --version && qemu-img --version
colima version 0.7.6
git commit: 3ab92f5

runtime: docker
arch: aarch64
client: v27.3.1
server: v27.3.1
limactl version 1.0.0
qemu-img version 9.1.1
Copyright (c) 2003-2024 Fabrice Bellard and the QEMU Project developers

Operating System

  • macOS Intel <= 13 (Ventura)
  • macOS Intel >= 14 (Sonoma)
  • Apple Silicon <= 13 (Ventura)
  • Apple Silicon >= 14 (Sonoma)
  • Linux

Output of colima status

INFO[0000] colima is running using macOS Virtualization.Framework
INFO[0000] arch: aarch64
INFO[0000] runtime: docker
INFO[0000] mountType: virtiofs
INFO[0000] socket: unix:///Users/matej.spiller-muys/.colima/default/docker.sock

Reproduction Steps

  1. upgrade lima
  2. colima stop
  3. colima start
  4. port forwarding is not working (not TCP not UDP)

Expected behaviour

I was actually looking forward to UDP socket with docker.

Additional context

Workaround is to run as:
LIMA_SSH_PORT_FORWARDER=true colima start
@matejsp matejsp changed the title Issue with latest lima 1.0 Issue with port forwarding and latest lima 1.0 Nov 6, 2024
@abiosoft
Copy link
Owner

abiosoft commented Nov 6, 2024

Thanks for reporting this.

@abiosoft abiosoft mentioned this issue Nov 6, 2024
Merged
@abiosoft
Copy link
Owner

abiosoft commented Nov 6, 2024

This has been created as a workaround #1182.

@abiosoft
Copy link
Owner

abiosoft commented Nov 6, 2024

@matejsp can you confirm if the workaround fixes it?

brew install --head colima

@k0d3r1s
Copy link

k0d3r1s commented Nov 6, 2024

@abiosoft it fixes it :) came here to report same issue but it is now working correctly

@matejsp
Copy link
Contributor Author

matejsp commented Nov 6, 2024

Nice. The workaround works. But it is still workaround.
Looking forward to proper solution so we can get UDP port forwarding working.

Should we open another task for enabling new port forwarding?
Why is it not working with new better and faster way?
Can you perhaps point me to what would need to be changed to achieve this?

@abiosoft
Copy link
Owner

abiosoft commented Nov 6, 2024

I am not yet sure and I would need to do more testing to verify the root cause.

Should we open another task for enabling new port forwarding?

@matejsp maybe we can utilise this issue for that #708.

@matejsp
Copy link
Contributor Author

matejsp commented Nov 6, 2024

Perfect!

@Philipp000 Philipp000 mentioned this issue Nov 6, 2024
Closed
5 tasks
@msogrin msogrin mentioned this issue Nov 7, 2024
Closed
5 tasks
@abiosoft
Copy link
Owner

abiosoft commented Nov 8, 2024

A new release has been published.

@jasperf jasperf mentioned this issue Nov 9, 2024
Closed
tmeijn pushed a commit to tmeijn/dotfiles that referenced this issue Nov 20, 2024
@tmeijn
build(deps): update abiosoft/colima to v0.8.0
ccfe779 
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [abiosoft/colima](https://github.com/abiosoft/colima) | minor | `v0.7.5` -> `v0.8.0` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>abiosoft/colima (abiosoft/colima)</summary>

### [`v0.8.0`](https://github.com/abiosoft/colima/releases/tag/v0.8.0)

[Compare Source](abiosoft/colima@v0.7.6...v0.8.0)

#### Highlights

This is a new release to address compatibility issues with [Lima v1.0.0](https://github.com/lima-vm/lima/releases/tag/v1.0.0).

-   Colima now utilises virtualization.framework on macOS when Qemu is not available. Fixes [#&#8203;1188](abiosoft/colima#1188).
-   SSH port-forwarding is retained for Lima v1.0.0. Fixes [#&#8203;1181](abiosoft/colima#1181), [#&#8203;1184](abiosoft/colima#1184), [#&#8203;1185](abiosoft/colima#1185).

#### Other Updates

-   Nested Virtualization is now supported with the `--nested-virtualization` flag or `nestedVirtualization` config for m3 devices.
-   The default disk size is now 100GiB.

#### Commits

-   core: retain SSH port forwarder by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1182
-   core: make qemu an optional dependency by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1187

**Full Changelog**: abiosoft/colima@v0.7.6...v0.8.0

### [`v0.7.6`](https://github.com/abiosoft/colima/releases/tag/v0.7.6)

[Compare Source](abiosoft/colima@v0.7.5...v0.7.6)

#### Highlights

This release introduces the `colima update` command to update the container runtime version (e.g. Docker version) without needing to upgrade Colima or wait for the next release.

#### Other Fixes

-   Avoid error when docker-certs directory does not exist. Fixes [#&#8203;1128](abiosoft/colima#1128).
-   Update host-gateway-ip in proxy settings for docker daemon. Fixes [#&#8203;956](abiosoft/colima#956), [#&#8203;1040](abiosoft/colima#1040).

#### Commits

-   docker: avoid error where the cache docker-certs directory does not exist by [@&#8203;AlexanderOMara](https://github.com/AlexanderOMara) in abiosoft/colima#1129
-   build(deps): bump golang.org/x/term from 0.23.0 to 0.24.0 by [@&#8203;dependabot](https://github.com/dependabot) in abiosoft/colima#1133
-   docs: fixed typo on Apple Silicon in README by [@&#8203;analogsignals](https://github.com/analogsignals) in abiosoft/colima#1137
-   docker: replace 127.0.0.1 with host gateway ip in docker daemon.json proxy settings by [@&#8203;rteeling-evernorth](https://github.com/rteeling-evernorth) in abiosoft/colima#1145
-   ci: bump Go version by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1148
-   net: update vmnet by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1152
-   build(deps): bump golang.org/x/term from 0.24.0 to 0.25.0 by [@&#8203;dependabot](https://github.com/dependabot) in abiosoft/colima#1154
-   Minor updates by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1156
-   ci: use older qemu version by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1160
-   build(deps): bump github.com/fatih/color from 1.17.0 to 1.18.0 by [@&#8203;dependabot](https://github.com/dependabot) in abiosoft/colima#1171
-   cli: add update command by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1178
-   core: update disk images by [@&#8203;abiosoft](https://github.com/abiosoft) in abiosoft/colima#1179

#### New Contributors

-   [@&#8203;AlexanderOMara](https://github.com/AlexanderOMara) made their first contribution in abiosoft/colima#1129
-   [@&#8203;analogsignals](https://github.com/analogsignals) made their first contribution in abiosoft/colima#1137
-   [@&#8203;rteeling-evernorth](https://github.com/rteeling-evernorth) made their first contribution in abiosoft/colima#1145

**Full Changelog**: abiosoft/colima@v0.7.5...v0.7.6

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40NDAuNyIsInVwZGF0ZWRJblZlciI6IjM3LjQ0MC43IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
@matejsp
Copy link
Contributor Author

matejsp commented Feb 3, 2025

Hey based on the workaround made with 1.0.0 it is now impossible to test with LIMA_SSH_PORT_FORWARDER set to false:
https://github.com/abiosoft/colima/pull/1182/files

Would it be possible to change to true value only if not passed from outside.

@abiosoft
Copy link
Owner

abiosoft commented Feb 3, 2025

Would it be possible to change to true value only if not passed from outside.

Yeah, that can be done.

@abiosoft
Copy link
Owner

abiosoft commented Feb 3, 2025

PR created #1263

@matejsp
Copy link
Contributor Author

matejsp commented Feb 3, 2025

Perfect! thank you. will test with latest lima changes if it works better

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

core: retain SSH port forwarder abiosoft/colima
3 participants
@abiosoft @matejsp @k0d3r1s