Skip to content

Space age values #50

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

Merged
merged 5 commits into from
Oct 23, 2024
Merged

Space age values #50

merged 5 commits into from
Oct 23, 2024

Conversation

Kariton
Copy link
Contributor

@Kariton Kariton commented Oct 22, 2024

fixes: #48

@Kariton Kariton requested a review from SQLJames as a code owner October 22, 2024 16:11
@Kariton
Copy link
Contributor Author

Kariton commented Oct 22, 2024

Changes based of: wube/factorio-data@1.1.110...2.0.8

@Kariton
Copy link
Contributor Author

Kariton commented Oct 22, 2024

I havent commented all @param values because i cant find any resources on them ATM

@SQLJames
Copy link
Owner

I believe the lint error for versions is not a str in the ci is related to the version in chart.yaml I believe it should be 2.0.0 not 2.0

@Kariton
Copy link
Contributor Author

Kariton commented Oct 22, 2024

hm.
server startet but still "rcon connection refused"

1.191 Info CommandLineMultiplayer.cpp:288: Maximum segment size = 100; minimum segment size = 25; maximum-segment-size peer count = 10; minimum-segment-size peer count = 20

dunno how i can debug / help here...

@SQLJames
Copy link
Owner

Based on what I can see the rcon port is 27015 but the test that it is running is trying to hit 30100

I am assuming they changed the file for the rcon configuration as well.

/bin/sh
      -ec
      RCONPW="$(cat /rconpw/rconpw)"
      ./rcon -a factorio-server-charts-n9ekxcwrpb-rcon:30100 -p $RCONPW -t rcon -T 180s /s Test

@Kariton
Copy link
Contributor Author

Kariton commented Oct 22, 2024

works on my machine. 🥇

# nc -v 172.19.11.1 30100
Ncat: Version 7.92 ( https://nmap.org/ncat )
Ncat: Connected to 172.19.11.1:30100.

log:

3715.829 Info RemoteCommandProcessor.cpp:248: New RCON connection from IP ADDR:({X.X:X:X:34954})

my rcon config is basically the values.yml default.

rcon:
  external: true
  type: LoadBalancer
  port: 30100
  passwordSecret: factorio-rcon-password

only updated whats within this PR...

# k get svc factorio-server-factorio-server-charts-rcon
NAME                                          TYPE           CLUSTER-IP      EXTERNAL-IP   PORT(S)           AGE
factorio-server-factorio-server-charts-rcon   LoadBalancer   10.42.159.241   172.19.11.1   30100:31365/TCP   74m
# k describe svc factorio-server-factorio-server-charts-rcon
Name:                     factorio-server-factorio-server-charts-rcon
Namespace:                factorio
Labels:                   app=factorio-server-factorio-server-charts
                          app.kubernetes.io/managed-by=Helm
                          chart=factorio-server-charts-1.2.5
                          heritage=Helm
                          release=factorio-server
Annotations:              field.cattle.io/publicEndpoints:
                            [{"addresses":["172.19.11.1"],"port":30100,"protocol":"TCP","serviceName":"factorio:factorio-server-factorio-server-charts-rcon","allNodes...
                          meta.helm.sh/release-name: factorio-server
                          meta.helm.sh/release-namespace: factorio
Selector:                 app=factorio-server-factorio-server-charts
Type:                     LoadBalancer
IP Family Policy:         SingleStack
IP Families:              IPv4
IP:                       10.42.159.241
IPs:                      10.42.159.241
LoadBalancer Ingress:     172.19.11.1
Port:                     rcon  30100/TCP
TargetPort:               27015/TCP
NodePort:                 rcon  31365/TCP
Endpoints:                10.41.4.47:27015
Session Affinity:         None
External Traffic Policy:  Cluster
Events:                   <none>

ill see what i can find later. it got early today. :)

@Kariton
Copy link
Contributor Author

Kariton commented Oct 23, 2024

it actually worked for me.... pipeline succeeded.

see: Kariton#1

can you please rerun yours?

@SQLJames
Copy link
Owner

Very strange, I ran it twice and first time it failed second time it worked.

@SQLJames SQLJames merged commit a31d030 into SQLJames:main Oct 23, 2024
2 checks passed
@zwolsman
Copy link

Release seems to fail on the main branch now, could you run it again? 😛

@SQLJames
Copy link
Owner

needed to update the pat for publishing releases its there now.

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

Successfully merging this pull request may close these issues.

Factorio: Space Age changes config values
3 participants