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

Make zksync default, add gnt-driver feature flag #784

Merged
merged 13 commits into from
Nov 16, 2020
Merged

Make zksync default, add gnt-driver feature flag #784

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
8037404
test multi payment
maaktweluit Nov 10, 2020
d6d54b6
Default driver
maaktweluit Nov 10, 2020
b066419
Updated readme with new feature flag
maaktweluit Nov 11, 2020
38a3c46
Update new branch name
maaktweluit Nov 12, 2020
7669d8f
Forgot multi driver flags
maaktweluit Nov 10, 2020
f1729a0
Update new branch name
maaktweluit Nov 12, 2020
bb4f615
temporary changes to the integration test command
maaktweluit Nov 13, 2020
4eefb1a
Updated github action files to build gnt-driver as well
maaktweluit Nov 13, 2020
714d551
Update asset-path to match yagna repo, removed filter for only multi …
maaktweluit Nov 13, 2020
ea8a021
Review comments: Make gnt and zksync drivers part of default features
maaktweluit Nov 13, 2020
ff3a3f9
Remove test branch from ya-int, updated python version to 3.8
maaktweluit Nov 13, 2020
2920608
Update ya-client to fix typo
maaktweluit Nov 13, 2020
e64df14
Revert parts of Cargo.lock
maaktweluit Nov 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 4 additions & 5 deletions .github/workflows/integration-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Integration test
on:
workflow_run:
workflows: ["CI"]
types:
types:
- completed

jobs:
Expand All @@ -17,10 +17,10 @@ jobs:
repository: 'golemfactory/yagna-integration'
token: ${{ secrets.YAGNA_WORKFLOW_TOKEN }}

- name: Configure python 3.7
- name: Configure python 3.8
uses: actions/setup-python@v2
with:
python-version: '3.7'
python-version: '3.8'

- name: Run setuptools
run: python setup.py develop
Expand All @@ -34,12 +34,11 @@ jobs:
- name: Run test suite
env:
GITHUB_API_TOKEN: ${{ secrets.YAGNA_WORKFLOW_TOKEN }}
run: python -m pytest test/yagna/e2e -svx --log-cli-level=DEBUG
run: python -m pytest test/yagna --assets-path=/home/runner/work/yagna/yagna/test/yagna/e2e/assets -svx --log-cli-level=DEBUG

- name: Upload test logs
uses: actions/upload-artifact@v2
if: always()
with:
name: goth-logs
path: /tmp/yagna-tests

8 changes: 4 additions & 4 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ license = "GPL-3.0"
edition = "2018"

[features]
default = ['gnt-driver', 'gftp/bin']
default = ['zksync-driver', 'gnt-driver', 'gftp/bin']
static-openssl = ["openssl/vendored"]
dummy-driver = ['ya-dummy-driver']
gnt-driver = ['ya-gnt-driver']
Expand Down Expand Up @@ -167,8 +167,8 @@ ya-sb-router = { path = "service-bus/router" }
ya-sb-util = { path = "service-bus/util" }

## CLIENT
ya-client = { git = "https://github.com/golemfactory/ya-client.git", rev = "fe426adc89682be018d65504c0bfb8ab9580fcae"}
ya-client-model = { git = "https://github.com/golemfactory/ya-client.git", rev = "fe426adc89682be018d65504c0bfb8ab9580fcae"}
ya-client = { git = "https://github.com/golemfactory/ya-client.git", rev = "80221a38c4cf8f4700da36598316fc8048e5c535"}
ya-client-model = { git = "https://github.com/golemfactory/ya-client.git", rev = "80221a38c4cf8f4700da36598316fc8048e5c535"}

## OTHERS
gftp = { path = "core/gftp" }
Expand Down
12 changes: 6 additions & 6 deletions core/payment/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,18 +19,18 @@ You can enable multiple drivers at the same time, use this table for the require

|Driver name|Platform name|Feature flag|Public explorer|Local|Testnet|Mainnet|
|-|-|-|-|-|-|-|
|zk-ngnt|ZK-NGNT|`ya-zksync-driver`|[zkscan](https://rinkeby.zkscan.io/)|x|x||
|ngnt|NGNT|`ya-ngnt-driver`|[etherscan](https://rinkeby.etherscan.io/token/0xd94e3dc39d4cad1dad634e7eb585a57a19dc7efe)|x|x||
|dummy|DUMMY|`ya-dummy-driver`|None|x|||
|zk-ngnt|ZK-NGNT|`zksync-driver`|[zkscan](https://rinkeby.zkscan.io/)|x|x||
|ngnt|NGNT|`gnt-driver`|[etherscan](https://rinkeby.etherscan.io/token/0xd94e3dc39d4cad1dad634e7eb585a57a19dc7efe)|x|x||
|dummy|DUMMY|`dummy-driver`|None|x|||

### Examples:

Build with only gnt driver:
Build with zksync + ngnt driver:
```
cargo build --release
```

Build with ngnt + zksync
Build with only zksync ( not recommended )
```
cargo build --release --features zksync-driver
cargo build --release --no-default-features --features zksync-driver
```
7 changes: 2 additions & 5 deletions core/payment/src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,8 @@ pub mod migrations {
struct _Dummy;
}

// // Uncomment zksync and comment ngnt, to test zksync invoice flow
// pub const DEFAULT_PAYMENT_PLATFORM: &str = "ZK-NGNT";
// pub const DEFAULT_PAYMENT_DRIVER: &str = "zksync";
pub const DEFAULT_PAYMENT_PLATFORM: &str = "NGNT";
pub const DEFAULT_PAYMENT_DRIVER: &str = "ngnt";
pub const DEFAULT_PAYMENT_PLATFORM: &str = "ZK-NGNT";
pub const DEFAULT_PAYMENT_DRIVER: &str = "zksync";

pub struct PaymentService;

Expand Down