Skip to content

Add restart option for failed/dismissed jobs - #187

Open
freds-dev wants to merge 4 commits into
mainfrom
freds-dev/restart-job
Open

Add restart option for failed/dismissed jobs#187
freds-dev wants to merge 4 commits into
mainfrom
freds-dev/restart-job

Conversation

@freds-dev

@freds-dev freds-dev commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Related to #41

Paired with eo-tools/eozilla-app#72.

Adds POST /jobs/{jobId}/restart to create a new job from the original request of a failed or dismissed job.

  • Adds the restart operation to the OpenAPI contract and generated service/routes.
  • Supports restart in the local service by retaining the original process request.
  • Supports restart in the Airflow service by replaying the original DAG-run configuration.
  • Rejects restarts for accepted, running, and successful jobs.

Checklist (strike out non-applicable):

  • Changes documented in CHANGES.md
  • Related issue exists and is referred to in the PR description and CHANGES.md
  • Added docstrings and API docs for any new/modified user-facing classes and functions
  • Changes/features documented in docs/*
  • Unit-tests adapted/added for changes/features
  • Test coverage remains or increases (target 100%)

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@freds-dev freds-dev changed the title Implement restart job functionality and update related tests and API … Add server-side restart support for failed jobs Jul 31, 2026
@freds-dev freds-dev changed the title Add server-side restart support for failed jobs Add restart option for failed/dismissed jobs Aug 3, 2026
Comment thread tools/gen_server.py
"from abc import ABC, abstractmethod\n",
"\n",
f"from .models import {model_list}\n",
f"from gavicore.models import {model_list}\n",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Service was moved from gavicore/service.py to gavicore/service/service.py, but the generator still used the old relative import.

@freds-dev
freds-dev marked this pull request as ready for review August 3, 2026 09:34
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.

1 participant