docker.el icon indicating copy to clipboard operation
docker.el copied to clipboard

Polls / New features

Open Silex opened this issue 3 years ago • 16 comments

This issue is there for when we need feedback on the package's direction.

Feel free to subscribe/unsubscribe from it.

Silex avatar Oct 06 '21 08:10 Silex

@spacegoing @davidshen84 @joshbax189 @marsam @Yuki-Inoue @gonewest818 @benswift @abcdw @stig

Silex avatar Oct 06 '21 08:10 Silex

[POLL] docker-machine is deprecated. I was thinking of removing support for it. Make some noise if you disagree :wink:

Silex avatar Oct 06 '21 08:10 Silex

Re-adding support for podman would be nice.

twmr avatar Oct 07 '21 00:10 twmr

There are alive fork of docker-machine maintained by gitlab. It will be nice if you restore support for it, but makes it optional.

codewylle avatar Oct 11 '21 18:10 codewylle

@codewylle interesting, but it still does not support json in their Go template format. This makes it a pain because we have to handle it separatly , and apparently you seem to be the only user.

foo@host:~$ ./docker-machine-Linux-x86_64 ls --format="{{ .Id }}"
foo@host:~$ ./docker-machine-Linux-x86_64 ls --format="{{ json .Id }}"
template: :1: function "json" not defined

But if it's valuable to you I can help you in making it a standalone package that you could maintain?

Silex avatar Oct 13 '21 12:10 Silex

@Silex thank you for readying to help. I will try to.

How can I inject docker-machine to docker transient?

codewylle avatar Oct 15 '21 17:10 codewylle

@codewylle: let's continue in #168

Silex avatar Oct 15 '21 18:10 Silex

Support for "devcontainers.json" ?

devcontainer.json becomes a bit of standard way of describing "development containers." Parts of it is Visual Studio Code specific, but parts are not.

https://code.visualstudio.com/docs/remote/devcontainerjson-reference

I think Emacs should have a way to handle it in some form. (= build the Docker image it "describes") Maybe this package is the right place.

devcontainer.json files appear in lots of projects.

behrica avatar Nov 28 '21 22:11 behrica

@behrica: That looks like a lot of work. What support do you have in mind? It's the first time I encounter it.

Silex avatar Nov 29 '21 07:11 Silex

I only think about a small subset, of "fields" from the devcontainer.json file

Firstly there is an existing cli tool, which should be "called" from emacs:

https://code.visualstudio.com/docs/remote/devcontainer-cli

This does a big chunk of the work already. It does a image build, as vscode would do.

This is the "first stage build". Then we would only need code, which does something with 5 fields from the "devcontainer.json";

  • postCreateCommand
  • postStartCommand
  • postAttachCommand
  • initializeCommand
  • onCreateCommand

-> 5 times "docker exec ..."

and the there are some options, which "configure" how the final "docker run" is executed:

  • mounts
  • ports ...

I have initial code for this, which basically is a sequence of calls (5 or 6) to "docker ....

This would therefore simulate what VSCode does when it "builds and starts" a devcontainer from a devcontainer.json file. All the VSCode specific fields in there, Emacs would ignore.

I was starting to build this as a standalone CLI tool (which maybe makes more sense) I am rather convinced, that "any IDE" on mid-term will support the "devcontainer.json" format, due to the popularity and reach of VSCode.

Curious about any comment from your side.

behrica avatar Nov 29 '21 08:11 behrica

Continuing in #174

Silex avatar Nov 29 '21 09:11 Silex

Option to use vterm (full-blown terminal emulator) with various commands.

noorul avatar Jan 15 '22 06:01 noorul

@noorul continuing in #179

Silex avatar Feb 08 '22 10:02 Silex

Just a heads-up: I'm working on the async implementation and I found aio which produce very clean code (compare https://github.com/Silex/docker.el/blob/aio/docker-network.el#L90-L94 vs https://github.com/Silex/docker.el/blob/async/docker-network.el#L115-L128 (the comparison is not completely fair but you get the idea: waiting on multiple callbacks is a pain in vanilla emacs)).

The thing is aio needs Emacs 26, so that means docker.el will require Emacs 26.

It's probably not a big deal for anyone but I was surprised in the past so let's see what happens :laughing:

Silex avatar Feb 08 '22 11:02 Silex

[POLL] docker-machine is deprecated. I was thinking of removing support for it. Make some noise if you disagree wink

Funny, because podman recently implemented new things to it's "podman machine" calls. For example, look at https://github.com/containers/podman/releases/ and you see in Podman v4.1.0 from May 2022 you see several enhancements to "podman machine".

Personally I don't care about this poll entry, as I don't use "podman machine".

holgerschurig avatar Jun 15 '22 07:06 holgerschurig

support for docker image history command

noorul avatar Jun 18 '22 04:06 noorul