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

[ENHANCEMENT]: better windows user clarity for quick start #399

Open
drdamour opened this issue Jul 7, 2023 · 5 comments
Open

[ENHANCEMENT]: better windows user clarity for quick start #399

drdamour opened this issue Jul 7, 2023 · 5 comments
Labels
enhancement New feature or request feature request

Comments

@drdamour
Copy link

drdamour commented Jul 7, 2023

Description

maybe i'm missing it, but quickstart assumes a lot, i'm thinking it assumes you have docker desktop set with the wsl backend so the containers can work as they appear to only be linux platform. is there a way to comfigure tmctl to work with a windows docker engine, or to leverage a wsl hosted docker engine without having to run tmctl in wsl directly?

Screenshots

Additional context

@drdamour drdamour added the enhancement New feature or request label Jul 7, 2023
@tzununbekov
Copy link
Member

@drdamour there are no direct requirements in tmctl for Windows to have Docker Desktop or WSL. Could you share your experience with the windows-version of tmctl on the host with the docker engine only?

@drdamour
Copy link
Author

drdamour commented Jul 20, 2023

it can't find the container because it's looking for a linux container. docker engine without wsl only runs windows containers AFAIK. do you offer your stuff in windows containers?

@tzununbekov
Copy link
Member

I am confused because linux containers (lxc) are nowhere near the CLI codebase. CLI uses Docker's client that provides the OS abstraction layer and I was pretty sure it should work on Windows with Docker engine. I don't currently have a Windows box in my hands but I'll try to sort this out and test it. Meanwhile, could you share the exact tmctl command and error that you receive in the Windows terminal? Thanks.

@drdamour
Copy link
Author

drdamour commented Jul 21, 2023

think your confusion lies within you thinking docker desktop and docker engine are same thing. i'm not docker expert because i use podman, but what docker desktop brought to the game was the mobyvm that delegated running the daeomon and linux containers in...which has now been supplanted by leveraged wsl2.

from https://learn.microsoft.com/en-us/virtualization/windowscontainers/deploy-containers/linux-containers

image

if there's a way to tell docker engine to use mobyvm...i dunno

b4 the days of the wsl2 engine you use to have to switch to linux containers in docker desktop (which fired up the mobyvm). i opened this ticket so you could make your requiremenets more clear.

here's the error you get with just docker engine

C:\Users\damourc\Downloads\tmctl_windows_amd64> ./tmctl create broker foo
2023/07/21 09:33:53 jde-catalyst | Updating manifest
2023/07/21 09:33:53 jde-catalyst | Starting container
Error: docker create: Error response from daemon: No such image: gcr.io/triggermesh/memory-broker:v1.4.0
Usage:
  tmctl create broker <name> [flags]

Examples:
tmctl create broker foo

Flags:
  -h, --help             help for broker
      --version string   TriggerMesh broker version. (default "v1.4.0")

2023/07/21 09:33:54 docker create: Error response from daemon: No such image: gcr.io/triggermesh/memory-broker:v1.4.0

the not foudn is cause it's looking for a windows architecture.

@tzununbekov
Copy link
Member

Ah, now I get the issue! My initial understanding was that our CLI does not work on Windows because it cannot find containers. But now I see that it simply cannot pull container images because we don't build them for the Windows platform. Hypothetically, we could build our components for Windows OS, but updating the docs to clarify the runtime requirements seems like a preferable option right now. /cc @jmcx, let's talk about this on our next call.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature request
Projects
None yet
Development

No branches or pull requests

3 participants