Skip to content

feat: Docker network management with resource attachment#1

Closed
aladdinb1 wants to merge 3 commits into
canaryfrom
feat/network-management
Closed

feat: Docker network management with resource attachment#1
aladdinb1 wants to merge 3 commits into
canaryfrom
feat/network-management

Conversation

@aladdinb1
Copy link
Copy Markdown
Collaborator

Summary

  • Adds Docker network management UI (Networks page) with create/edit/delete based on upstream Feat/add network management Dokploy/dokploy#3774, plus the four Greptile-flagged fixes from the original PR
  • Resource-level attachment: applications and 6 database types (libsql, mariadb, mongo, mysql, postgres, redis) can attach to custom networks via the Advanced tab. Compose intentionally omitted (compose attaches via its YAML, not Swarm spec)
  • Server-scoped networks: networks created on a custom server are routed to that Docker daemon over SSH; the resource picker filters to networks on the same server
  • 15 schema + service tests borrowed from PR feat!: Network Management System v1 Dokploy/dokploy#2811 patterns

Test plan

  • Unit tests pass (__test__/network/* — 15 cases)
  • E2E on local Dokploy host: create network, attach to app, deploy, verify swarm service joined network
  • E2E on remote Hetzner CX23 server: remote-test-net appears only on remote daemon, my-custom-net only on local — confirms server-scoped routing
  • Resource picker correctly filters networks when app moved between servers (verified live)

@autofix-troubleshooter
Copy link
Copy Markdown

Hi! I'm the autofix logoautofix.ci troubleshooter bot.

It looks like you correctly set up a CI job that uses the autofix.ci GitHub Action, but the autofix.ci GitHub App has not been installed for this repository. This means that autofix.ci unfortunately does not have the permissions to fix this pull request. If you are the repository owner, please install the app and then restart the CI workflow! 😃

@BSalaeddin BSalaeddin force-pushed the feat/network-management branch from 4073f0b to dde344b Compare May 18, 2026 22:01
@BSalaeddin BSalaeddin closed this May 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants