-
Notifications
You must be signed in to change notification settings - Fork 71
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
Improve Stuck Message Management #1469
Comments
This would be extremely welcome @dirkmc!! |
@TippyFlitsUK is there anything above that could be changed / added to make it more useful for you? |
we do not need the feature and wish to be able to turn this off if implemented. thanks the constant mpool watch most likely is too expensive for what we gain in the end and the replace command button is almost impossible to be implemented "right". besides the button to replace this most likely brings no benefit to users. educating how to set proper message fee limits in lotus/boost/... is way more important than UX improvements on crafting replacement messages. replacing messages should be painful to do, its nothing that should be casually done anyways - its a hint at bad config params regarding fees. |
Thanks f8, but I think a better UX can bring people a clear roadmap for setting gas fees. Not all operators are as clever as you did in the network. |
Hey Meta, i honestly doubt it makes sense to build a UX that allows the user to just pay to cover up configuration shortcomings. i think it's the wrong way to go, besides the implementation (if any kind of loop is involved) possibly being "dangerous" (we have seen miners pay tens of thousands of $$$ for single messages due to msg replace UX going off the rails under unexpected network conditions). as soon as the user will be in a position to use a button like that as a tool he doesn't need the button. compared to the command line replace commands a GUI version to actually make this a tool rather than a "pay for config fails" solution will be so clunky that it is hardly useful. but i might be wrong. as long as this is always a user triggered onetime event - why not, sure. but any solution beyond that, every inch of automation poisons the code base, i (and most likely a lot of other people) will not want that code base in an essential piece of software.
is all i am asking for here. |
I am closing this as we have completed all the improvements except the automatic/manual message replacement of the stuck message. The remaining work about message replacement is no longer viable for the engineering efforts required to implement it. We are no longer seeing stuck messages frequently. |
Background
Message Pool UX
Lotus CLI
Lotus has a CLI tool that can be used to see messages in the message pool:
Boost UI
Boost has a Message Pool page in the Web UI that shows a list of messages from the SP that are pending (they haven't been added to a block).
![Screenshot 2023-05-26 at 9 47 55 AM](https://private-user-images.githubusercontent.com/169124/241154256-c9ee834c-8d7a-49dc-beb0-7d33622cc255.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg5NzUyMzAsIm5iZiI6MTczODk3NDkzMCwicGF0aCI6Ii8xNjkxMjQvMjQxMTU0MjU2LWM5ZWU4MzRjLThkN2EtNDlkYy1iZWIwLTdkMzM2MjJjYzI1NS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA4JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwOFQwMDM1MzBaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0xMTAyMDA3N2JjODhkNWFhOWJiOGIxNDU1NGRmZDkwMzg4NTBmMTQ1YmU1YjI3NzJjYTgzNjI3MzIwZTc2MjdiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.EoGenZGJwU-ADpXtfJwm5D9zdY9pR6ID-odb2ViK99o)
Replacing a message
Lotus has a CLI command to replace messages that are stuck in the message pool - the message contents remain the same but the user can change the pricing parameters to make the message more attractive to miners, and therefore more likely to get added to a block.
Proposals
1. Surface Stuck Messages
Currently the SP only finds out that messages are stuck in the Message Pool if they
Solutions
1. Monitor the Message Pool
Boost should
2. Storage Deal List warning✅ Completed in #1480
Boost should display a warning message in the Storage Deal List if the Publish Storage Deals message gets stuck for more than the expected confirmation time.
Currently boost just shows "Awaiting Publish Confirmation"
![Screenshot 2023-05-26 at 10 01 18 AM](https://private-user-images.githubusercontent.com/169124/241157475-674c5750-f6f3-443c-8a72-a48a6e44464c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzg5NzUyMzAsIm5iZiI6MTczODk3NDkzMCwicGF0aCI6Ii8xNjkxMjQvMjQxMTU3NDc1LTY3NGM1NzUwLWY2ZjMtNDQzYy04YTcyLWE0OGE2ZTQ0NDY0Yy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMjA4JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDIwOFQwMDM1MzBaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT04ZmJhNGY0YjNkNDNlN2EzNDE0Y2Y2M2YxZjhhMTk3Yzc2NTU1NTUzNWZlMmM5OTQ0OTlkNTNiZmMxMGMyNDJiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.BGAdoLNSPy1g75wxhnqvBU_8woFIJnjlNs1yWDktlTM)
Boost should also show
2. Improve Message Pool UX
Solutions
1. Additional Message Information
On the Message Pool page, in the information displayed for each message there should also be
Fixes:
2. Replace Message Widget
Next to each message there should be a button called "Replace Message" that shows a widget to allow the SP to replace a stuck message.
3. Base Fee history
At the top of the page there should be a small graph showing base fee history over the last 24 hours:
The text was updated successfully, but these errors were encountered: