Skip to content
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

omni-node: add offchain worker #7479

Merged

Conversation

iulianbarbu
Copy link
Contributor

@iulianbarbu iulianbarbu commented Feb 5, 2025

Description

Copy pasted the parachain-template-node offchain worker setup to omni-node-lib for both aura and manual seal nodes.

Closes #7447

Integration

Enabled offchain workers for both polkadot-omni-node and polkadot-parachain nodes. This would allow executing offchain logic in the runtime and considering it on the node side.

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>
Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>
Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>
@iulianbarbu iulianbarbu added T0-node This PR/Issue is related to the topic “node”. T9-cumulus This PR/Issue is related to cumulus. labels Feb 5, 2025
@iulianbarbu iulianbarbu self-assigned this Feb 5, 2025
@iulianbarbu
Copy link
Contributor Author

/cmd prdoc --audience runtime_dev,node_dev --bump patch

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Command "" has started 🚀 See logs here

Copy link
Contributor

github-actions bot commented Feb 5, 2025

Command "" has finished ✅ See logs here

@iulianbarbu
Copy link
Contributor Author

/cmd prdoc --audience runtime_dev --bump patch

Copy link
Contributor

@serban300 serban300 left a comment

Choose a reason for hiding this comment

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

LGTM !

Signed-off-by: Iulian Barbu <iulian.barbu@parity.io>
Copy link
Contributor

@skunert skunert left a comment

Choose a reason for hiding this comment

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

Nice work!

@iulianbarbu iulianbarbu enabled auto-merge February 5, 2025 18:47
@iulianbarbu iulianbarbu added this pull request to the merge queue Feb 5, 2025
Merged via the queue into paritytech:master with commit 87f4f3f Feb 5, 2025
201 of 207 checks passed
@iulianbarbu iulianbarbu deleted the ib-add-offchain-worker-to-omni-node branch February 5, 2025 19:49
@iulianbarbu iulianbarbu added the A4-needs-backport Pull request must be backported to all maintained releases. label Feb 6, 2025
@paritytech-cmd-bot-polkadot-sdk

Created backport PR for stable2407:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-7479-to-stable2407
git worktree add --checkout .worktree/backport-7479-to-stable2407 backport-7479-to-stable2407
cd .worktree/backport-7479-to-stable2407
git reset --hard HEAD^
git cherry-pick -x 87f4f3f0df5fc0cc72f69e612909d4d213965820
git push --force-with-lease

@paritytech-cmd-bot-polkadot-sdk

Created backport PR for stable2409:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-7479-to-stable2409
git worktree add --checkout .worktree/backport-7479-to-stable2409 backport-7479-to-stable2409
cd .worktree/backport-7479-to-stable2409
git reset --hard HEAD^
git cherry-pick -x 87f4f3f0df5fc0cc72f69e612909d4d213965820
git push --force-with-lease

@paritytech-cmd-bot-polkadot-sdk

Created backport PR for stable2412:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-7479-to-stable2412
git worktree add --checkout .worktree/backport-7479-to-stable2412 backport-7479-to-stable2412
cd .worktree/backport-7479-to-stable2412
git reset --hard HEAD^
git cherry-pick -x 87f4f3f0df5fc0cc72f69e612909d4d213965820
git push --force-with-lease

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A4-needs-backport Pull request must be backported to all maintained releases. T0-node This PR/Issue is related to the topic “node”. T9-cumulus This PR/Issue is related to cumulus.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Polkadot Omni Node: Offchain Worker not running
3 participants