-
ErCiccione
Reminder: meeting today at 17:00 UTC
-
garth
-
ErCiccione
Hey folks, it's meeting time! The agenda is here:
monero-project/meta #700
-
ErCiccione
This is another meeting dedicated to the upcoming network upgrade. The logs of the last meeting are here:
monero-project/meta #684#issuecomment-1100806784
-
ErCiccione
but before we start, quick round of greetings to see who is here
-
ErCiccione
Hi everyone!
-
UkoeHB
hi
-
rbrunner
Hello
-
selsta
hi
-
jberman[m]
hello
-
ErCiccione
not crowded today. Let's wait a couple minutes more
-
selsta
should still be fine for a small status update
-
ooo123ooo123[m]
or not fine
-
ErCiccione
alright, let's go on. Hopefully people will join along the way
-
ErCiccione
so
-
ErCiccione
The detailed plan for the network upgrade is here:
monero-project/meta #630
-
ErCiccione
As far as i can tell, almost all the PRs we wanted in are already merged, except for multisig:
monero-project/monero #8149
-
ErCiccione
There are other PRs related to multisig. An overview is here:
monero-project/monero #8237
-
ErCiccione
8149 needs to be squashed and then a final review, am i correct?
-
selsta
.merges
-
xmr-pr
8046 8220 8226 8235 8262 8277 8278 8279
-
selsta
just so that we can see what is already in the merge queue
-
selsta
UkoeHB: kayabaNerve approved 8149, is there anything else to do for it?
-
UkoeHB
hopefully vtnerd can take a look
-
UkoeHB
since there were additions
-
sech1
hello
-
ErCiccione
ok, so we need a last review for vtnerd and then squash and merge.
-
ooo123ooo123[m]
and exploit
-
UkoeHB
exploit?
-
ErCiccione
UkoeHB: please ignore, probably a troll
-
hyc
hi, sorry wasn't watching the clock
-
ErCiccione
so, is multisig the only thing left? We need to have everything ready by may 16th
-
selsta
why may 16th?
-
UkoeHB
there is also 8220, I think that is waiting for merge
-
hyc
I should have gitian arm macosx ready to merge before then
-
UkoeHB
oh it's in queue
-
selsta
Branch/feature complete ok
-
sech1
May 16th was decided on the previus meeting
-
sech1
or the one before that
-
ErCiccione
selsta: yep, eature/branch complete by may 16th we said
-
ErCiccione
-
rbrunner
I hope to have my "nice to have" ready by then, I am in the home stretch with jberman reviewing:
monero-project/monero #8076
-
jberman[m]
doin my best on reviewing 8076 so we can get that in too, and hopefully 7760 but I doubt it on that one
-
ErCiccione
was about to ask you, that would be a good pr to have for the hard fork rbrunner
-
rbrunner
I am hopeful :)
-
ErCiccione
What about the multisig good to have UkoeHB?
monero-project/monero #8237
-
ErCiccione
some of those look mostly ready
-
UkoeHB
8220 is in merge queue, the other ones need more work and review (won't be ready by the 16th)
-
UkoeHB
the other ones aren't security-related
-
selsta
we can still merge things after may 16th, it's just when we try to branch
-
selsta
this "feature complete" date was never really strict
-
ErCiccione
i thought the point was to have everything ready as soon as possible, so there was one month of testing between may 16th and release date (june 16th)
-
ErCiccione
the testnet hard fork is also set for may 16th
-
garth
ErCiccione: question about your suggestion for the readme and multisig fixes: are the multi sig fixes a consensus breaking change?
-
UkoeHB
they are not
-
garth
Well then I don’t think they should be in the network upgrade history
-
ErCiccione
nope, just a major fix
-
ErCiccione
i thought we listed major changes in the readme, if that's not the case, sure better not add it
-
rbrunner
Well, as far as multisig is concerned, it's something like a break, with "before" and "after" ...
-
rbrunner
Just not strictly consensu
-
rbrunner
*consensus
-
garth
The list is already pretty long. But if people want non-consensus breaking upgrades listed in the network upgrade history that’s fine with me
-
ErCiccione
Let's continue that conversation in the issue garth
-
garth
It’s already ringsize 16, bulletproofs+, view tags, and fee adjustment
-
garth
Ok cool
-
ErCiccione
now, anybody with updates or questions, before we decide the date of the stagenet fork, as requested?
-
selsta
stagenet fork is usually 1 week before mainnet
-
ErCiccione
so July 9th
-
ErCiccione
Then the definitive plan is:
-
ErCiccione
-
ErCiccione
should we set a block height for the testnet fork?
-
hyc
the date is getting close, seems like we should
-
ErCiccione
yeah
-
rbrunner
We could leave that to the kind person doing the PR for that, calculating something for May 16
-
ErCiccione
we should really come up with something today IMO, or it gets too late
-
hyc
who will do the PR?
-
ErCiccione
that's why i wouldn't wait 😛
-
jberman[m]
I'll do it
-
ErCiccione
Cool. Thanks. do you already have a height in mind jberman?
-
jberman[m]
15: 1982800
-
jberman[m]
16: 1983520 (1 day later)
-
ErCiccione
In any case, if we don't set a date today we should do it as soon as possible 🙂
-
ErCiccione
oh, perfect
-
ErCiccione
So, to update the plan:
-
ErCiccione
-
ErCiccione
Any comments or something related to the network upgrade that you would like to discuss? Otherwise we can conclude the meeting
-
UkoeHB
thanks ErCiccione
-
ErCiccione
Alright then, thank you all for coming 🙂
-
ErCiccione
meeting adjourned!
-
ErCiccione
we can set another one for next week if we want? Maybe another for quick updates?
-
ErCiccione
we can also decide later on, but important dates are approaching
-
selsta
next saturday again?
-
jberman[m]
works for me
-
ErCiccione
would be ok for me
-
selsta
.merge+ 8280 8281 8300 8301
-
xmr-pr
Added
-
selsta
UkoeHB:
monero-project/monero #8293 is this simple enough or would you prefer a second review?
-
UkoeHB
selsta: it’s pretty simple, maybe could kayabaNerve take a look as well?
-
kayabaNerve
So long as an inversion is slower than a mul, which I'm assuming... ACK
-
selsta
.merge+ 8293
-
xmr-pr
Added
-
selsta
ty
-
jeffro256[m]
Hey y'all... According to this guy:
monero-project/monero #8309, the 17.3.2 source code doesn't compile on OpenBSD 7.1. I figured out that it was b/c their OpenSSL removed API elements that the in-tree unbound depended on. For security purposes, and so that OpenBSD can compile from source from the v17 branch, can we get that patched out ?
-
jeffro256[m]
selsta I saw that you made this commit to the master branch:
monero-project/monero #7773
-
jeffro256[m]
Will replaying that commit onto release-v0.17 work correctly ?
-
selsta
I intentionally didn't apply it to release-v0.17 because it might break build for some people that depend on the in tree one
-
selsta
e.g. the GUI had to do changes to address this, that's why I would prefer to change this in master only
-
selsta
jeffro256[m]: they closed the issue
-
jeffro256[m]
Is there an official plan to stop supporting it in release builds in the future?
-
selsta
we will branch to release-v0.18 soon from master and that will include everything from master
-
jeffro256[m]
Oh right because of the upcoming fork, sounds good thanks