Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Add: Icarus-Dedicated-Server #1996

Merged
merged 8 commits into from
Dec 9, 2022

Conversation

BolverBlitz
Copy link
Contributor

Description

Adds the current Icarus Dedicated Server (Beta) to run in pterodactyl via wine.

Checklist for all submissions

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you tested and reviewed your changes with confidence that everything works?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:

New egg Submissions

  1. Does your submission pass tests (server is connectable)?
  2. Does your egg use a custom docker image?
    • Have you tried to use a generic image?
    • Did you PR the necessary changes to make it work?
  3. Have you added the egg to the main README.md and any other README files in subdirectories of the egg (e.g /game_eggs) according to the alphabetical order?
  4. Have you added a unique README.md for the egg you are adding according to the alphabetical order?

@QuintenQVD0
Copy link
Collaborator

QuintenQVD0 commented Dec 9, 2022

@BolverBlitz the apt update and apt install line are still there

If you read the comment in our script it tells you if you use the debian installer that both lines are not needed

@BolverBlitz
Copy link
Contributor Author

Ok, sorry. I didn´t read that command.

@QuintenQVD0
Copy link
Collaborator

QuintenQVD0 commented Dec 9, 2022

and a other note, the app id should be in a variable as that variable is used to auto update the server. for that read the comment on the first lines of the steamcmd install script

And there is also no auto update variable

(SRCDS_APPID , AUTO_UPDATE )

@gOOvER
Copy link
Contributor

gOOvER commented Dec 9, 2022

The DefaultGame.ini is Not shipped with the Server. you need to pull it from their repo

@BolverBlitz
Copy link
Contributor Author

@gOOvER I can´t find this file on the discord or the the Github so i don´t know what i´m supposed to do.

@QuintenQVD0
Copy link
Collaborator

@BolverBlitz It looks like as goover suggested, the ServerSettings.ini must be pulled from somewhere or added to this pr and let the egg install script pull it from here. It also looks like -UserDir= is required because the default save game is here %localappdata%\Icarus\ Maybe wine will Handel that I have no idea

@BolverBlitz
Copy link
Contributor Author

BolverBlitz commented Dec 9, 2022

@QuintenQVD0 The ServerSettings.ini file should be generated. And all my test deploys do have it.
grafik

From the dev Wiki: NOTE: As of current the settings file gets generated on first prospect/lobby
Since they try to include it anyway and the server beeing in beta, i see no reason to add it to the install skript.

You might be correct with the save game, however in the current state it does save all placed down things in a world (Restart persistent)
Looking into my files, its stored in /Icarus/Saved/PlayerData/DedicatedServer/Prospects/xxx.json

@QuintenQVD0
Copy link
Collaborator

If it auto generates that config and auto saves to a path in /home/container then it is fine that way, then no modfications are needed

@QuintenQVD0
Copy link
Collaborator

If you can just provide a full startup log, then It is fine for me, Then I will merge it

@BolverBlitz
Copy link
Contributor Author

@QuintenQVD0
Copy link
Collaborator

https://nc.ebg.pw/fetch/6ffb7321log091222

what is this at the end?

0258:fixme:advapi:ReportEventW (00000000CAFE4242,0x0001,0x0000,0xc0000001,0000000000000000,0x0001,0x00000000,0000000000B9CC88,0000000000000000): stub
0258:err:eventlog:ReportEventW L"OnResUserTicket : No player found"
0298:fixme:cryptnet:check_ocsp_response_info check responder id

@BolverBlitz
Copy link
Contributor Author

BolverBlitz commented Dec 9, 2022

I don´t know exactly. Looking it up on the discord it seems perfectly fine for now.
The other 2 messages i did not find on the discord.

I tested every commit by joining, building, saving, restarting and verifying the stuff is there.
grafik

@QuintenQVD0 QuintenQVD0 self-requested a review December 9, 2022 17:21
@parkervcp parkervcp merged commit 895a2e0 into pelican-eggs:master Dec 9, 2022
@gOOvER
Copy link
Contributor

gOOvER commented Dec 9, 2022

@BolverBlitz It looks like as goover suggested, the ServerSettings.ini must be pulled from somewhere or added to this pr and let the egg install script pull it from here. It also looks like -UserDir= is required because the default save game is here %localappdata%\Icarus\ Maybe wine will Handel that I have no idea

userdir is Not needed and make Problems.

i will post more suggetions tomorrow..i have nightshitft and must Work atm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants