Skip to content

Commit 5a83b55

Browse files
committed
Merge branch '282-dump-dir-mounting' into 'master'
fix: remove the auto-mount of the dump location because it has to be done explicitly when starting the DLE Docker container (#282) Closes #282 See merge request postgres-ai/database-lab!315
2 parents f179fed + 63d3a2e commit 5a83b55

File tree

1 file changed

+0
-8
lines changed
  • pkg/retrieval/engine/postgres/logical

1 file changed

+0
-8
lines changed

pkg/retrieval/engine/postgres/logical/dump.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import (
1515

1616
"github.com/docker/docker/api/types"
1717
"github.com/docker/docker/api/types/container"
18-
"github.com/docker/docker/api/types/mount"
1918
"github.com/docker/docker/api/types/network"
2019
"github.com/docker/docker/client"
2120
"github.com/jackc/pgx/v4"
@@ -563,13 +562,6 @@ func (d *DumpJob) buildHostConfig(ctx context.Context) (*container.HostConfig, e
563562

564563
hostConfig.NetworkMode = d.getContainerNetworkMode()
565564

566-
// Mount the dump location directory.
567-
hostConfig.Mounts = append(hostConfig.Mounts, mount.Mount{
568-
Type: mount.TypeBind,
569-
Source: d.DumpOptions.DumpLocation,
570-
Target: d.DumpOptions.DumpLocation,
571-
})
572-
573565
return hostConfig, nil
574566
}
575567

0 commit comments

Comments
 (0)