Skip to content

Comments

Fix/mutate job state when using flexmeasures jobs run-job#1929

Merged
Flix6x merged 5 commits intomainfrom
fix/mutate-job-state
Jan 26, 2026
Merged

Fix/mutate job state when using flexmeasures jobs run-job#1929
Flix6x merged 5 commits intomainfrom
fix/mutate-job-state

Conversation

@Flix6x
Copy link
Contributor

@Flix6x Flix6x commented Jan 26, 2026

Description

  • Have flexmeasures jobs run-job update job metadata and move between registries (e.g. to failed or finished)
  • Added changelog item in documentation/changelog.rst
  • Added changelog item in documentation/cli/change_log.rst

Look & Feel

  • Before this PR, when running a scheduling job, the job metadata would not contain the scheduler_info, because the job was not being executed within an rq context.
  • After this PR, the scheduler_info in the job's metadata is updated.

How to test

No new test added.

Further Improvements

None in scope (see related items).

Related Items

#1927 introduces the feature to run a job without changing anything, but that PR still needs work. This PR contains only the fix.

Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x Flix6x added this to the 0.31.0 milestone Jan 26, 2026
@Flix6x Flix6x requested a review from nhoening January 26, 2026 10:46
@Flix6x Flix6x self-assigned this Jan 26, 2026
@Flix6x Flix6x added bug Something isn't working CLI jobs labels Jan 26, 2026
@read-the-docs-community
Copy link

read-the-docs-community bot commented Jan 26, 2026

Documentation build overview

📚 flexmeasures | 🛠️ Build #31146490 | 📁 Comparing 667ae02 against latest (22fff1a)


🔍 Preview build

Show files changed (7 files in total): 📝 6 modified | ➕ 1 added | ➖ 0 deleted
File Status
changelog.html 📝 modified
genindex.html 📝 modified
py-modindex.html 📝 modified
_autosummary/flexmeasures.utils.html 📝 modified
_autosummary/flexmeasures.utils.job_utils.html ➕ added
api/v3_0.html 📝 modified
cli/change_log.html 📝 modified

Copy link
Contributor

@nhoening nhoening 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.
I believe that the PR title & description should mention that (in my understanding), this only affects this one CLI command.

@Flix6x Flix6x changed the title Fix/mutate job state Fix/mutate job state when using flexmeasures jobs run-job Jan 26, 2026
…using work_on_rq with positional arguments

Signed-off-by: F.N. Claessen <felix@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
@Flix6x Flix6x merged commit 0c78ee5 into main Jan 26, 2026
8 checks passed
@Flix6x Flix6x deleted the fix/mutate-job-state branch January 26, 2026 11:37
@Flix6x Flix6x mentioned this pull request Jan 26, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working CLI jobs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants