Skip to content

feat: adapt requeuing when session is hibernated - #1185

Open
sgaist wants to merge 1 commit into
mainfrom
feat/improve-requeuing-timing
Open

feat: adapt requeuing when session is hibernated#1185
sgaist wants to merge 1 commit into
mainfrom
feat/improve-requeuing-timing

Conversation

@sgaist

@sgaist sgaist commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Describe your changes

Currently, the operator checks on all sessions every 10 seconds.

This is not needed anymore once a pod has been hibernated.
Therefore queue for the MaxHibernationDuration.

If it's awaken before, there will be an event that will trigger an earlier reconciliation.

Once a pod has been hibernated, queue for the MaxHibernationDuration.
If it's awaken before, there will be an event and otherwise, it
avoids checking it every 10 seconds.
@sgaist
sgaist requested review from a team and olevski as code owners July 21, 2026 13:57

@leafty leafty left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This change prevents prometheus metrics to be collected for a paused session.

See:

RecordAmaltheaSessionMetrics(amaltheasession)

@sgaist

sgaist commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator Author

Thinking about this a bit more:

  • Can we consider bringing the interval to say 60s ? I don't think there's a need for a lower resolution check for hibernated sessions
  • Should we consider using a different mechanism to log hibernated sessions statistics ?

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