Skip to content

Conversation

uosjead
Copy link

@uosjead uosjead commented Oct 10, 2025

Description

Add connection_max_age as a configurable option to the runner config

Migrations required

No

Verification

Please mention how you test the changes.

Copy link
Contributor

Hey @uosjead! 👋

Thank you for your contribution to the project. Please refer to the contribution rules for a quick overview of the process.

Make sure that this PR clearly explains:

  • the problem being solved
  • the best way a reviewer and you can test your changes

With submitting this PR you confirm that you hold the rights of the code added and agree that it will published under this LICENSE.

The following ChatOps commands are supported:

  • /help: notifies a maintainer to help you out

Simply add a comment with the command in the first line. If you need to pass more information, separate it with a blank line from the command.

This message was generated automatically. You are welcome to improve it.

maximum_concurrent_jobs = optional(number, 10)
prometheus_listen_address = optional(string, "")
sentry_dsn = optional(string, "__SENTRY_DSN_REPLACED_BY_USER_DATA__")
connection_max_age = optional(string, "15m")
Copy link
Author

Choose a reason for hiding this comment

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

@kayman-mk
Copy link
Collaborator

Could you please add some context? What problems have you seen?

@uosjead
Copy link
Author

uosjead commented Oct 10, 2025

Could you please add some context? What problems have you seen?

Yes - currently seeing some occasional connectivity issues running behind a NAT gateway - where connections are limited to 5 minutes. The gitlab-runner config by defaults limits connections to 15 minutes. Was looking to set the connection_max_age setting with the module, but could not find an option to do so.

Let me know if you need any further information.

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