Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

[PRIV] Prepare private transaction support #538

Merged
merged 5 commits into from
Jan 22, 2019

Conversation

iikirilov
Copy link
Contributor

PR description

Adds an Orion client with integration tests.

The purpose of this PR is to prepare for supporting private transaction in Pantheon. To support private transaction it is required to access an enclave from Pantheon. As Orion is an enclave created by the same organisation that maintains Pantheon it seems natural to support the Orion enclave first.

No functionality is added to Pantheon in this PR.

@CLAassistant
Copy link

CLAassistant commented Jan 10, 2019

CLA assistant check
All committers have signed the CLA.

@iikirilov iikirilov changed the title Prepare private transaction support [PRIV] Prepare private transaction support Jan 10, 2019
@antonydenyer antonydenyer force-pushed the orion branch 2 times, most recently from 75cb854 to c9614c6 Compare January 14, 2019 20:09
@macfarla macfarla self-assigned this Jan 15, 2019
testutil/build.gradle Outdated Show resolved Hide resolved
build.gradle Outdated Show resolved Hide resolved
@antonydenyer antonydenyer force-pushed the orion branch 2 times, most recently from da60b01 to c57080e Compare January 17, 2019 10:28
Copy link
Contributor

@NicolasMassart NicolasMassart left a comment

Choose a reason for hiding this comment

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

Do we have to include libraries binaries in the repository ?

@antonydenyer antonydenyer force-pushed the orion branch 2 times, most recently from ed337c7 to f5b76ca Compare January 18, 2019 08:50
@antonydenyer
Copy link
Contributor

Do we have to include libraries binaries in the repository ?

Yes but no! Until Consensys/orion#197 is fixed we need some kind of binary. When that's done we can switch over to using the snapshot.

The alternative is we ignore Orion integration and acceptance tests on CI and force developers to be running Orion locally.

@lucassaldanha
Copy link
Contributor

Yes but no! Until PegaSysEng/orion#197 is fixed we need some kind of binary. When that's done we can switch over to using the snapshot.

If all you need is an Orion binary with the latest changes, you can get it here! https://bintray.com/consensys/binaries/orion/0.1.0-SNAPSHOT

@rojotek
Copy link
Member

rojotek commented Jan 21, 2019

For today I am ok with the idea of us having the libs included. As soon as Orion ships with a pom file, it will be possible to delete these libs. Until we do that, we need to support the approach of including the files directly, in order to unblock the work in this area of the codebase.

Copy link
Member

@rojotek rojotek left a comment

Choose a reason for hiding this comment

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

lgtm

@rojotek rojotek dismissed macfarla’s stale review January 21, 2019 10:28

changes requested are either done, or scheduled for the future once supporting work is done.

@antonydenyer antonydenyer force-pushed the orion branch 5 times, most recently from e78b8c6 to 2619bf7 Compare January 22, 2019 09:03
antonydenyer added a commit to antonydenyer/pantheon that referenced this pull request Jan 22, 2019
commit 646fc06
Author: Antony Denyer <git@antonydenyer.co.uk>
Date:   Tue Jan 22 07:30:15 2019 +0000

    Use snapshot build of orion

commit c63a1bd
Author: Antony Denyer <git@antonydenyer.co.uk>
Date:   Mon Jan 21 14:57:37 2019 +0000

    update build image

commit bb98293
Author: Antony Denyer <git@antonydenyer.co.uk>
Date:   Mon Jan 14 16:51:01 2019 +0000

    Use ephemeral instance of Orion for integration test

    - Start orion on random port
    - test payloads can be send and received

commit 9c9dd38
Author: Puneetha <puneetha17@gmail.com>
Date:   Fri Jan 11 10:52:46 2019 +0000

    Add integration tests

commit 2441d30
Author: Puneetha <puneetha17@gmail.com>
Date:   Fri Jan 11 10:50:05 2019 +0000

    Add Orion module
@rojotek rojotek merged commit cf9e307 into PegaSysEng:master Jan 22, 2019
@Puneetha17 Puneetha17 deleted the orion branch January 24, 2019 18:14
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.

8 participants