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

Private DA metering #4101

Closed
Tracked by #5002
PhilWindle opened this issue Jan 17, 2024 · 0 comments
Closed
Tracked by #5002

Private DA metering #4101

PhilWindle opened this issue Jan 17, 2024 · 0 comments

Comments

@PhilWindle
Copy link
Collaborator

PhilWindle commented Jan 17, 2024 •
edited by just-mitch
Loading

Write e2e tests demonstrating:

  1. Fees pertaining to DA costs resulting from private execution being correctly calculated.
  2. Fees pertaining to DA costs being correctly transferred.
  3. Transactions being reverted either due to app logic reverts or DA limits exceeded and marked as failed in the rollup.
  4. The correct operation of fee preparation and distribution functions.

Since we do not have L2 gas metering yet, fees specified as part of the transaction will be applied solely to DA gas.
@PhilWindle PhilWindle added this to the Fees Phase 1 milestone Jan 17, 2024
@github-project-automation github-project-automation bot moved this to Todo in A3 Jan 17, 2024
@alexghr alexghr mentioned this issue Jan 30, 2024
Closed
@just-mitch just-mitch changed the title Complete e2e tests Complete e2e tests with DA metering Feb 27, 2024
@just-mitch just-mitch changed the title Complete e2e tests with DA metering DA metering Mar 6, 2024
This was referenced Mar 6, 2024
Closed
Closed
@just-mitch just-mitch modified the milestones: Fees Phase 1, ITN - Fees Mar 6, 2024
@just-mitch just-mitch changed the title DA metering Private DA metering Mar 19, 2024
@just-mitch just-mitch mentioned this issue Mar 28, 2024
Closed
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
A3
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add gas costs to TxEffect AztecProtocol/aztec-packages
2 participants
@PhilWindle @just-mitch