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.

Sign up for GitHub

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

Jump to bottom

Serve multi tenants from a single node #72

Merged
merged 33 commits into from
Jan 29, 2021
Merged

Serve multi tenants from a single node #72

merged 33 commits into from
Jan 29, 2021

Conversation

trung
Copy link
Contributor

@trung trung commented Dec 17, 2020

Specs and scenarios for Consensys/quorum#1105

Additional improvements:

  • Unlock multiple Ethereum accounts
  • Allow to inject local binaries to containers for faster feedback loop during development
  • Improve polling transaction receipts logic
  • Add assertion logic for contract extension status polling

trung and others added 28 commits January 8, 2021 09:38
@trung
add multitenancy specs and glue code implementation
a95706b
@trung
support additional vol binding for development
d1c04dd
@prd-fox @trung
Don't construct error observable until an error actually occurs.
749cafc 
Make the contract returned optional, for the cases where we try to deploy a contract but expect it to fail.
@prd-fox @trung
Don't construct error observable until an error actually occurs.
ed70c39 
Make the contract returned optional, for the cases where we try to deploy a contract but expect it to fail.
@trung
added scenarios for eth_sendTransaction flow
4f75e25
@prd-fox @trung
Split public transactions into node-signed and self-managed scenarios.
7bed58e
@trung
remove party.tm param, use only from.tm to align with scope design up…
5ef4aee 
…date
@trung
add eth_call scenarios and make other scenarios more readable and cle…
9f7519c 
…arer
@nicolae-leonte-go @trung
multitenant bypass tx simulation controls
4b15475
@trung
add assertion for contract extension status check
23e939d
@nicolae-leonte-go @trung
update sneaky wrapper tests to reflect the post execution checks impl…
fa864d7 
…emented in quorum
@nicolae-leonte-go @trung
update sneaky wrapper tests to reflect the post execution checks impl…
da00fe1 
…emented in quorum
@trung
add contract extension scenarios with multi tenancy support
9b7c03e
@trung
add contract extension scenarios with multi tenancy support
b11f869
@trung
add contract extension scenarios with multi tenancy support
6871e10
@trung
refactor multitenancy specs to reflect a meaningful setup
4cc32c5
@trung
add acceptance tests for contract extension
6de3f8a
@nicolae-leonte-go @trung
update multitenancy contract extension tests
6c8e085
@trung
resolve conflicts
8c72ec4
@nicolae-leonte-go @trung
update multitenancy contract extension tests
a431a06
@trung
add negative tests
7020852
@trung
add negative tests
47967e8
@nicolae-leonte-go @trung
update multitenancy contract extension tests
3b10171
@trung
add multitenant test execution to the suite
da4b4a1
@trung
add scenario steps per review feedbacks
769fc6a
@trung
update scenarios to check for 0 log events instead of 'not authorized'
ce67cd7
@nicolae-leonte-go @trung
change behavior for logs and receipts retrieval to match the changes …
d0521a6 
…in quorum
@trung
add scenarios for events handling in transaction receipt
0bfb3c7
trung added 3 commits January 22, 2021 15:19
@trung
add scenarios fo contract code access
6bf4de9
@trung
run acceptance tests with custom images
5cbfaa6
@trung
update README with new improvements
7cb8f92 
- Run tests with custom Docker images in Github Actions workflow
- Run tests with local binaries
nmvalera
nmvalera approved these changes Jan 28, 2021
View reviewed changes
Copy link
Contributor

@nmvalera nmvalera left a comment

Choose a reason for hiding this comment

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

LGTM

@nmvalera nmvalera merged commit fb4c349 into Consensys:master Jan 29, 2021
@trung trung deleted the multitenant branch January 29, 2021 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nmvalera nmvalera nmvalera approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@trung @nmvalera @prd-fox @nicolae-leonte-go