Skip to content

L2 public testnet deployment and related fixes #751

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

Merged
merged 43 commits into from
Nov 23, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
5e36583
fix: use path.join to manage path concat
abarmat Sep 21, 2022
0ca43ff
feat: add publish and mint to the CLI
abarmat Sep 21, 2022
45ce81e
chore: add reports for recent audits (#710)
pcarranzav Sep 28, 2022
b6219d3
chore: run GitHub actions when it makes sense (#707)
tmigone Sep 29, 2022
04cfd3a
chore: provide sane defaults for graph config (#721)
tmigone Oct 3, 2022
fdc0bae
chore: remove rinkeby and kovan references (#729)
tmigone Oct 4, 2022
f1c5fe7
feat: add optional expression logging to GRE (#728)
tmigone Oct 4, 2022
0b7055a
test: print gas report, and ensure evm is killed
pcarranzav Oct 13, 2022
3f16e5a
Merge pull request #732 from graphprotocol/pcv/test-gas-cat-report
pcarranzav Oct 14, 2022
7e5ac05
feat(gre): add hardhat-secure-accounts to GRE (#696)
tmigone Oct 28, 2022
598ecc3
chore: update arbitrum sdk to 3.0-rc1
tmigone Sep 26, 2022
a7ba535
fix(task): accept ownership only if pending owner matches
tmigone Sep 26, 2022
0aed5eb
feat(task): split unpause task into protocol and bridge tasks
tmigone Sep 26, 2022
a3fa719
feat(e2e): run L1 and L2 together in e2e tests
tmigone Sep 26, 2022
ad11bdf
feat(e2e): fetch deployment data from testnodes
tmigone Sep 27, 2022
e9cab36
chore: update README.md
tmigone Sep 27, 2022
7eb7ba2
fix(e2e): allow skipping bridge tests
tmigone Sep 27, 2022
1436a0c
fix(e2e): update github action to multi layer e2e tests
tmigone Sep 27, 2022
d9977b5
fix(e2e): bump gas limit for l2 scenario txs
tmigone Sep 28, 2022
6a9a999
chore(e2e): run nitro addressbook setup only on localnitro networks
tmigone Sep 28, 2022
9729f43
fix: allow e2e to run only on L2
tmigone Oct 3, 2022
85c2964
fix(e2e): use fixed gasLimit for CI send to L2
tmigone Oct 3, 2022
ee8c7f3
fix(e2e): use fixed gasLimit for redeem in CI
tmigone Oct 6, 2022
967e085
e2e: add bridge e2e tests (#720)
tmigone Oct 31, 2022
ae1d70b
fix: gateways now unpaused on e2e
tmigone Nov 2, 2022
60afb9e
Merge branch 'dev' into l2-public-testnet
abarmat Nov 2, 2022
b1be78b
chore: fix automerge wrong import
tmigone Oct 31, 2022
ccaf653
fix: remove unused task folder
abarmat Nov 2, 2022
dff1845
fix: wrong imports and params
abarmat Nov 2, 2022
5f1a3c4
fix: additional post merge fixes
tmigone Nov 3, 2022
32fc123
fix: update nitro task for nitro v2.0.8
tmigone Nov 3, 2022
5daa3c1
chore: add deployed addresses for L2 public testnet
abarmat Nov 4, 2022
fb40819
fix: minor tweaks to e2e
tmigone Nov 7, 2022
3605081
v1.17.0
abarmat Nov 7, 2022
1dd6f06
chore: bump npm package version to v2.0.0-testnet
tmigone Nov 7, 2022
93f6939
Merge branch 'pcv/l2-bridge' into l2-public-testnet
pcarranzav Nov 23, 2022
dcc2cec
fix(e2e): no need to explicitly disable secure accounts
pcarranzav Nov 23, 2022
f45173c
Merge remote-tracking branch 'origin/dev' into l2-public-testnet
pcarranzav Nov 23, 2022
12e54e5
chore: bump @arbitrum/sdk to 3.0.0
pcarranzav Nov 23, 2022
43e719b
chore: bump package version to 2.0.0
pcarranzav Nov 23, 2022
4e2872b
fix: set flat bonding curve in all L2 configs
pcarranzav Nov 23, 2022
51c0441
test(e2e): reenable finalizeInboundTransfer revert check
pcarranzav Nov 23, 2022
aa85104
fix: use correct governance addresses on Arbitrum One
pcarranzav Nov 23, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: wrong imports and params
  • Loading branch information
abarmat committed Nov 2, 2022
commit dff1845d7016e0ed1d2f10d294b47bd51e66f6f9
5 changes: 3 additions & 2 deletions cli/commands/contracts/disputeManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,9 @@ import { constants, utils, Wallet } from 'ethers'
import { createAttestation, Attestation, Receipt } from '@graphprotocol/common-ts'

import { logger } from '../../logging'
import { sendTransaction, getChainID, getProvider, toGRT, randomHexBytes } from '../../network'
import { sendTransaction, getProvider, toGRT, randomHexBytes } from '../../network'
import { loadEnv, CLIArgs, CLIEnvironment } from '../../env'
import { getChainID } from '../../../test/lib/testHelpers'

const { HashZero } = constants
const { defaultAbiCoder: abi, arrayify, concat, hexlify } = utils
Expand All @@ -18,7 +19,7 @@ interface ChannelKey {
async function buildAttestation(receipt: Receipt, signer: string, disputeManagerAddress: string) {
const attestation = await createAttestation(
signer,
getChainID(),
await getChainID(),
disputeManagerAddress,
receipt,
'0',
Expand Down
2 changes: 1 addition & 1 deletion scripts/e2e
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ function configure_bridge () {

# Copy required arbitrum contract addresses to the local arbitrum address book
if [[ "$L1_NETWORK" == *"localnitro"* ]]; then
npx hardhat nitro:address-book-setup --deployment-file "$ARBITRUM_DEPLOYMENT_FILE" --arbitrum-address-book "$ARBITRUM_ADDRESS_BOOK" --disable-secure-accounts
npx hardhat nitro:address-book-setup --deployment-file "$ARBITRUM_DEPLOYMENT_FILE" --arbitrum-address-book "$ARBITRUM_ADDRESS_BOOK"
fi

# Configure the bridge
Expand Down
1 change: 1 addition & 0 deletions tasks/deployment/nitro.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ export const TASK_NITRO_SETUP_ADDRESS_BOOK = 'nitro:address-book-setup'
export const TASK_NITRO_FETCH_DEPLOYMENT_FILE = 'nitro:fetch-deployment-file'

task(TASK_NITRO_FUND_ACCOUNTS, 'Funds protocol accounts on Arbitrum Nitro testnodes')
.addFlag('disableSecureAccounts', 'Disable secure accounts on GRE')
.addOptionalParam('graphConfig', cliOpts.graphConfig.description)
.addOptionalParam('privateKey', 'The private key for Arbitrum testnode genesis account')
.addOptionalParam('amount', 'The amount to fund each account with')
Expand Down