Skip to content

feat: Add new workflow to update the maintainers servers list#3010

Merged
qdm12 merged 13 commits intoqdm12:masterfrom
shwoop:shwoop-update-servers-list
Mar 3, 2026
Merged

feat: Add new workflow to update the maintainers servers list#3010
qdm12 merged 13 commits intoqdm12:masterfrom
shwoop:shwoop-update-servers-list

Conversation

@shwoop
Copy link
Copy Markdown
Contributor

@shwoop shwoop commented Nov 24, 2025

Description

A new workflow has been created that will run the update servers action in maintainer mode, then raise/merge a PR with the changes.

Trivial checks are performed:

  • the static servers list file has been altered
  • no other files have been altered
  • the static servers list file is valid json

The workflow will run once every two months, on the first of the month, at 03:11UTC.
The workflow can also be triggered manually.

You can see some invocations on my fork here: https://github.com/shwoop/gluetun/actions/workflows/update-servers-list.yml
An example merged PR (since reverted): shwoop#16

Issue

#2919

Assertions

  • I am aware that we do not accept manual changes to the servers.json file
  • I am aware that any changes to settings should be reflected in the wiki

@shwoop shwoop marked this pull request as ready for review November 24, 2025 14:55
Copy link
Copy Markdown
Owner

@qdm12 qdm12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome thanks!!! 💯

@shwoop shwoop force-pushed the shwoop-update-servers-list branch from c31178d to 80616b0 Compare December 1, 2025 09:44
@shwoop
Copy link
Copy Markdown
Contributor Author

shwoop commented Dec 1, 2025

Updated testing:

To generate the PR, tha now reverted changes were applied to run in the forked repo, triggered by the PR, and ignoring the checks regarding file chages. 80616b0

@shwoop shwoop requested a review from qdm12 December 1, 2025 10:29
@qdm12 qdm12 mentioned this pull request Dec 23, 2025
2 tasks
@qdm12 qdm12 force-pushed the master branch 3 times, most recently from 27f74e4 to fe3d4a9 Compare January 24, 2026 17:56
@qdm12 qdm12 force-pushed the master branch 2 times, most recently from d0247a1 to 0eeee5c Compare February 25, 2026 04:24
Copy link
Copy Markdown
Owner

@qdm12 qdm12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's get this party started!!! 💯

Sorry I took forever to re-review this

@qdm12 qdm12 merged commit 11cd62f into qdm12:master Mar 3, 2026
2 checks passed
mort666 pushed a commit to mort666/gluetun that referenced this pull request Mar 8, 2026
@shwoop shwoop deleted the shwoop-update-servers-list branch March 12, 2026 20:10
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