Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

manager: fix "unknown resources cleanup" traceback #131

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

praiskup
Copy link
Owner

@praiskup praiskup commented Nov 5, 2023

When a new on-demand pool is configured in pools.yaml, the cleanup logic is typically executed before a new resource is started there, and thus also before the database entry is created.

Fixes: #128

@praiskup praiskup force-pushed the praiskup-unknown-pool-traceback branch 2 times, most recently from ace8f11 to 8b02e21 Compare November 5, 2023 07:18
When a new on-demand pool is configured in pools.yaml, the cleanup logic
is typically executed before a new resource is started there, and thus
also before the database entry is created.

Fixes: #128
Fixes: #115
@praiskup praiskup force-pushed the praiskup-unknown-pool-traceback branch from 8b02e21 to a8635ff Compare November 6, 2023 14:26
@praiskup praiskup merged commit de133c4 into main Nov 6, 2023
2 checks passed
@praiskup praiskup deleted the praiskup-unknown-pool-traceback branch January 10, 2024 15:59
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.

Freshly added "on demand" pools break manager
1 participant