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.

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

remove sentry node #49

Open
nicolasochem opened this issue Jun 3, 2021 · 7 comments
Open

remove sentry node #49

nicolasochem opened this issue Jun 3, 2021 · 7 comments

Comments

@nicolasochem
Copy link
Contributor

nicolasochem commented Jun 3, 2021

I get many missed bakes and endorsements since florence activation or v9 release, even with v9.2 which was supposed to have a fix for it.

I am trying now to run with "naked nodes" not protected by sentries, to see if it fixes it. I will post my findings.

We are an exception, no one else (kiln or big bakers) are using sentries for their baking operations.

I will monitor for a few days and sees if it fixes it. If it does, I will push the code for sentry removal here.

@denver-s
Copy link
Contributor

denver-s commented Jun 3, 2021

Main problem for missed endorsements is that the private node is not reliably connecting to the public nodes (due to bad local DNS probably).

If you choose to remove sentry nodes:

  • is security compromised? can it be still set as private node, with about 50 public trusted nodes as peers?
  • you could also "lower" the machine type (right now it costs 200 euro/month).

@nicolasochem
Copy link
Contributor Author

nicolasochem commented Jun 3, 2021 via email

@denver-s
Copy link
Contributor

denver-s commented Jun 7, 2021

How's it going?

@nicolasochem
Copy link
Contributor Author

I still have missed endorsements occasionally despite removing sentry nodes.

@nicolasochem
Copy link
Contributor Author

Nodes have plenty of cpu and ram, but it takes about 5 seconds to validate each block (completed in 7.650s ):

│ Jun  7 16:12:14.101 - validator.chain: Request pushed on 2021-06-07T16:12:13.846-00:00, treated in 1.351ms, completed in 251ms                                                                                                                             │
│ Jun  7 16:13:10.632 - validator.block: block BL1DAnEVpT7L1UAzTpBbPQvWmgGGA7RbxPa9oEbBTync4TchpWG successfully validated                                                                                                                                    │
│ Jun  7 16:13:10.632 - validator.block: Request pushed on 2021-06-07T16:13:02.982-00:00, treated in 7.187us, completed in 7.650s                                                                                                                            │
│ Jun  7 16:13:10.816 - prevalidator.NetXdQprcVkpa.PsFLorenaUUu: switching to new head BL1DAnEVpT7L1UAzTpBbPQvWmgGGA7RbxPa9oEbBTync4TchpWG                                                                                                                   │
│ Jun  7 16:13:10.816 - prevalidator.NetXdQprcVkpa.PsFLorenaUUu:  Request pushed on 2021-06-07T16:13:10.644-00:00, treated in 376us, completed in 171ms                                                                                                      │
│ Jun  7 16:13:10.925 - validator.chain: Update current head to Block Hash BL1DAnEVpT7L1UAzTpBbPQvWmgGGA7RbxPa9oEbBTync4TchpWG (level 1505379, timestamp 2021-06-07T16:12:58-00:00, fitness 01::00000000000cf863), same branch                               │
│ Jun  7 16:13:10.925 - validator.chain: Request pushed on 2021-06-07T16:13:10.631-00:00, treated in 1.958ms, completed in 289ms                                                                                                                             │
│ Jun  7 16:14:24.541 - validator.block: block BM5UfHWamYapeUANsVfpkyJNVWGboEGyohNFJEa2kG8tRqEWn8Q successfully validated                                                                                                                                    │
│ Jun  7 16:14:24.541 - validator.block: Request pushed on 2021-06-07T16:14:19.223-00:00, treated in 11.776us, completed in 5.317s                                                                                                                           │
│ Jun  7 16:14:25.329 - prevalidator.NetXdQprcVkpa.PsFLorenaUUu: switching to new head BM5UfHWamYapeUANsVfpkyJNVWGboEGyohNFJEa2kG8tRqEWn8Q                                                                                                                   │
│ Jun  7 16:14:25.329 - prevalidator.NetXdQprcVkpa.PsFLorenaUUu:  Request pushed on 2021-06-07T16:14:24.565-00:00, treated in 2.302ms, completed in 761ms                                                                                                    │
│ Jun  7 16:14:25.461 - validator.chain: Update current head to Block Hash BM5UfHWamYapeUANsVfpkyJNVWGboEGyohNFJEa2kG8tRqEWn8Q (level 1505380, timestamp 2021-06-07T16:13:58-00:00, fitness 01::00000000000cf864), same branch                               │
│ Jun  7 16:14:25.461 - validator.chain: Request pushed on 2021-06-07T16:14:24.540-00:00, treated in 5.146ms, completed in 912ms                                                                                                                             │
│ Jun  7 16:15:12.834 - validator.block: block BM2VtPURmfixu7F6t9Sgb5mDfHamQ18WTryDi28d1RMWpS4VtSa successfully validated                                                                                                                                    │
│ Jun  7 16:15:12.834 - validator.block: Request pushed on 2021-06-07T16:15:06.737-00:00, treated in 4.495us, completed in 6.96s                                                                                                                             │
│ Jun  7 16:15:13.081 - prevalidator.NetXdQprcVkpa.PsFLorenaUUu: switching to new head BM2VtPURmfixu7F6t9Sgb5mDfHamQ18WTryDi28d1RMWpS4VtSa                                                                                                                   │
│ Jun  7 16:15:13.081 - prevalidator.NetXdQprcVkpa.PsFLorenaUUu:  Request pushed on 2021-06-07T16:15:12.847-00:00, treated in 282us, completed in 233ms                                                                                                      │
│ Jun  7 16:15:13.183 - validator.chain: Update current head to Block Hash BM2VtPURmfixu7F6t9Sgb5mDfHamQ18WTryDi28d1RMWpS4VtSa (level 1505381, timestamp 2021-06-07T16:14:58-00:00, fitness 01::00000000000cf865), same branch                               │
│ Jun  7 16:15:13.183 - validator.chain: Request pushed on 2021-06-07T16:15:12.834-00:00, treated in 877us, completed in 346ms    

On Florencenet it's much faster.

@denver-s
Copy link
Contributor

denver-s commented Jun 9, 2021

I noticed an improvement without sentry nodes, so I'm in favor of removing them.

@nicolasochem
Copy link
Contributor Author

OK, will do.

I opened a ticket on tezos repo where they claim that it will get better once more bakers upgrade to 9.2

https://gitlab.com/tezos/tezos/-/issues/1446

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants