starred/cli
1
0
Fork 0
mirror of https://github.com/docker/cli.git synced 2026-09-27 01:40:31 -04:00
Code Issues Packages Projects Releases Wiki Activity

cli/command/service: preserve mount order on force update

Browse Source 
Signed-off-by: Kamyar mofakhami <41609894+kawmy@users.noreply.github.com>
This commit is contained in:
Kamyar mofakhami
2026-08-21 18:54:45 +03:30
parent 28f756087e
commit a79a16bd63
2 changed files with 30 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
cli/command/service/update.go
+4 -2
View File
@@ -352,8 +352,10 @@ func updateService(ctx context.Context, apiClient client.NetworkAPIClient, flags
if err := updateIsolation(flagIsolation, &cspec.Isolation); err != nil {
return err
}
if err := updateMounts(flags, &cspec.Mounts); err != nil {
return err
if anyChanged(flags, flagMountAdd, flagMountRemove) {
if err := updateMounts(flags, &cspec.Mounts); err != nil {
return err
}
}
updateSysCtls(flags, &task.ContainerSpec.Sysctls)
cli/command/service/update_test.go
+26
View File
@@ -250,6 +250,32 @@ func TestUpdateMounts(t *testing.T) {
assert.Check(t, is.Equal("/tokeep", mounts[1].Target))
}
func TestUpdateServiceForcePreservesMountOrder(t *testing.T) {
flags := newUpdateCommand(nil).Flags()
assert.NilError(t, flags.Set("force", "true"))
spec := &swarm.ServiceSpec{
TaskTemplate: swarm.TaskSpec{
ContainerSpec: &swarm.ContainerSpec{
Mounts: []mount.Mount{
{Type: mount.TypeVolume, Source: "z-volume", Target: "/data/z"},
{Type: mount.TypeVolume, Source: "a-volume", Target: "/data/a"},
{Type: mount.TypeVolume, Source: "m-volume", Target: "/data/m"},
},
},
},
}
err := updateService(context.Background(), nil, flags, spec)
assert.NilError(t, err)
assert.Equal(t, spec.TaskTemplate.ForceUpdate, uint64(1))
assert.DeepEqual(t, spec.TaskTemplate.ContainerSpec.Mounts, []mount.Mount{
{Type: mount.TypeVolume, Source: "z-volume", Target: "/data/z"},
{Type: mount.TypeVolume, Source: "a-volume", Target: "/data/a"},
{Type: mount.TypeVolume, Source: "m-volume", Target: "/data/m"},
})
}
func TestUpdateMountsWithDuplicateMounts(t *testing.T) {
flags := newUpdateCommand(nil).Flags()
flags.Set("mount-add", "type=volume,source=vol4,target=/toadd")