Skip to content

Conversation

@jturel
Copy link
Contributor

@jturel jturel commented Jan 19, 2026

The first request to hit the server that initializes dynflow is excruciatingly long. It turns out that dynflow disconnects the db pool before maybe increasing the pool size. In katello devel the database.yml pool size matches dynflow's target size which highlights the problem.

Ping illustrates it perfectly:

Before:

foreman_tasks:
    Status:          ok
    Server Response: Duration: 10137ms

After:

foreman_tasks:
    Status:          ok
    Server Response: Duration: 117ms

@jturel
Copy link
Contributor Author

jturel commented Jan 19, 2026

@adamruzicka what do you think?

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

In general I don't have any objections, thank you for polishing something that has been sitting there for almost a decade.

The comments are not hard requirements, but rather "while we're at it"-style comments

Copy link
Contributor

@adamruzicka adamruzicka left a comment

Choose a reason for hiding this comment

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

Looks good

@adamruzicka adamruzicka merged commit f64d11c into Dynflow:master Jan 21, 2026
12 checks passed
@adamruzicka
Copy link
Contributor

Thank you @jturel !

@jturel jturel deleted the db_pool_disconnect branch January 21, 2026 15:42
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