Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
3 tasks done
RealTriassic opened this issue Dec 4, 2023 · 1 comment
Labels
not confirmed Report seems plausible but requires additional testing or 3rd part confirmation.

Comments

@RealTriassic
Copy link
Contributor

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?

  • I have searched the existing issues before opening this issue.
  • 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.
  • 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 RealTriassic added the not confirmed Report seems plausible but requires additional testing or 3rd part confirmation. label Dec 4, 2023
@Boy132
Copy link
Contributor

Boy132 commented Dec 5, 2023

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not confirmed Report seems plausible but requires additional testing or 3rd part confirmation.
Projects
None yet
Development

No branches or pull requests

2 participants