Add custom volumes as an option#4401
Conversation
|
Minor UI changes to make this work:
|
|
Hey @jdrouet can you take a look at this PR? |
|
Is this feature forgotten? |
|
@Marahin I'm ready here to provide any updates to get this into kitematic. Unfortunately the contributors are showing bad faith by simply ignoring it. |
jdrouet
left a comment
jdrouet
left a comment
There was a problem hiding this comment.
didn't check the feature by itself yet but there are some changes to make in your PR ;)
| } else { | ||
| icon = <a onClick={this.handleRemoveVolume.bind(this, index)} className="only-icon btn btn-action small"><span | ||
| className="icon icon-delete"></span></a>; | ||
| } |
There was a problem hiding this comment.
we should probably take this piece of code out of this function
There was a problem hiding this comment.
Hmm, I'm not sure where to place it. Could I leave this one up to you? I was following the same code style as the other ContainerSetting pages:
|
I just wanted to drop-in and say thank you @jdrouet for finding time to check this out, it's something I'm looking forward to see in Kitematic. 👍 |
|
Thank you @jdrouet! |
|
Hey, is there any progress on reviewing this @jdrouet ? How can we help? cc @stereokai |
|
Ship it! 🚀Please 👍 |
|
come on, we need this feature in Kitematic |
|
@jdrouet so what's happening with this PR? |
|
🙌 |
|
🎉 |
|
@aaomidi Your PR has been approved |
|
Nothing on my side unfortunately. |
Merges Pull Request: docker-archive-public#3462 .. docker-archive-public#3692 .. docker-archive-public#4401 .. docker-archive-public#5306 They are Fix Login. and adding stuff. temporarily simplifying the building. and remove the test.
Merges Pull Request: docker-archive-public#3462 .. docker-archive-public#3692 .. docker-archive-public#4401 .. docker-archive-public#5306 They are Fix Login. and adding stuff. temporarily simplifying the building. and remove the test. add building on TC with tag. and pushing release to github.
|
What can be done to get this merged? What are the blockers? |
|
https://github.com/DockStation/dockstation fits the bill. |
|
Bump - pls merge |
|
What happened on releasing? It has been more than 1 year since the PR pushed ! |
|
So slow :/ |
|
If there is interest I'll push a release of the binaries to my fork... |
I instinctively checked your repo for releases but https://github.com/DockStation/dockstation was a nice workaround. With this slow merging I would say everyone should ignore kitematic and support dockstation instead. |
|
Yeah I don't think I want to be responsible for maintaining a fork of this :D |
14 participants
#1923 #4399
These issues were closed with no resolution. I decided to go ahead and make these changes.
The code changes how the volume section is handled to be the same as the environment variables. Allow custom volumes and deleting of any existing volumes.
This PR can be further extended to ensure that if a volume change broke Docker, it can revert back to a cached list of mounts.