starred/panel-pelican-dev
1
0
Fork 0
mirror of https://github.com/pelican-dev/panel.git synced 2026-05-04 18:00:48 +03:00
Code Issues 41 Packages Projects Releases 10 Wiki Activity

[PR #1584] [MERGED] Fix auto deploy docker command not including the container argument #1470

New Issue
Closed
opened 2026-02-05 18:14:54 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-05 18:14:54 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/pelican-dev/panel/pull/1584
Author: @IThundxr
Created: 8/10/2025
Status: ✅ Merged
Merged: 9/3/2025
Merged by: @rmartinoscar

Base: main ← Head: patch-1

📝 Commits (2)

  • f74d33f Fix auto deploy docker command not including the container argument
  • deabf38 Fix missing wings container name in AutoDeploy

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 app/Services/Nodes/NodeAutoDeployService.php (+1 -1)

📄 Description

docker exec requires the container as the argument, which currently the command doesn't pass so it gets parsed as: docker exec configure ... in container wings, rather then: docker exec wings configure ... in container wings.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/pelican-dev/panel/pull/1584 **Author:** [@IThundxr](https://github.com/IThundxr) **Created:** 8/10/2025 **Status:** ✅ Merged **Merged:** 9/3/2025 **Merged by:** [@rmartinoscar](https://github.com/rmartinoscar) **Base:** `main` ← **Head:** `patch-1` --- ### 📝 Commits (2) - [`f74d33f`](https://github.com/pelican-dev/panel/commit/f74d33fdd4dd57e7bf89c0b41899209053e8627f) Fix auto deploy docker command not including the container argument - [`deabf38`](https://github.com/pelican-dev/panel/commit/deabf38b886f17ff79f54a1897dc087852442164) Fix missing wings container name in AutoDeploy ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/Services/Nodes/NodeAutoDeployService.php` (+1 -1) </details> ### 📄 Description `docker exec` requires the container as the argument, which currently the command doesn't pass so it gets parsed as: docker exec configure ... in container wings, rather then: docker exec wings configure ... in container wings. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 18:14:54 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
not confirmed
pull-request
Mirrored from GitHub Pull Request
 👑 sponsor
💰 fund
🔴 hard
🟡 medium
🟡 medium
🟡 medium
🟢 easy
🟢 easy
🟢 easy
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#1470
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?