fix: bump to celestia-app v0.13.3 #2257

rootulp · 2023-05-25T19:21:00Z

Closes celestiaorg/celestia-app#1816 which was originally reported by @jcstein.

@renaynay is it possible to release v0.10.1 after this merges?

Testing

Before (with celestia-node v0.10.0)

$ curl -X GET http://0.0.0.0:26659/namespaced_data/e8e5f679bf7116cb/height/559108
"share sequence has 1 shares but needed 0 shares"%

After

$ curl -X GET http://0.0.0.0:26659/namespaced_data/e8e5f679bf7116cb/height/559108
{"data":["AEh1kC0LWsf6EWtnsXprMIEAAAAABCt42ozTazfbdwAAYFVEUSPL0Q1xW6Jtapu1KnEZZZW4RI2SENkRjGVVl2pZ3ZsuFTUaQUfHNBGXpNVOlbjFDkVd0mrJOenUpe5KeloLQYlsp2f/vf99gefd00bU2Pbh1u2UHCXmBRfh/AXXGYmDZfblBNTVrcCk+fBQJpKMPHCFO4hJwd1jSKkobUPkk6LrrFa1sNyC2KdfU9M7Vida1GR+H4+36v5PC+SuuV0zV98MOuuDexFGGw0UKP3wRhcK4x65qnO2DNLBNBtIo8qWcrc7fIOPI+7IOg3HsXmTHqMtzXxRlk+kF2MlUwtMQ0GatgXlZ46OCZ6e0jWkE7kgu524ULFzQ8EiivPFmq87hWCaLaS5aUtlznB71EXO57dci615Lm0HyJONyefjeFNfEzQmosG0I5AWc3khxjoSPdij4YnHY7l0hNIkzy/r9TFXV+ZQMfXZOpiGgTRStgbtECKZ5DFR8tPBAEsDtvwv+6b+CBf5Yv9mWGqEHEyzgzTdnlKEIntmt1794plkL+YIv4jxlfssTCuoijwzwMzZAdO+gDRkh9dK28uGu0jzTOVhfe9LIVrnLNif0KUbkWqfzvSJ7H1ce/jYqDTxLvmBPK3IQ1JVkVdOH9mHpIaIc9KxdntCpw+aPaTZOKqsMU4B9RW3h6j0LsFLyUlNu39W9k+dfRy9w2fZUsC045D2xu3H9+x1Vb2l2KndJTF4OVT0m+9AnTC0Nbx1ebpbLw1Mc4A0vFdylops7FB4uSxwY8HhmwIlxXbYVJdaXkrK0jObDwLTHCFNsoxyLMC2Gwj4jI99HFm655e9ubbj07m1t1aZPNJ8NZiGgzSF1eyhPXT+20bLlDkVbDHnXWU/VhTSA+vBwSt7+XHGYJozpKEorRkUo/AbFzfxMLix3L93aXfT9Jc1/pnhj57vf/odD0xzhTSMsQvN5cox4e7wVAf5zzT42yxdQjwvU3/s7xMZzWOf6mhyE8LqOSjTodpXTnN9sSbuuUEKJKzAbljwbYlnwOnVk+wPmhukmTzPmf3Dz8as4QJ61a0jtiE6cfLEQ5GnMjjOTz5ZXHwUTDsFabnBKEypUrBy/1Is3eYqRyHbmXW29AqoeNd88M6cujEJTPOENEJM7cyI9y6uhTFws8UL1WtOjBWOPn4iWCIgOTTM4h6Ydvr/We3DvF/N/cPvjZtU07KTMsxWH9yPRzPXy4oeyREjBjQwDQ9p1ZxKaVSTpUgk1R9cq8fsybZSeYZYT358VaEKm8ZzBNO8Ia2nK47STPft2xbPKBbhqvmoBJxTUvJGAgld+pkeTGwIpvlCmojMCZnoPswyLpXMPWwValWdC3rzlGDUhOhApyZ8+XsUmEaEtOk6q/c895prbA8li9SZP3WKrTH7ymMgMmJB8oNyTo8Kpp3p/jcAAP//G9WQYQE=","","AEfeb1e6eRNt3KaCZD0QZQAAAAAAAtZ42trhx/DDe8Hr2Z1HlninxP079eFExckvVjeZfBfJm5j9zJdRWTm9dO3yJhW9BcfW836ROTUv+IXvshpbTav5v2uKX729VbPj0qGNBsc+LKhvScl10zsAMW+m/GZXJ+812yrffRAKFS9mKm7cOqGySX9Cj5jMVf71r/Y1qegvWPI15nyfW0LltT/dZ9u0mqutb3SeaJn4u8PwOqfoj4g9r0DmGUDNM9z/d3aPyUN3/sl+lW2dmQecf04+5nyyY4/3n9zpyjdUpxNrnhHUvLq6d9cctpzdpyVZe/rf8sknauJZWgtzlibFLD9YkdDtsJ9Y80yg5m04bHPt0oa7ewpE4yoCOswrfGefe3d7d61/juKW5QuuazEQa54Z1Lze+ypML/hfnsjXudGnEKLxfWUX38yThXm3FvIGr9H6YbaRWPMsoOb97zWQ//Dv6f24+x/ZIo7ft/fUbHvZKhMVobN0cXnkqy+LiTXPCmqeBXenbfjjIy9WP9N4LeUycemtJ1PVgl8YePLP4uudnqrcSqx5NlDzxL5b3rF+PUXZ/Z5RT/LXoKsdx+e9/ep062WQdcfzr6krnJtUDBb8OvWkYJnMrlC1K0e1vqxUW5CyKDh+1mrVrjqpR1PubeFPB5lnBzVvkunhXWFzXlz623JlUuCaPcxv9r67JntObNbTH1uiCyx95hJrngPUvLZGvjv8xcUl8+oap2aZtqrd/d7Bmpe/ePdEE4168c3/PxJrnhPUvMg2Savj+n4vvF50Vu5/8szyl+IHy5C67Jjj7g2LilrrNYk1zwVq3pvPf9+5fXo8U/7NHGWHuSviDeoeqsxd++/66fXZ71Qa5l0i1jw3qHnJb9dmW1ndzgt0Fd+UmXmYo2vahmXLv25nybY3OGsx/2EvseZ5QM1ju/6Py8T3Pl9Q1BPNvfYVE+W2F1rOnPhQYVrjwuk+bDzLiDXP6wAgAAD//zfPbngB"],"height":559108}%

codecov-commenter · 2023-05-25T19:32:06Z

Codecov Report

Merging #2257 (26c01b4) into main (cd8d0b9) will decrease coverage by 5.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2257      +/-   ##
==========================================
- Coverage   55.87%   50.49%   -5.38%     
==========================================
  Files         216      150      -66     
  Lines       14218     9305    -4913     
==========================================
- Hits         7944     4699    -3245     
+ Misses       5484     4209    -1275     
+ Partials      790      397     -393

see 152 files with indirect coverage changes

celestiaorg/celestia-node#2257

* change RPC to P-OPS * deps: change to test version of node celestiaorg/celestia-node#2257 * fix: version for image * use sha * change path...

@Ferret-san

batcher: fix duplicate cli flag name docker: use latest devnet image bump local celestia devnet to 0.9.1 docker: wait for da to boot op-ndoe: decodeeETHData - use uint32 for height proposer: fix da rpc, estimate gas op-node: fix failed to parse frame error proposer: fix revert proposer: restore gas limit; add explanation proposer: nit add celestia logo, update readme Update README.md style size plus not x docs: edit intro Update README.md Update README.md nit docs: add section on deposit bridge add workaround from @Ferret-san Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> close code block Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> fix: typo Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> Update README.md docs: add bridging section from guidance from @Ferret-san Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> deps: bump local-devnet to 0.9.2 docker: fix da container name op-node: gracefully retry da connection err test: update e2e tests for celestia da docs: update readme deps: update local-celestia-devnet to v0.9.5 Add DALC support (#47) * deps: add vars to devnet-up.sh script * add new docker-compose and script * fix: add testnet-up to makefile * add testnet-clean and testnet-down make cmds * match other instances of localhost * fix: paths fix: paths in Makefile * remove unnecessary vars * docs: update link in readme * fix: use light node image * add celestia light node image * fix: start DA as dependency to batcher * docs: copy edits * docker: testnet - mount celestia light volume * celestia: remove hardcoded key; add README * Update README.md * Update op-celestia/blockspacerace-data/README.md * edit readme * docker: celestia startup - fix permission denied * revert changes from dde9c93 --------- Co-authored-by: Javed Khan <tuxcanfly@gmail.com> fix: update path to node store (#52) devnet: increase block times (#50) * devnet: increase block times * proposer: bump network timeout * proposer timeout: 5 -> 5s * proposer: tweak timeout to 180s * fix: ymlup --------- Co-authored-by: joshcs.eth <46639943+jcstein@users.noreply.github.com> deps: bump celestia-node & local-celestia-devnet image version to v0.10.0 (#56) * deps: bump celestia-node v0.9.5 to v0.10.0 * change RPC * swap RPC * deps: bump v0.9.5 to v0.10.0 local-celestia-devnet op-node: log namespaced data details (#94) * op-node: log namespaced data details * op-node: log hex Dependabot and Fork Sync (#93) * .github: add sync fork workflow and update dependabot to only check .github and go.mod * .github: update dependabot.yml fix: resolve `max_subscriptions` and` share sequence` errors (#120) * change RPC to P-OPS * deps: change to test version of node celestiaorg/celestia-node#2257 * fix: version for image * use sha * change path... docs: add wiki to README proposer: allow non finalized true -> false (#133) deps: update to node RC for Arabica (#132) * deps: update to node RC for Arabica * deps: update for arabica * add changes from #133 to test * storage in arabica-6 not arabica-8 * deps: update RPC * deps: add host.docker.internal for testing local light node (not docker) * Update docker-compose-devnet.yml * Revert "deps: add host.docker.internal for testing local light node (not docker)" This reverts commit 09490fd. * Revert "Update docker-compose-devnet.yml" This reverts commit cccaf66. * increase namespace ID to 28 bytes * update RPC and to arabica-8 for node store * deps: bump node to rc2 * Update docker-compose-testnet.yml * deps: bump go-cnc to 0.4.1, fix breaking changes * Update docker-compose-devnet.yml * bump gas fee * revert 71a3d18 this is so that #133 can be merged before this PR --------- Co-authored-by: Javed Khan <tuxcanfly@gmail.com> deps: bump celestia-node to v0.11.0-rc2 (#137) * deps: bump celestia-node to v0.11.0-rc2 * docs: match celestia-node#2277 deps: remove trailing slash (#139) op-node: continue on retrieval error (#142) op-node: calldata source - reset pipeline on err (#144) op-node: da-rpc, namespace-id - use environment flags (#158) * op-node: use global string for da flags * op-node: use flag name * testnet: better healthcheck * testnet: restore rpc flags testnet: update node rc, flags for arabica-9 (#157) * testnet: update node rc, flags for arabica-9 * go mod: bump go-cnc to 0.4.2-pre * testnet: bump celestia-node to v0.11.0-rc6 * txmgr: submit pfb - handle unexpected response code; bump fee * nit: typo * go mod: bump go-cnc to v0.4.2 * txmgr: submit pfb - handle unexpected response - check tx hash * txmgr: bump gas limit * txmgr: add additional height sanity check devnet: bump local-celestia-devnet to v0.11.0-rc6 (#168) * devnet: fix op-batcher depends_on * devnet: bump local-celestia-devnet to v0.11.0-rc6 * devnet: da - bump start_period and interval * testnet: use new namespace id * fix: celestia-node version and remove deprecated flags --------- Co-authored-by: joshcs.eth ᵍᵐ <mail@jcstein.dev> deps: local-celestia-devnet -> v0.11.0-rc8 (#175) da: use celestia-openrpc (#187) * op-node: use openrpc to get blobs * op-batcher: use openrpc to submit blobs * op-celestia-: use openrpc to fetch blobs * da: update rpc port * docker: update rpc port * readme: devnet - rm unused port * docker: devnet - restore platform docker: testnet add debug log flag (#189) da: celestia-openrpc - use blob commitment (#196) * op-batcher: use openrpc to submit blobs * op-celestia-: use openrpc to fetch blobs * da: update rpc port * docker: update rpc port * readme: devnet - rm unused port * docker: devnet - restore platform * da: use blob commitment instead of tx index * celestia: use binary interfaces * go mod: tidy * deps: bump celestia-openrpc@v0.1.3-pre * pkg celestia: test - add invalid case * deps: restore celestia-openrpc@v0.1.2 da: use celestia da; update config da: go mod tidy da: update devnet docker-compose.yml da: fix devnet script; op-stack-go docker; op-celestia .github: update base repo owner (#223) * .github: update base repo owner * add use of PAT for sync action --------- Co-authored-by: Matthew Sevey <mjsevey@gmail.com> .github: fix base branch (#226) op-server: bump celestia-openrpc to v0.3.0 go mod tidy chore: update devnet version to v0.12.1 (#245) * chore: update devnet version to v0.12.1 * docs: update readme to v0.12.1 devnet * chore: update celestia description * chore: update celestia-node to v0.12.0 da: move batcher changes from op-service to batcher/driver major rewrite and tech debt cleanup * batcher changes moved from op-service to batcher driver * da flags consolidated into da.rpc, da.namespace-id, da.auth-token * batcher / proposer da flags removed * batcher fetches da config from op-node / same as rollup config da: celestia-openrpc -> celestia-da da: log blob submit and request da: log prefix da: reuse rollup config for da rpc url da: use env var for da rpc url da: update local celestia devnet image da: reuse da client; bump local celestia devnet version da: move da client to new file; fix devnet version da: rpc - default to localhost da: bump local-celestia-devnet:v0.12.2 da: update README da: rm op-celestia da: calldata version prefix, eth fallback

@Ferret-san

batcher: fix duplicate cli flag name docker: use latest devnet image bump local celestia devnet to 0.9.1 docker: wait for da to boot op-ndoe: decodeeETHData - use uint32 for height proposer: fix da rpc, estimate gas op-node: fix failed to parse frame error proposer: fix revert proposer: restore gas limit; add explanation proposer: nit add celestia logo, update readme Update README.md style size plus not x docs: edit intro Update README.md Update README.md nit docs: add section on deposit bridge add workaround from @Ferret-san Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> close code block Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> fix: typo Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> Update README.md docs: add bridging section from guidance from @Ferret-san Co-Authored-By: Diego <31937514+Ferret-san@users.noreply.github.com> deps: bump local-devnet to 0.9.2 docker: fix da container name op-node: gracefully retry da connection err test: update e2e tests for celestia da docs: update readme deps: update local-celestia-devnet to v0.9.5 Add DALC support (#47) * deps: add vars to devnet-up.sh script * add new docker-compose and script * fix: add testnet-up to makefile * add testnet-clean and testnet-down make cmds * match other instances of localhost * fix: paths fix: paths in Makefile * remove unnecessary vars * docs: update link in readme * fix: use light node image * add celestia light node image * fix: start DA as dependency to batcher * docs: copy edits * docker: testnet - mount celestia light volume * celestia: remove hardcoded key; add README * Update README.md * Update op-celestia/blockspacerace-data/README.md * edit readme * docker: celestia startup - fix permission denied * revert changes from dde9c93 --------- Co-authored-by: Javed Khan <tuxcanfly@gmail.com> fix: update path to node store (#52) devnet: increase block times (#50) * devnet: increase block times * proposer: bump network timeout * proposer timeout: 5 -> 5s * proposer: tweak timeout to 180s * fix: ymlup --------- Co-authored-by: joshcs.eth <46639943+jcstein@users.noreply.github.com> deps: bump celestia-node & local-celestia-devnet image version to v0.10.0 (#56) * deps: bump celestia-node v0.9.5 to v0.10.0 * change RPC * swap RPC * deps: bump v0.9.5 to v0.10.0 local-celestia-devnet op-node: log namespaced data details (#94) * op-node: log namespaced data details * op-node: log hex Dependabot and Fork Sync (#93) * .github: add sync fork workflow and update dependabot to only check .github and go.mod * .github: update dependabot.yml fix: resolve `max_subscriptions` and` share sequence` errors (#120) * change RPC to P-OPS * deps: change to test version of node celestiaorg/celestia-node#2257 * fix: version for image * use sha * change path... docs: add wiki to README proposer: allow non finalized true -> false (#133) deps: update to node RC for Arabica (#132) * deps: update to node RC for Arabica * deps: update for arabica * add changes from #133 to test * storage in arabica-6 not arabica-8 * deps: update RPC * deps: add host.docker.internal for testing local light node (not docker) * Update docker-compose-devnet.yml * Revert "deps: add host.docker.internal for testing local light node (not docker)" This reverts commit 09490fd. * Revert "Update docker-compose-devnet.yml" This reverts commit cccaf66. * increase namespace ID to 28 bytes * update RPC and to arabica-8 for node store * deps: bump node to rc2 * Update docker-compose-testnet.yml * deps: bump go-cnc to 0.4.1, fix breaking changes * Update docker-compose-devnet.yml * bump gas fee * revert 71a3d18 this is so that #133 can be merged before this PR --------- Co-authored-by: Javed Khan <tuxcanfly@gmail.com> deps: bump celestia-node to v0.11.0-rc2 (#137) * deps: bump celestia-node to v0.11.0-rc2 * docs: match celestia-node#2277 deps: remove trailing slash (#139) op-node: continue on retrieval error (#142) op-node: calldata source - reset pipeline on err (#144) op-node: da-rpc, namespace-id - use environment flags (#158) * op-node: use global string for da flags * op-node: use flag name * testnet: better healthcheck * testnet: restore rpc flags testnet: update node rc, flags for arabica-9 (#157) * testnet: update node rc, flags for arabica-9 * go mod: bump go-cnc to 0.4.2-pre * testnet: bump celestia-node to v0.11.0-rc6 * txmgr: submit pfb - handle unexpected response code; bump fee * nit: typo * go mod: bump go-cnc to v0.4.2 * txmgr: submit pfb - handle unexpected response - check tx hash * txmgr: bump gas limit * txmgr: add additional height sanity check devnet: bump local-celestia-devnet to v0.11.0-rc6 (#168) * devnet: fix op-batcher depends_on * devnet: bump local-celestia-devnet to v0.11.0-rc6 * devnet: da - bump start_period and interval * testnet: use new namespace id * fix: celestia-node version and remove deprecated flags --------- Co-authored-by: joshcs.eth ᵍᵐ <mail@jcstein.dev> deps: local-celestia-devnet -> v0.11.0-rc8 (#175) da: use celestia-openrpc (#187) * op-node: use openrpc to get blobs * op-batcher: use openrpc to submit blobs * op-celestia-: use openrpc to fetch blobs * da: update rpc port * docker: update rpc port * readme: devnet - rm unused port * docker: devnet - restore platform docker: testnet add debug log flag (#189) da: celestia-openrpc - use blob commitment (#196) * op-batcher: use openrpc to submit blobs * op-celestia-: use openrpc to fetch blobs * da: update rpc port * docker: update rpc port * readme: devnet - rm unused port * docker: devnet - restore platform * da: use blob commitment instead of tx index * celestia: use binary interfaces * go mod: tidy * deps: bump celestia-openrpc@v0.1.3-pre * pkg celestia: test - add invalid case * deps: restore celestia-openrpc@v0.1.2 da: use celestia da; update config da: go mod tidy da: update devnet docker-compose.yml da: fix devnet script; op-stack-go docker; op-celestia .github: update base repo owner (#223) * .github: update base repo owner * add use of PAT for sync action --------- Co-authored-by: Matthew Sevey <mjsevey@gmail.com> .github: fix base branch (#226) op-server: bump celestia-openrpc to v0.3.0 go mod tidy chore: update devnet version to v0.12.1 (#245) * chore: update devnet version to v0.12.1 * docs: update readme to v0.12.1 devnet * chore: update celestia description * chore: update celestia-node to v0.12.0 da: move batcher changes from op-service to batcher/driver major rewrite and tech debt cleanup * batcher changes moved from op-service to batcher driver * da flags consolidated into da.rpc, da.namespace-id, da.auth-token * batcher / proposer da flags removed * batcher fetches da config from op-node / same as rollup config da: celestia-openrpc -> celestia-da da: log blob submit and request da: log prefix da: reuse rollup config for da rpc url da: use env var for da rpc url da: update local celestia devnet image da: reuse da client; bump local celestia devnet version da: move da client to new file; fix devnet version da: rpc - default to localhost da: bump local-celestia-devnet:v0.12.2 da: update README da: rm op-celestia da: calldata version prefix, eth fallback

* change RPC to P-OPS * deps: change to test version of node celestiaorg/celestia-node#2257 * fix: version for image * use sha * change path...

fix: bump to celestia-app v0.13.3

26c01b4

rootulp self-assigned this May 25, 2023

github-actions bot added the external Issues created by non node team members label May 25, 2023

rootulp added kind:fix Attached to bug-fixing PRs area:core_and_app Relationship with Core node and Celestia-App labels May 25, 2023

fix: lint

119e2f5

rootulp marked this pull request as ready for review May 25, 2023 19:58

rootulp requested review from renaynay, Wondertan, vgonkivs, distractedm1nd and walldiss as code owners May 25, 2023 19:58

rootulp mentioned this pull request May 25, 2023

share sequence has 1 shares but needed 0 shares celestiaorg/celestia-app#1816

Closed

jcstein added a commit to celestiaorg/optimism that referenced this pull request May 25, 2023

deps: change to test version of node

4d767c7

celestiaorg/celestia-node#2257

Wondertan approved these changes May 26, 2023

View reviewed changes

renaynay approved these changes May 26, 2023

View reviewed changes

Wondertan enabled auto-merge (squash) May 26, 2023 14:30

Merge branch 'main' into rp/share-sequence-fix

dc8f0a5

Wondertan merged commit e5efcb0 into celestiaorg:main May 26, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: bump to celestia-app v0.13.3 #2257

fix: bump to celestia-app v0.13.3 #2257

rootulp commented May 25, 2023 •

edited

Loading

codecov-commenter commented May 25, 2023

fix: bump to celestia-app v0.13.3 #2257

fix: bump to celestia-app v0.13.3 #2257

Conversation

rootulp commented May 25, 2023 • edited Loading

Testing

codecov-commenter commented May 25, 2023

Codecov Report

rootulp commented May 25, 2023 •

edited

Loading