Development meeting held @ 3PM UTC in grincoin#dev channel on Keybase. Meeting lasted ~ 40 min.
Notes are truncated, and conversations sorted based on topic and not always chronological. Quotes are edited for brevity and clarity, and not always exact.
Community attendance:
- antiochp
- jaspervdm
- joltz
- lehnberg
- mcmmike
- phyro
- quentinlesceller
- vegycslol
- yeastplume
(apologies if I missed someone - submit a PR or contact @lehnberg to add)
- yeastplume: Past couple of weeks have been fairly quiet on the development front, though plenty of activities on rfcs and other ideas, so we can get right into them.
The proposed agenda was accepted, with "v4.1.0 release" added to the agenda on @antiochp's request.
-
joltz: Have been working with @mcmmike on some detection solutions. He has a great infrastructure already deployed for monitoring and so we are working to parse the logs to give useful feedback on "risk due to reorg" from the network. We still don't have a specific algo defined yet for no. of reorgs, depth, and time. If anyone has interest there please reach out to myself or Mike or feel free to comment here.
- 👍: lehnberg, yeastplume, quentinlesceller, phyro
-
lehnberg: No update on formula from my end. Anyone who wants, feel free to take this point over from me.
- joltz: I'll roll it into work with Mike. We'll propose something and then if people don't think it is as accurate as we'd like it to be, we can tweak it 👍
- 👍: lehnberg
- joltz: I'll roll it into work with Mike. We'll propose something and then if people don't think it is as accurate as we'd like it to be, we can tweak it 👍
-
mcmmike: My idea was:
- Lot of reorgs within $period_time
- If depth > 4 == might be done on purpose
I am not sure at which
depth
it might be done on purpose. -
joltz: That is a good place to start. I'm wondering if we can come up with a depth for "definitely done on purpose or a major outage of a major miner". I think I'd want to know at 5 depth, again at 10 depth and by beaten over the head with it at 25.
- 👍: mcmmike
- mcmmike: Ok we can start with these numbers, once we see an event like this we can all discuss if the numbers need to be adjusted. But atm we are not seeing more then "1" on all our oldest archive nodes.
-
joltz: We should also check number of confirmations required by major ecosystem services like exchanges. If they require 10, we want to know before then.
-
antiochp: Yeah I don't think we have seen a reorg beyond depth 1 or maybe 2 since early days of 1.0.
-
joltz: Then ideally this can be fed to a network health indicator widget that could be embedded in a website or something. Can color code network health based on this (and extend to other variables as we go to get a more complete picture).
- mcmmike: Right now we have some api calls checking our systems and displaying it here https://grinnode.live/global-health-check.
-
vegycslol: I think 3 is already highly unlikely to be random. How long it takes for a new block to be propagated to every node?
- joltz: Agreed based on antioch's comment. Maybe we want first indicator at 3 instead of 5.
- mcmmike: Perhaps we can come up with a widget or some api calls an exchange could query (with full documentatio).
- 👍: joltz
-
mcmmike: I can share it internally it's a private github repo, we did not want to make it public atm as the code is more or less quick and dirty and we are not good programmers. :) And credits are due to warmbeer (on discord) he wrote most of it.
-
joltz: Sounds good. Let's find a time this week or next to collaborate some more. If anyone here is interested ping me or mike and we can start a group chat or something 👍
-
mcmmike: Sure. Afk now (reading backlog and messages later).
-
yeastplume: Okay thanks @mcmmike.
- 👍: mcmmike
- yeastplume: So, we have diff adjustment, the fee rfc, and PIBD assignment. Jasper is working on PIBD now, what's the state of the other two?
- quentinlesceller: @tromp?
- antiochp: Have not seen @tromp around today.
- yeastplume: Ahh, I think he mentioned having something else on. Okay, well on daa I think there is still discussion going on around ntp and timestamps.
- yeastplume: And fees are looking like there's some agreement, will let tromp catch us up when he's around again.
- joltz: @paouky has been doing great work getting documentation up to speed with others in #docs. I think we wanted a 60-90 day window with exchanges to notify them of the required update, so that would likely be some time next month?
- 👍: yeastplume
- joltz: In that window we'd likely want to provide some extra support to the integrators as they go to help make sure we don't have major service downtimes after the last hf.
- yeastplume: Yes, definitely. And I look forward to a period of stability in that regard.
- joltz: Likewise. Do we know if any exchanges have already integrated slatepack or just the wallets so far? Would have been nice to get early exchange feedback but we'll survive.
- yeastplume: I don't know myself, would be good if anyone has more information on that. But yes, we'll survive.
- joltz: It would be nice to attempt to have some more formal rails in place to at least guide the conversations and consensus building but with so much up in the air around current governance structure, it is even more challenging to reason about today.
- yeastplume: Yeah, completely agreed. Will leave it there then for now.
- antiochp: Oh I guess I wanted to propose cutting 4.1.0 for node this week and forgot to add it to the agenda.
- yeastplume: Heh, sure, let's discuss that. We haven't really been doing any particular 4.1 planning with the focus on 5.
- antiochp: Yeah this would just be an interim release as we bumped the p2p protocol version recently. To support "commit only" inputs.
- yeastplume: When were you thinking of releasing it?
- antiochp: I think it is good to go now, at least to tag it and have a limited release of it this week. Then announce later this week or early next week.
- yeastplume: Sure, I think that's fine unless anyone sees any reason not to.
- 👍: joltz
- antiochp: Ok I can aim to do that later today unless there are any objections.
- 👍: joltz, jaspervdm, jaspervdm, yeastplume, yeastplume, lehnberg
Meeting adjourned.