Skip to content

expire_jobs: Also expire 'unknown' jobs#139

Merged
zmc merged 1 commit intomainfrom
expire-unknown
Jan 22, 2026
Merged

expire_jobs: Also expire 'unknown' jobs#139
zmc merged 1 commit intomainfrom
expire-unknown

Conversation

@zmc
Copy link
Member

@zmc zmc commented Jan 22, 2026

No description provided.

Signed-off-by: Zack Cerza <zack@cerza.org>
@dmick dmick self-requested a review January 22, 2026 19:37
Copy link
Member

@dmick dmick left a comment

Choose a reason for hiding this comment

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

I'd have to understand a lot more about what 'expiring' really means to understand if there a consequences or leftover resources that change from the different types, but this certainly adds a parallel 'unknown' type.

@zmc
Copy link
Member Author

zmc commented Jan 22, 2026

I'd have to understand a lot more about what 'expiring' really means to understand if there a consequences or leftover resources that change from the different types, but this certainly adds a parallel 'unknown' type.

it simply sets the status to dead
https://github.com/ceph/paddles/pull/139/files#diff-9912df481fb01bb9b54c418a26a9e11400c416a805f72bc80af3f4af58e58e50R54-R55

@zmc zmc merged commit acc3c57 into main Jan 22, 2026
3 checks passed
@zmc zmc deleted the expire-unknown branch January 22, 2026 19:46
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