Skip to content

Adding energy_tracer.add_pstate_change in server_on_turn_onoff_hosts#77

Closed
marcodamico wants to merge 2 commits into
oar-team:mainfrom
marcodamico:energy-tracer-fix
Closed

Adding energy_tracer.add_pstate_change in server_on_turn_onoff_hosts#77
marcodamico wants to merge 2 commits into
oar-team:mainfrom
marcodamico:energy-tracer-fix

Conversation

@marcodamico

@marcodamico marcodamico commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

Read contributing guidelines.

Describe what the pull request does
Adding a missing call of energy_tracer.add_pstate_change in server_on_turn_onoff_hosts.
I am not 100% sure it is necessary, but the old code did it.

This branch already includes #76, so it should be approved after #76 is merged.

Checklist

Branch name.

  • Descriptive and short
  • Use hyphens to separate words

Branch content.

  • Only dedicated to the problem.
  • Based on Batsim's official main branch.
  • Straightforward. Just a sequence of commits. Does not contain merge commits.
  • Test results are not worse than before. How to run Batsim tests?

@bleuse

bleuse commented Apr 15, 2026

Copy link
Copy Markdown
Collaborator

Commit 9f1f914 looks like a good catch indeed.
I'll see how we deal with the integration of this commit regardless of the status of #76 or this PR.

@Mommessc

Copy link
Copy Markdown
Collaborator

The second commit has been cherry-picked into the main branch (713313e), thank you!

@Mommessc Mommessc closed this Apr 16, 2026
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.

3 participants