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

[pull] main from abiosoft:main #12

Open
wants to merge 286 commits into
base: main
Choose a base branch
from
Open

[pull] main from abiosoft:main #12

wants to merge 286 commits into from

Conversation

pull[bot]
Copy link

@pull pull bot commented May 3, 2022

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label May 3, 2022
abiosoft and others added 29 commits July 19, 2022 10:24
* chore: update nix shell

* chore: remove redundant code

* core: update iso images

* layer: ensure hostname sync

* cli: add clone command

* k8s: update default to v1.24.3+k3s1

* core: make docker/k8s context activation optional

* k8s: remove as active context on shutdown

* chore: fix typo
* vm: propagate upstream support for mountPoint

* mount: fix mountPath in layer

* vm: avoid dir warning messages in ssh command

* vm: improve mount folder detection for ssh
* vm: add support for provision scripts in config
* misc: minor typo in docs

* net: introduce configurable network driver

* net: fix default in template

* ci: fix lint errors

* net: fix regression

* net: addn docker hostname for slirp network
…20816042650-89efd2bdb96 (#402)

Signed-off-by: Akihiro Suda <[email protected]>

Signed-off-by: Akihiro Suda <[email protected]>
Observed the following when running colima in a terminal with somehow garbled settings (`stty size` printing `0 0`):
```
$ colima start
INFO[0000] starting colima                              
INFO[0000] runtime: docker                              
INFO[0000] preparing network ...                         context=vm
INFO[0000] creating and starting ...                     context=vm
panic: runtime error: integer divide by zero

goroutine 25 [running]:
github.com/abiosoft/colima/util/terminal.(*verboseWriter).printScreen(0xc000172fc0)
        github.com/abiosoft/colima/util/terminal/output.go:115 +0x1b3
github.com/abiosoft/colima/util/terminal.(*verboseWriter).refresh(0xc000172fc0)
        github.com/abiosoft/colima/util/terminal/output.go:65 +0x92
github.com/abiosoft/colima/util/terminal.(*verboseWriter).Write(0xc000172fc0, {0xc00030c000, 0x72, 0x0?})
        github.com/abiosoft/colima/util/terminal/output.go:48 +0x8c
io.copyBuffer({0x279f558, 0xc000172fc0}, {0x17cf2c0, 0xc000122338}, {0x0, 0x0, 0x0})
        io/io.go:428 +0x204
io.Copy(...)
        io/io.go:385
os/exec.(*Cmd).writerDescriptor.func1()
        os/exec/exec.go:311 +0x3a
os/exec.(*Cmd).Start.func1(0x0?)
        os/exec/exec.go:444 +0x25
created by os/exec.(*Cmd).Start
        os/exec/exec.go:443 +0x795
```

While we can't expect the output to look particularly good in such an environment, the tool should at least not panic, obscuring whether or not an action succeeded.
* core: revert iso version temporarily

* vm: fix lima warnings for colimaState

* chore: upgrade gvisor to fix cpuinfo warnings

* chore: go mod tidy
* net: fix network-address regression on macOS

* k3s: update to v1.25

* net: fix regressed DNS settings
* net: fix dns regression with network address

* net: consider gvproxy condition

* net: make gvproxy the default for consistency
* vm: make auto ssh config generation optional

* k3s: fix kubernetes version bug

* k3s: update to latest stable release

* vm: ignore running status when --force is passed
* feat: add kubernetes-disable flag

* Update cmd/start.go

Co-authored-by: Abiola Ibrahim <[email protected]>

Co-authored-by: Abiola Ibrahim <[email protected]>
* net: fix vmnet build script

* net: modify vmnet build script

* net: add socket_vmnet binary

* misc: remove inefficient code

* net: fix vmnet build script

* net: port to socket_vmnet

* net: fix vmnet_socket

* net: fix network socket when gvproxy is enabled
- Add command to create empty config file required by scan plugin.
- Update scan version
abiosoft and others added 30 commits August 6, 2024 22:12
Signed-off-by: Abiola Ibrahim <[email protected]>

The downloader currently has undesired behaviour of downloading twice
when fetching the possible redirect URL of the asset to be downloaded.

This resolves it by ensuring the initial request is an HEAD request.
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
* chore: calculate checksums after signing assets by codesign

This commit fixes checksums of assets for macOS.

From colima v0.7.1, assets for macOS are signed by codesign.

- 3415501
- #1094

But due to this checksums became invalid.

```console
$ gh release download abiosoft/colima v0.7.1

$ cat *.sha256sum
9db4f5617eb3a16ef4abca4d515a7e81b80f7f8646a8c3fe50e4eb6e6e1a8ccb *colima-Darwin-arm64
e136336f3b000fda729f4266d1af573dcf3378568d4c5433c6b63350f960f296 *colima-Darwin-x86_64
f866550da41a4d1df7437fbf2e42449e213281ac1784d2f4d905e7bd13c5c772 *colima-Linux-aarch64
95cdef8ffc9fa2620a9a4c0771c4eab727b5d654341a0b6fcc70842c2bc38486 *colima-Linux-x86_64

$ sha256sum colima-Darwin-arm64 colima-Darwin-x86_64 colima-Linux-aarch64 colima-Linux-x86_64
df99434b42f0f85da5fa86063f86d7e818a19fcb46da528fe4201d9f6356744e  colima-Darwin-arm64
abdf3069c8e6d35b1b23730cb320729eb53800b1049a4973f550e1ee4f62fa99  colima-Darwin-x86_64
f866550da41a4d1df7437fbf2e42449e213281ac1784d2f4d905e7bd13c5c772  colima-Linux-aarch64
95cdef8ffc9fa2620a9a4c0771c4eab727b5d654341a0b6fcc70842c2bc38486  colima-Linux-x86_64
```

codesign changed assets, so we need to generate checksum files after running codesign.

* ci: move the if statement
Signed-off-by: Abiola Ibrahim <[email protected]>
We have observed some users not realizing container data is lost when doing delete.
This change does not modify scripted behaviour, as -f / force toggles maintains same logic.
Signed-off-by: Abiola Ibrahim <[email protected]>

This conforms to the expected behaviour over the network and all
interactions with Incus should be via the IP address.
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
Signed-off-by: Abiola Ibrahim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet