Skip to content

Automations: made scheduler requests idempotent#28978

Draft
EvanHahn wants to merge 1 commit into
mainfrom
automations-scheduler-idempotency-key
Draft

Automations: made scheduler requests idempotent#28978
EvanHahn wants to merge 1 commit into
mainfrom
automations-scheduler-idempotency-key

Conversation

@EvanHahn

@EvanHahn EvanHahn commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

closes NY-1396
ref #29490

More accurately, this sends the new idempotencyKey option to the scheduler.

This will help reduce unnecessary pressure on the scheduler.

@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 887a281a-8f51-430e-9298-04a70f55f9d4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch automations-scheduler-idempotency-key

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@nx-cloud

nx-cloud Bot commented Jun 29, 2026

Copy link
Copy Markdown

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci


View your CI Pipeline Execution ↗ for commit 9c3e560

Command Status Duration Result
nx run ghost:test:ci:integration ✅ Succeeded 3m 3s View ↗
nx run @tryghost/admin:build ✅ Succeeded 7s View ↗
nx run ghost-monorepo:lint:boundaries ✅ Succeeded 14s View ↗
nx run-many -t lint -p ghost,@tryghost/adapter-... ✅ Succeeded 16s View ↗
nx run ghost:test:legacy ✅ Succeeded 3m 8s View ↗
nx run ghost:test:integration ✅ Succeeded 3m 11s View ↗
nx run ghost:test:e2e ✅ Succeeded 2m 39s View ↗
nx run-many --target=build --projects=tag:publi... ✅ Succeeded 1s View ↗
nx run-many -t test:unit -p ghost,@tryghost/ada... ✅ Succeeded 29s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-07-21 15:19:47 UTC

@EvanHahn
EvanHahn force-pushed the automations-scheduler-idempotency-key branch from aea7218 to 9c3e560 Compare July 21, 2026 15:01
closes https://linear.app/ghost/issue/NY-1396

More accurately, this sends the new `idempotencyKey` option to the
scheduler.

This will help reduce unnecessary pressure on the scheduler.
@EvanHahn
EvanHahn force-pushed the automations-scheduler-idempotency-key branch from 9c3e560 to c89dd75 Compare July 21, 2026 15:04
@codecov

codecov Bot commented Jul 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.55%. Comparing base (9534ef0) to head (c89dd75).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #28978   +/-   ##
=======================================
  Coverage   74.55%   74.55%           
=======================================
  Files        1604     1605    +1     
  Lines      140913   140918    +5     
  Branches    17192    17200    +8     
=======================================
+ Hits       105052   105064   +12     
+ Misses      34828    34821    -7     
  Partials     1033     1033           
Flag Coverage Δ
e2e-tests 76.63% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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