Skip to content

Fixes #39093 - add container gateway db connection options#532

Merged
evgeni merged 1 commit intotheforeman:masterfrom
ianballou:39093-container-connections-fpc
Feb 26, 2026
Merged

Fixes #39093 - add container gateway db connection options#532
evgeni merged 1 commit intotheforeman:masterfrom
ianballou:39093-container-connections-fpc

Conversation

@ianballou
Copy link
Contributor

Exposes the new container gateway DB tuning options: theforeman/puppet-foreman_proxy#896

@ianballou
Copy link
Contributor Author

I think the tests are going to fail until theforeman/puppet-foreman_proxy#896 is merged.

@evgeni
Copy link
Member

evgeni commented Feb 25, 2026

I think the tests are going to fail until theforeman/puppet-foreman_proxy#896 is merged.

Correct.
Plus you need to bump the minimum version of that to 30.1.0 for theforeman/puppet-foreman_proxy#897

@evgeni
Copy link
Member

evgeni commented Feb 25, 2026

@ekohl @ehelms is the fact that we need to bump the lower bound for foreman_proxy a breaking change? I think not, but wanted to be sure.

@ehelms
Copy link
Member

ehelms commented Feb 25, 2026

I think not.

@ianballou
Copy link
Contributor Author

Correct.
Plus you need to bump the minimum version of that to 30.1.0 for theforeman/puppet-foreman_proxy#897

This would probably explain my testing issues, nice, I'll fix it up.

@ianballou ianballou force-pushed the 39093-container-connections-fpc branch from 4b76a4d to 4533cff Compare February 25, 2026 20:59
@ianballou
Copy link
Contributor Author

Plus you need to bump the minimum version of that to 30.1.0 for theforeman/puppet-foreman_proxy#897

Done!

@ianballou
Copy link
Contributor Author

Correct.
Plus you need to bump the minimum version of that to 30.1.0 for theforeman/puppet-foreman_proxy#897

This would probably explain my testing issues, nice, I'll fix it up.

Eh, never mind, whenever I apply the changes here, I end up with kafo complaining that no Puppet module parser is installed. Any tips on how to test this to make sure we're fully set?

@evgeni
Copy link
Member

evgeni commented Feb 26, 2026

Install puppet strings

@evgeni
Copy link
Member

evgeni commented Feb 26, 2026

or let's add packit: #533

@ianballou
Copy link
Contributor Author

Perfect, I was able to run the installer after using puppet's gem binary to install puppet-strings.

# satellite-installer --full-help | grep container-gateway
    --foreman-proxy-content-container-gateway-database-max-connections                   Maximum number of database connections for the container gateway (current: UNDEF)
    --reset-foreman-proxy-content-container-gateway-database-max-connections             Reset container_gateway_database_max_connections to the default value (UNDEF)
    --foreman-proxy-content-container-gateway-database-pool-timeout                      Database connection pool timeout in seconds for the container gateway (current: UNDEF)
    --reset-foreman-proxy-content-container-gateway-database-pool-timeout                Reset container_gateway_database_pool_timeout to the default value (UNDEF)

@ianballou
Copy link
Contributor Author

# satellite-installer --foreman-proxy-content-container-gateway-database-max-connections 60  --foreman-proxy-content-container-gateway-database-pool-timeout 65
---
# Container Gateway for Katello
...
:sqlite_db_path: /var/lib/foreman-proxy/smart_proxy_container_gateway.db
:db_connection_string: postgres:///container_gateway
:db_max_connections: 60
:db_pool_timeout: 65

@evgeni evgeni merged commit 2949c46 into theforeman:master Feb 26, 2026
8 checks passed
@ianballou ianballou deleted the 39093-container-connections-fpc branch February 26, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants