Testnet June 2018 Mandatory Upgrade Testing Thread

Started by Rob Andrews, June 06, 2018, 09:45:32 PM

Previous topic - Next topic

noxpost

Quick comment/note about the new proposal view (which is really nice to see!) - it would be great if the spacing on the amounts was done the same way as in the rest of the wallet.

For instance, instead of:
8000000

It read:
8 000 000

Not a major thing by any means, but something to consider for a later leisure.

Rob Andrews

Quote from: noxpost on July 01, 2018, 12:48:19 PM
Quick comment/note about the new proposal view (which is really nice to see!) - it would be great if the spacing on the amounts was done the same way as in the rest of the wallet.

For instance, instead of:
8000000

It read:
8 000 000

Not a major thing by any means, but something to consider for a later leisure.

Thanks for the heads up, Yes, we'll add this to a coming leisure.
If you dont see it in the next version after the mandatory please add an issue to github.

Rob Andrews

#122
So here is a recap on where we are:

I'm nervous about the background prayers, so I put it back in the foreground serially (during load).  I tested this last night.  1.1.3.7 is checked in for linux and now compiling for windows.

As far as test cases, I did test the paid-proposals filter in prod and it passed.  The two proposals left in Prod are the ones we voted down to enter the replacement proposals.  They fall off after 31 days (since the entry date).

As far as test case coverage, to my knowledge everything else passed.

If anyone has any concerns please apprise us.



EDIT:  Windows version is out also.


Today,  I will consolidate all of our release notes together for the mandatory version and we can assess if we are ready for release.


Please take 1.1.3.7 for a spin to ensure nothing is awry.


togoshigekata

Updated to v1.1.3.7

left it running for a few hours, mnsync status  was stuck syncing, I was worried I was on a fork,
so I did a clean and reindex, mnsync status is still stuck syncing,
my masternode shows as Missing,
so I enabled it again and its in PRE_ENABLED state

getblockhash 46512
6268ef875***ea90589

Rob Andrews

Quote from: togoshigekata on July 02, 2018, 09:12:27 PM
Updated to v1.1.3.7

left it running for a few hours, mnsync status  was stuck syncing, I was worried I was on a fork,
so I did a clean and reindex, mnsync status is still stuck syncing,
my masternode shows as Missing,
so I enabled it again and its in PRE_ENABLED state

getblockhash 46512
6268ef875***ea90589

My hash matches.. let me do some sanc testing to make sure we are emitting PODC superblocks.


Rob Andrews

TheSnat, could you please see if you introduced this bug with the PODC filters?

https://github.com/biblepay/biblepay/issues/28


togoshigekata

My debug.log got filled up, I think its from leaving debug=true and debugmaster=true on for all this time, saved last 100 lines of log and removed it and commented out the config options

Rob Andrews

#127
Quote from: togoshigekata on July 03, 2018, 07:25:23 AM
My debug.log got filled up, I think its from leaving debug=true and debugmaster=true on for all this time, saved last 100 lines of log and removed it and commented out the config options

Only in testnet, or in prod?

EDIT: Ill run two nodes in prod on 1138 and see if something should be done if so we can rerelease a patch.

EDIT:  Is there anything in your saved file we should remove?

togoshigekata

This was in testnet, on v1.1.3.7.
Looks like its up to 21MB already hmmm

Im seeing a ton of this message:

ProcessBlockFound: Generated: ######
CPID has solved prior blocks. Contextual check block failed. CPID ######### ERROR: ProcessnewBlock: AcceptBlock FAILED
PrcoessBlockFound -- ProcessNewBlock() failed, block not accepeted
CBlock(hash= **** CTransaction ** CtxIn ** CTxOut

Maybe its because of too small amount of miners in testnet?

Hmmm I accidentally put a $ symbol in front of debugmaster=true instead of a hashtag to turn it off, I wonder what that did

Rob Andrews

Quote from: togoshigekata on July 03, 2018, 11:53:32 AM
This was in testnet, on v1.1.3.7.
Looks like its up to 21MB already hmmm

Im seeing a ton of this message:

ProcessBlockFound: Generated: ######
CPID has solved prior blocks. Contextual check block failed. CPID ######### ERROR: ProcessnewBlock: AcceptBlock FAILED
PrcoessBlockFound -- ProcessNewBlock() failed, block not accepeted
CBlock(hash= **** CTransaction ** CtxIn ** CTxOut

Maybe its because of too small amount of miners in testnet?

Hmmm I accidentally put a $ symbol in front of debugmaster=true instead of a hashtag to turn it off, I wonder what that did

Try removing or renaming debugmaster=true.  The parser probably interpreted that wrong.

Im using 231K on linux and 269K on windows in Prod on 1138 after running for a half hour and being fully synced.

21 Meg means something is really wrong on the node.  Could be those settings...

My log is clean now but I did release a small patch, but it only removed 2 messages.


Rob Andrews

Quote from: thesnat21 on July 04, 2018, 09:18:22 AM
submitted fix for the recent transaction issue to github.

Thanks, will look into this asap!

Been making great progress on stratis, need to put this down...



Dynaxius

Hi, Rob. Is the current plan for the upcoming BBP migration involving Stratis to create a fork of Stratis, or are you hoping to leverage the new sidechain tech they have released in recent months and actually deploy BBP beneath the Stratis Blockchain?

https://academy.stratisplatform.com/Sidechains/sidechains-introduction.html

Rob Andrews

Quote from: Dynaxius on August 14, 2018, 02:57:03 PM
Hi, Rob. Is the current plan for the upcoming BBP migration involving Stratis to create a fork of Stratis, or are you hoping to leverage the new sidechain tech they have released in recent months and actually deploy BBP beneath the Stratis Blockchain?

https://academy.stratisplatform.com/Sidechains/sidechains-introduction.html

We ported our POBH and kjv.cpp into stratis, so not sidechain, but actual real chain, and full compatibility with biblepay-qt production.  Except we haven't started porting the governance module yet, so you cant have a stratis Sanctuary (yet).