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.

Sign up for GitHub

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

Jump to bottom

feat: support increase l2 tx max cycles #852

Merged
merged 3 commits into from
Nov 12, 2022
Merged

feat: support increase l2 tx max cycles #852

merged 3 commits into from
Nov 12, 2022

Conversation

jjyr
Copy link
Collaborator

@jjyr jjyr commented Nov 11, 2022 •
edited
Loading

  1. add fork config
  2. fetch consensus parameters from fork config

Configuration changes:

  1. move backend_switches to fork.backend_forks
  2. move debug_backend_switches to debug_backend_forks

@jjyr jjyr force-pushed the enable-l2-tx-cycles-limit-fork branch from 3de51aa to 171c2be Compare November 11, 2022 07:59
@gw-bot
Copy link

gw-bot bot commented Nov 11, 2022 •
edited
Loading

Running integration test

Workflow Run Id: 3443431848

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@godwokenrises godwokenrises deleted a comment from gw-bot bot Nov 11, 2022
@Flouse Flouse added the P-High label Nov 11, 2022
@gw-bot
Copy link

gw-bot bot commented Nov 11, 2022 •
edited
Loading

Running integration test

Workflow Run Id: 3443607280

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

keroro520
keroro520 previously approved these changes Nov 11, 2022
View reviewed changes
Flouse added a commit to godwokenrises/godwoken-docker-prebuilds that referenced this pull request Nov 11, 2022
@Flouse
Bump GODWOKEN_REF to refs/pull/852/head
c4ae3e8 
# godwokenrises/godwoken#852
@jjyr jjyr requested review from blckngm and keroro520 November 11, 2022 16:01
@gw-bot
Copy link

gw-bot bot commented Nov 11, 2022 •
edited
Loading

Running integration test

Workflow Run Id: 3446341314

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

Flouse
Flouse reviewed Nov 12, 2022
View reviewed changes
Copy link
Collaborator

@Flouse Flouse left a comment •
edited
Loading

Choose a reason for hiding this comment

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

/itest

@jjyr jjyr merged commit 3f7f2dc into godwokenrises:develop Nov 12, 2022
@jjyr jjyr deleted the enable-l2-tx-cycles-limit-fork branch November 12, 2022 16:32
This was referenced Nov 13, 2022
Merged
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blckngm blckngm blckngm approved these changes

@Flouse Flouse Flouse approved these changes

@zeroqn zeroqn Awaiting requested review from zeroqn

@keroro520 keroro520 Awaiting requested review from keroro520

Assignees

@jjyr jjyr

Labels
P-High
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jjyr @Flouse @keroro520 @blckngm