Skip to content

fix(backend/scheduler): Unbreak Scheduler.get_execution_schedules #9919

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

Pwuts
Copy link
Member

@Pwuts Pwuts commented May 7, 2025

Changes 🏗️

  • In get_graph_execution_schedules, skip jobs when their kwargs can't be parsed as GraphExecutionJobArgs
  • Rename methods of Scheduler to clarify their scope (scheduled graph executions)

Checklist 📋

For code changes:

  • I have clearly listed my changes in the PR description
  • I have made a test plan
  • I have tested my changes according to the test plan:
    • Go to /library/agents/[id] (which calls GET /api/schedules)
      • -> GET /api/schedules request returns HTTP 200

@Pwuts Pwuts requested a review from a team as a code owner May 7, 2025 15:55
@Pwuts Pwuts requested review from Swiftyos and aarushik93 and removed request for a team May 7, 2025 15:55
@github-project-automation github-project-automation bot moved this to 🆕 Needs initial review in AutoGPT development kanban May 7, 2025
Copy link

netlify bot commented May 7, 2025

Deploy Preview for auto-gpt-docs-dev canceled.

Name Link
🔨 Latest commit 245d8f7
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/681b8379fc092500087afcad

@github-actions github-actions bot added platform/backend AutoGPT Platform - Back end size/m labels May 7, 2025
@Pwuts Pwuts changed the title fix(backend/scheduler): Exclude non-graph execution jobs from get_graph_execution_schedules fix(backend/scheduler): Unbreak Scheduler.get_execution_schedules May 7, 2025
Copy link

netlify bot commented May 7, 2025

Deploy Preview for auto-gpt-docs canceled.

Name Link
🔨 Latest commit 245d8f7
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/681b837917a13700088461c0

@Pwuts Pwuts requested review from ntindle and majdyz and removed request for Swiftyos May 7, 2025 15:56
Copy link

deepsource-io bot commented May 7, 2025

Here's the code health analysis summary for commits ac8ef9b..245d8f7. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Python LogoPython✅ Success
❗ 13 occurences introduced
🎯 14 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@github-project-automation github-project-automation bot moved this from 🆕 Needs initial review to 👍🏼 Mergeable in AutoGPT development kanban May 7, 2025
@Pwuts Pwuts enabled auto-merge May 7, 2025 16:02
@Pwuts Pwuts added this pull request to the merge queue May 7, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 7, 2025
@Pwuts Pwuts added this pull request to the merge queue May 8, 2025
Merged via the queue into dev with commit 433b76b May 8, 2025
23 checks passed
@Pwuts Pwuts deleted the pwuts/open-2486-fix-get-apischedules-error-on-report_late_executions-job branch May 8, 2025 11:51
@github-project-automation github-project-automation bot moved this from 👍🏼 Mergeable to ✅ Done in AutoGPT development kanban May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform/backend AutoGPT Platform - Back end size/m
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants