[Egg] Force updating egg doesn't update server variables correctly #119

Closed
opened 2026-02-05 17:09:30 +03:00 by OVERLORD · 1 comment
Owner

Originally created by @notAreYouScared on GitHub (Jun 27, 2024).

Issue 1: Force Importing over an existing egg that has new variables; new variables are not added to existing servers... The server will have to be removed and re-created.

Issue 2: Force Importing over an existing egg with another egg that is not the same breaks all servers due to all the variables being deleted and re-created, giving them new variables_ids in the egg_variable table and not updating them in the server_variables table.

Issue 1 can be reproduced by importing a new egg with one or more variables. Create a new server, export an egg, then add one or more variables to the json file and force importing over the existing egg. Verify the added variables are on the egg variables screen. Go to the server assigned to this egg and see that the newly added variables are not present. 

Issue 2 can be reproduced by importing a Minecraft egg, creating a server, force-importing rust or another egg other than the original egg, and attempting to view the server that's assigned that egg. 

image

Originally created by @notAreYouScared on GitHub (Jun 27, 2024). Issue 1: Force Importing over an existing egg that has new variables; new variables are not added to existing servers... The server will have to be removed and re-created. Issue 2: Force Importing over an existing egg with another egg that is not the same breaks all servers due to all the variables being deleted and re-created, giving them new `variables_id`s in the `egg_variable` table and not updating them in the `server_variable`s table. Issue 1 can be reproduced by importing a new egg with one or more variables. Create a new server, export an egg, then add one or more variables to the json file and force importing over the existing egg. Verify the added variables are on the egg variables screen. Go to the server assigned to this egg and see that the newly added variables are not present.  Issue 2 can be reproduced by importing a Minecraft egg, creating a server, force-importing rust or another egg other than the original egg, and attempting to view the server that's assigned that egg.  ![image](https://github.com/pelican-dev/panel/assets/1757840/ffa6caeb-778e-42b7-9a65-00075a2f9ba2)
Author
Owner

@notAreYouScared commented on GitHub (Jul 21, 2024):

Closing in favor of new issue.

@notAreYouScared commented on GitHub (Jul 21, 2024): Closing in favor of new issue.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/panel-pelican-dev#119