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

DSL precondition to avoid creating orphan processes #291

Merged
merged 3 commits into from
Jan 9, 2020
Merged

DSL precondition to avoid creating orphan processes #291

merged 3 commits into from
Jan 9, 2020

Conversation

CjHare
Copy link
Contributor

@CjHare CjHare commented Jan 7, 2020

Signed-off-by: Christopher Hare chris.hare@consensys.net

PR description

State check to ensure that the non-unique node name (the is being used as a key in a map) does not inadvertently orphan an existing process sharing the same name.

Fixed Issue(s)

Christopher Hare added 2 commits January 7, 2020 13:18
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
Signed-off-by: Christopher Hare <chris.hare@consensys.net>
@CjHare CjHare merged commit 5535203 into hyperledger:master Jan 9, 2020
@CjHare CjHare deleted the avoid-orphan-processes branch January 9, 2020 03:50
AbdelStark pushed a commit to matkt/besu that referenced this pull request Jan 9, 2020
Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>
AbdelStark added a commit that referenced this pull request Jan 9, 2020
* refactor nat manager

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Add Tuweni to Plugin-APIs (#295)

Generally, byte[] -> Bytes of some form.  Most of the changes are the
side effect of the type changes or chaning to the names of Tuweni
equivilant calls (getHexString->toHexString, etc).

UnformattedData -> Bytes
Log Topics went from Hash to Bytes32
Difficulty went to UInt256 to match core impl.
Quantity lost BinaryData and is just getValue() and toHexString()

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* DSL precondition to avoid creating orphan processes (#291)

Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Fix AT failure because of additional field sent by Orion (#299)

Signed-off-by: Stefan Pingel <stefan.pingel@consensys.net>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatService.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove method

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Comment waitForFile: "besu.networks"

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* spotless apply

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* test

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* create file if not exist

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove useless code

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* fix waitForFile method

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Co-authored-by: CJ Hare <CjHare@users.noreply.github.com>
Co-authored-by: pinges <16143240+pinges@users.noreply.github.com>
Co-authored-by: Abdelhamid Bakhta <45264458+abdelhamidbakhta@users.noreply.github.com>
GregTheGreek pushed a commit to ChainSafe/besu that referenced this pull request Feb 4, 2020
…ger#190  (hyperledger#298)

* refactor nat manager

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Add Tuweni to Plugin-APIs (hyperledger#295)

Generally, byte[] -> Bytes of some form.  Most of the changes are the
side effect of the type changes or chaning to the names of Tuweni
equivilant calls (getHexString->toHexString, etc).

UnformattedData -> Bytes
Log Topics went from Hash to Bytes32
Difficulty went to UInt256 to match core impl.
Quantity lost BinaryData and is just getValue() and toHexString()

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* DSL precondition to avoid creating orphan processes (hyperledger#291)

Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Fix AT failure because of additional field sent by Orion (hyperledger#299)

Signed-off-by: Stefan Pingel <stefan.pingel@consensys.net>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatService.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove method

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Comment waitForFile: "besu.networks"

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* spotless apply

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* test

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* create file if not exist

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove useless code

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* fix waitForFile method

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Co-authored-by: CJ Hare <CjHare@users.noreply.github.com>
Co-authored-by: pinges <16143240+pinges@users.noreply.github.com>
Co-authored-by: Abdelhamid Bakhta <45264458+abdelhamidbakhta@users.noreply.github.com>
edwardmack pushed a commit to ChainSafe/besu that referenced this pull request Feb 4, 2020
Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
edwardmack pushed a commit to ChainSafe/besu that referenced this pull request Feb 4, 2020
…ger#190  (hyperledger#298)

* refactor nat manager

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Add Tuweni to Plugin-APIs (hyperledger#295)

Generally, byte[] -> Bytes of some form.  Most of the changes are the
side effect of the type changes or chaning to the names of Tuweni
equivilant calls (getHexString->toHexString, etc).

UnformattedData -> Bytes
Log Topics went from Hash to Bytes32
Difficulty went to UInt256 to match core impl.
Quantity lost BinaryData and is just getValue() and toHexString()

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* DSL precondition to avoid creating orphan processes (hyperledger#291)

Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Fix AT failure because of additional field sent by Orion (hyperledger#299)

Signed-off-by: Stefan Pingel <stefan.pingel@consensys.net>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatService.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove method

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Comment waitForFile: "besu.networks"

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* spotless apply

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* test

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* create file if not exist

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove useless code

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* fix waitForFile method

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Co-authored-by: CJ Hare <CjHare@users.noreply.github.com>
Co-authored-by: pinges <16143240+pinges@users.noreply.github.com>
Co-authored-by: Abdelhamid Bakhta <45264458+abdelhamidbakhta@users.noreply.github.com>
edwardmack pushed a commit to ChainSafe/besu that referenced this pull request Feb 4, 2020
Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
edwardmack pushed a commit to ChainSafe/besu that referenced this pull request Feb 4, 2020
…ger#190  (hyperledger#298)

* refactor nat manager

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* restart tests

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Add Tuweni to Plugin-APIs (hyperledger#295)

Generally, byte[] -> Bytes of some form.  Most of the changes are the
side effect of the type changes or chaning to the names of Tuweni
equivilant calls (getHexString->toHexString, etc).

UnformattedData -> Bytes
Log Topics went from Hash to Bytes32
Difficulty went to UInt256 to match core impl.
Quantity lost BinaryData and is just getValue() and toHexString()

Signed-off-by: Danno Ferrin <danno.ferrin@gmail.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* DSL precondition to avoid creating orphan processes (hyperledger#291)

Signed-off-by: Christopher Hare <chris.hare@consensys.net>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Fix AT failure because of additional field sent by Orion (hyperledger#299)

Signed-off-by: Stefan Pingel <stefan.pingel@consensys.net>
Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatService.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Update NatServiceType.java

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove method

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* SPDX-License-Identifier: Apache-2.0

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* Comment waitForFile: "besu.networks"

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* spotless apply

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* test

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* create file if not exist

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* remove useless code

Signed-off-by: Abdelhamid Bakhta <abdelhamid.bakhta@consensys.net>

* fix waitForFile method

Signed-off-by: Karim TAAM <karim.t2am@gmail.com>

Co-authored-by: Danno Ferrin <danno.ferrin@shemnon.com>
Co-authored-by: CJ Hare <CjHare@users.noreply.github.com>
Co-authored-by: pinges <16143240+pinges@users.noreply.github.com>
Co-authored-by: Abdelhamid Bakhta <45264458+abdelhamidbakhta@users.noreply.github.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
siladu pushed a commit to siladu/besu that referenced this pull request Oct 28, 2024
* wip

* more wip

* update contributor docs

* add testing docs

* few more updates

* update main readme

* fix spelling error

* fix more spelling

* fix line width for paragraph

* rm extra new line

* moar
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

Successfully merging this pull request may close these issues.

2 participants