Uniswap / routing-api

GNU General Public License v3.0
150 stars 185 forks source link

feat: migrate 1% of base traffic to new tenderly sim endpoint #910

Closed jsy1218 closed 1 day ago

jsy1218 commented 1 day ago

we see higher timeout to old tenderly sim endpoint on base: Screenshot 2024-11-08 at 10.18.49 AM.png

Screenshot 2024-11-08 at 10.19.28 AM.png

we migrate 1% of the traffic to new tenderly sim endpoint, to mitigate the timeout.

I tested locally with base traffic 100%. I can see the tenderly sim success:

Screenshot 2024-11-08 at 10.22.52 AM.png

Also direct hit against the new tenderly endpoint to make sure it returns gas used:

Get Warp

Copy link

Copy block
curl https://base.gateway.tenderly.co/{API_KEY} \
    -X POST \
    -H "Content-Type: application/json" \
    -d '{
   "id":1,
   "jsonrpc":"2.0",
   "method":"tenderly_estimateGasBundle",
   "params":[
      [
         {
            "from":"0x4bB6B2EFE7036020Ba6F02a05602546C9F25bF28",
            "to":"0xd9aAEc86B65D86f6A7B5B1b0c42FFA531710b6CA",
            "data":"0x095ea7b3000000000000000000000000000000000022d473030f116ddee9f6b43ac78ba3ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff"
         },
         {
            "from":"0x4bB6B2EFE7036020Ba6F02a05602546C9F25bF28",
            "to":"0x000000000022D473030F116dDEE9F6B43aC78BA3",
            "data":"0x87517c45000000000000000000000000d9aaec86b65d86f6a7b5b1b0c42ffa531710b6ca0000000000000000000000003fc91a3afd70395cd496c647d5a6cc9d4b2b7fad000000000000000000000000ffffffffffffffffffffffffffffffffffffffff0000000000000000000000000000000000000000000000000000000067c6ea8d"
         },
         {
            "from":"0x4bB6B2EFE7036020Ba6F02a05602546C9F25bF28",
            "to":"0x3fC91A3afd70395Cd496C647d5a6CC9D4B2b7FAD",
            "data":"0x24856bc300000000000000000000000000000000000000000000000000000000000000400000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000000301050400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000003000000000000000000000000000000000000000000000000000000000000006000000000000000000000000000000000000000000000000000000000000001a0000000000000000000000000000000000000000000000000000000000000022000000000000000000000000000000000000000000000000000000000000001200000000000000000000000000000000000000000000000000000000000000002000000000000000000000000000000000000000000001aabfc45f893dc5200000000000000000000000000000000000000000000000000000000000c0e26326400000000000000000000000000000000000000000000000000000000000000a0000000000000000000000000000000000000000000000000000000000000000100000000000000000000000000000000000000000000000000000000000000590b3e328455c4059eeb9e3f84b5543f74e24e7e1b00271042000000000000000000000000000000000000060001f4833589fcd6edb6e08f4c7c32d4f71b54bda02913000064d9aaec86b65d86f6a7b5b1b0c42ffa531710b6ca0000000000000000000000000000000000000000000000000000000000000000000000000000600000000000000000000000000b3e328455c4059eeb9e3f84b5543f74e24e7e1b0000000000000000000000005d64d14d2cf4fe5fe4e65b1c7e3d11e18d49309100000000000000000000000000000000000000000000001107035dd3ae32000000000000000000000000000000000000000000000000000000000000000000600000000000000000000000000b3e328455c4059eeb9e3f84b5543f74e24e7e1b0000000000000000000000008f24a47dcfa0e8bf0ee514e629ca7ff5ad1e68e1000000000000000000000000000000000000000000001a9af5429ac02e200000"
         }
      ],
      "0x151fb93"
   ]
}'

context menu
{"id":1,"jsonrpc":"2.0","result":[{"gas":"0xca66","gasUsed":"0xc906"},{"gas":"0xbab2","gasUsed":"0xbab2"},{"gas":"0x71516","gasUsed":"0x6a8e4"}]}%                                                                                                  
jsy1218 commented 1 day ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jsy1218 and the rest of your teammates on Graphite Graphite

socket-security[bot] commented 1 day ago

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@uniswap/smart-order-router@4.8.0 Transitive: environment, eval, filesystem, network, shell +100 114 MB

🚮 Removed packages: npm/@uniswap/smart-order-router@4.7.8

View full report↗︎

graphite-app[bot] commented 1 day ago

Graphite Automations

"Request reviewers once CI passes on routing-api repo" took an action on this PR • (11/08/24)

4 reviewers were added and 1 assignee was added to this PR based on 's automation.

jsy1218 commented 1 day ago

Merge activity