Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

3.0.0-CI00130 #414

Merged
merged 7 commits into from
Jul 30, 2019
Merged

3.0.0-CI00130 #414

merged 7 commits into from
Jul 30, 2019

Conversation

erikzhang
Copy link
Member

No description provided.

@shargon
Copy link
Member

shargon commented Jul 26, 2019

The rest of the error will come in other PR?

Copy link
Member

@shargon shargon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is good to me, still having errors in neo-cli, but step by step

Copy link
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You adjust almost everything, @erikzhang. Like magic.
Running this preview release gonna be very good and interesting.

@erikzhang
Copy link
Member Author

Is good to me, still having errors in neo-cli, but step by step

What errors?

@shargon
Copy link
Member

shargon commented Jul 28, 2019

It doesn't compile to me

@vncoelho
Copy link
Member

vncoelho commented Jul 28, 2019

OMG, this guy is the master yoda.

No errors, @erikzhang:

Step 43/63 : RUN dotnet publish /opt/neoCli/neo-cli/neo-cli.csproj -o neo-cli -c Release -r ubuntu.16.04-x64
 ---> Running in 451cc52612e9
Microsoft (R) Build Engine version 15.9.20+g88f5fadfbe for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
Services/ConsoleServiceBase.cs(302,34): warning CS0168: The variable 'ex' is declared but never used [/opt/neoCli/neo-cli/neo-cli.csproj]
  neo-cli -> /opt/neoCli/neo-cli/bin/Release/netcoreapp2.1/ubuntu.16.04-x64/neo-cli.dll
  neo-cli -> /opt/neoCli/neo-cli/neo-cli/
Removing intermediate container 451cc52612e9
 ---> b273a309f635

@vncoelho
Copy link
Member

Compiled with these commits:

NEO_CLI_BLOCKCHAIN_URL=https://github.com/neo-project/neo-cli.git
NEO_CLI_BRANCH=3.0/neo-122
NEO_CLI_COMMIT=d19206bc743c71fdc81644bf7f88143fab744fa3
NEO_BLOCKCHAIN_URL=https://github.com/neo-project/neo.git
NEO_BLOCKCHAIN_BRANCH=master
NEO_BLOCKCHAIN_COMMIT=a9f46bc7d351933e7d8666665c49f850ed5d8e24
NEO_PLUGINS_URL=https://github.com/neo-project/neo-plugins.git
NEO_PLUGINS_BRANCH=master
NEO_PLUGINS_COMMIT=3774004b6a4bb45034458fd90698079640f6079e

LOCAL_NEO_CLI_BLOCKCHAIN="false"
LOCAL_NEO_CLI_BLOCKCHAIN_URL="localfiles/neo-cli"
LOCAL_NEO_BLOCKCHAIN="false"
LOCAL_NEO_BLOCKCHAIN_URL="localfiles/neo"
LOCAL_NEO_PLUGINS="false"
LOCAL_NEO_PLUGINS_URL="localfiles/neo-plugins"

SET_NEO_VM="true"
NEO_VM_URL=https://github.com/neo-project/neo-vm.git
NEO_VM_BRANCH=master
NEO_VM_COMMIT=332db08ada15916e69451fc9a23075574ac511a1

LOCAL_NEO_VM="false"
LOCAL_NEO_VM_URL="localfiles/neo-vm"

PLUGINS_LIST="
ApplicationLogs
CoreMetrics
ImportBlocks
RpcNep5Tracker
RpcSecurity
RpcWallet
SimplePolicy
StatesDumper
"

@shargon
Copy link
Member

shargon commented Jul 28, 2019

Travis have errors too

@erikzhang
Copy link
Member Author

I have no compilation errors here.

shargon
shargon previously approved these changes Jul 29, 2019
@shargon
Copy link
Member

shargon commented Jul 29, 2019

Now i haven't got compilation errors .... but the unit test still with errors

@superboyiii
Copy link
Member

Start consensus failed!
image

@vncoelho
Copy link
Member

Hi @superboyiii, you are ahead of us! ahueahuea
That is great that you are already testing.

Let's check it out. We need to double check the logic for simple policy and all initialization, the plugin might be not needed anymore.

@erikzhang erikzhang changed the title 3.0.0-CI00122 3.0.0-CI00130 Jul 29, 2019
@erikzhang
Copy link
Member Author

Fixed

@superboyiii
Copy link
Member

Cool! Works well now. Good fix!
image

@superboyiii
Copy link
Member

superboyiii commented Jul 30, 2019

"IsLowPriority" need to be removed in SimplePolicy since Transcation.get_IsLowPrioriy() has been removed.

image

neo-plugins#109

Acutally, almost every neo-plugin need to be adjusted in NEO3 since they all have some issues when installed.

@vncoelho
Copy link
Member

@superboyiii, you are right.
I think that maybe it is better to abolish the simplepolicy, I am not sure yet.
neo-project/neo#963

@shargon shargon merged commit 009757b into master Jul 30, 2019
@shargon shargon deleted the 3.0/neo-122 branch July 30, 2019 16:00
@superboyiii
Copy link
Member

superboyiii commented Jul 31, 2019

A conflict comes out in Neo3.0.0-CI00130 if an external node makes transaction via a wallet which is opened by consensus node at the same time.
Neo#970

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

Successfully merging this pull request may close these issues.

5 participants