panel icon indicating copy to clipboard operation
panel copied to clipboard

Cannot transfer server to node that has no resources left despite overallocate being set to -1.

Open RealTriassic opened this issue 2 years ago • 2 comments

Current Behavior

Currently, if you try to transfer a server to a node that, according to the resource limits set on the panel for the node, has no resources left despite overallocate being set to -1, the panel will refuse and respond with "The node you selected does not have the required disk space or memory available to accommodate this server." despite overallocate being set to -1 in which case it should ignore the resource limits set on the panel and attempt the transfer.

Expected Behavior

The server transfer attempt should proceed, even if there are not enough resources, if the node's overallocate field is set to -1.

Steps to Reproduce

  1. Create a node on the panel
  2. Set overallocate to -1 for the node
  3. Try transfer a server that goes over the resource limits of the node

Panel Version

1.11.5

Wings Version

1.11.8

Games and/or Eggs Affected

No response

Docker Image

No response

Error Logs

No response

Is there an existing issue for this?

  • [X] I have searched the existing issues before opening this issue.
  • [X] I have provided all relevant details, including the specific game and Docker images I am using if this issue is related to running a server.
  • [X] I have checked in the Discord server and believe this is a bug with the software, and not a configuration issue with my specific system.

RealTriassic avatar Dec 04 '23 03:12 RealTriassic

Can confirm this. Setting overallocation to -1 is not actually disabling the checks. Furthermore, it acts as "underallocation" so it will use 99% of the allocated resources for the checks.

Either the descriptions of the fields are wrong or this feature got lost somewhere.

Relevant code lines: https://github.com/pterodactyl/panel/blob/1.0-develop/app/Models/Node.php#L228-L229 (for transfers) https://github.com/pterodactyl/panel/blob/1.0-develop/app/Services/Deployment/FindViableNodesService.php#L85-L86 (for deployment)

https://github.com/pterodactyl/panel/blob/1.0-develop/resources/views/admin/nodes/new.blade.php#L124 https://github.com/pterodactyl/panel/blob/1.0-develop/resources/views/admin/nodes/new.blade.php#L143

Boy132 avatar Dec 05 '23 09:12 Boy132