Skip to content

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

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

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

Conversation

@ianballou

Copy link
Copy Markdown
Contributor

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

@ianballou

Copy link
Copy Markdown
Contributor Author

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

@evgeni

evgeni commented Feb 25, 2026

Copy link
Copy Markdown
Member

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

evgeni commented Feb 25, 2026

Copy link
Copy Markdown
Member

@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

ehelms commented Feb 25, 2026

Copy link
Copy Markdown
Member

I think not.

@ianballou

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

evgeni commented Feb 26, 2026

Copy link
Copy Markdown
Member

Install puppet strings

@evgeni

evgeni commented Feb 26, 2026

Copy link
Copy Markdown
Member

or let's add packit: #533

@ianballou

Copy link
Copy Markdown
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
Copy Markdown
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