Skip to content

Conversation

@SQLJames
Copy link
Owner

@SQLJames SQLJames commented Nov 9, 2024

Fix for #45

@SQLJames
Copy link
Owner Author

SQLJames commented Nov 9, 2024

QA involved downloading a specific version of a mod, getting the latest and then also making sure that it respects the update mod flag

kubectl logs -n factorio -l app=factorio-factorio-server-charts -c download-factorio-mods --tail=-1
+ echo 'Running mod-downloader.sh script...'
+ bash /scripts/mod-downloader.sh
Running mod-downloader.sh script...
{"level":"info","ts":1731115924.239907,"caller":"download/official.go:59","msg":"Download Official Params","Name":"factorissimo-2-notnotmelon","Name":"factorissimo-2-notnotmelon","destination":"/factorio/mods","version":"3.5.11","factorioVersion":"v2.0","user":"xxxxxxxxxxxxxxxxxxxx","token":"xxxxxxxxxxxxxxxxxxxx"}
{"level":"info","ts":1731115924.2399685,"caller":"download/official.go:67","msg":"built url","url":"https://mods.factorio.com/api/mods/factorissimo-2-notnotmelon?username=xxxxxxxxxxxxxxxxxxxx&token=xxxxxxxxxxxxxxxxxxxx"}
{"level":"info","ts":1731115924.4533417,"caller":"download/official.go:100","msg":"Version supplied, attempting to pull that version."}
{"level":"info","ts":1731115924.4534082,"caller":"download/official.go:110","msg":"Viable Release","file":"factorissimo-2-notnotmelon_3.5.11.zip"}
Mod update completed.
+ echo 'Finished mod-downloader.sh script'
+ ls -althR /factorio/mods
Finished mod-downloader.sh script
/factorio/mods:
total 5.1M
drwxr-sr-x 2  845 845 4.0K Nov  9 01:32 .
-rw-r--r-- 1  845 845    0 Nov  9 01:32 .mod_update_complete
-rw-r--r-- 1  845 845 5.1M Nov  9 01:32 factorissimo-2-notnotmelon_3.5.11.zip
drwxrwsr-x 6 root 845 4.0K Nov  9 01:32 ..
-rw-r--r-- 1  845 845    7 Nov  9 01:32 factorioVersion
kubectl logs -n factorio -l app=factorio-factorio-server-charts -c download-factorio-mods --tail=-1
Running mod-downloader.sh script...
+ echo 'Running mod-downloader.sh script...'
+ bash /scripts/mod-downloader.sh
{"level":"info","ts":1731116431.7871141,"caller":"download/official.go:59","msg":"Download Official Params","Name":"factorissimo-2-notnotmelon","Name":"factorissimo-2-notnotmelon","destination":"/factorio/mods","version":"","factorioVersion":"v2.0","user":"xxxxxxxxxxxxxxxxxxxx","token":"xxxxxxxxxxxxxxxxxxxx"}
{"level":"info","ts":1731116431.787194,"caller":"download/official.go:67","msg":"built url","url":"https://mods.factorio.com/api/mods/factorissimo-2-notnotmelon?username=xxxxxxxxxxxxxxxxxxxx&token=xxxxxxxxxxxxxxxxxxxx"}
{"level":"info","ts":1731116431.999727,"caller":"download/official.go:110","msg":"Viable Release","file":"factorissimo-2-notnotmelon_3.5.13.zip"}
Mod update completed.
Finished mod-downloader.sh script
+ echo 'Finished mod-downloader.sh script'
+ ls -althR /factorio/mods
/factorio/mods:
total 5.1M
drwxr-sr-x 2  845 845 4.0K Nov  9 01:40 .
-rw-r--r-- 1  845 845    0 Nov  9 01:40 .mod_update_complete
-rw-r--r-- 1  845 845 5.1M Nov  9 01:40 factorissimo-2-notnotmelon_3.5.13.zip
drwxrwsr-x 6 root 845 4.0K Nov  9 01:40 ..
-rw-r--r-- 1  845 845    7 Nov  9 01:40 factorioVersion
kubectl logs -n factorio -l app=factorio-factorio-server-charts -c download-factorio-mods --tail=-1
+ echo 'Running mod-downloader.sh script...'
+ bash /scripts/mod-downloader.sh
Running mod-downloader.sh script...
Mod update already completed previously. Skipping.
Finished mod-downloader.sh script
+ echo 'Finished mod-downloader.sh script'
+ ls -althR /factorio/mods
/factorio/mods:
total 5.1M
-rw-rw-r--  1  845 845    7 Nov  9 01:46 factorioVersion
drwxrwsr-x 11 root 845 4.0K Nov  9 01:46 ..
drwxrwsr-x  2  845 845 4.0K Nov  9 01:44 .
-rw-rw-r--  1  845 845  373 Nov  9 01:44 mod-list.json
-rw-rw-r--  1  845 845  570 Nov  9 01:40 mod-settings.dat
-rw-rw-r--  1  845 845    0 Nov  9 01:40 .mod_update_complete
-rw-rw-r--  1  845 845 5.1M Nov  9 01:40 factorissimo-2-notnotmelon_3.5.13.zip

@SQLJames SQLJames merged commit 473b334 into main Nov 9, 2024
4 checks passed
@SQLJames SQLJames deleted the feature/factorio-mod-downloader branch November 10, 2024 13:04
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.

2 participants