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

feat: use minio for python sdk doctest #281

Merged
merged 1 commit into from
Mar 6, 2023
Merged

Conversation

leric7
Copy link
Collaborator

@leric7 leric7 commented Mar 5, 2023

Description

If we use GCP for python sdk doctest, PRs from forked repo can't pass CI test.
By using minio for python sdk doctest, this can be resolved.

Summary of changes

  • Added docker compose for minio in python SDK.
  • Updated Github CI action not to use AWS credentials for GCP bucket.

How test the changes

Related issues

Keywords for linking issues
Python SDK test is failing on #259 , because it's created from forked PR.

Operational checklist

  • All new functionality is covered by tests
  • Any related documentation has been changed or added

@vercel
Copy link

vercel bot commented Mar 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
eth-kvstore-gui ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 5, 2023 at 10:21PM (UTC)
exchange-oracle ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 5, 2023 at 10:21PM (UTC)
fortune-exchange-mumbai ✅ Ready (Inspect) Visit Preview Mar 5, 2023 at 10:21PM (UTC)
fortune-reputation-oracle-mumbai ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 5, 2023 at 10:21PM (UTC)
human-protocol-escrow-dashboard ✅ Ready (Inspect) Visit Preview Mar 5, 2023 at 10:21PM (UTC)
job-launcher-client ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 5, 2023 at 10:21PM (UTC)
job-launcher-server ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 5, 2023 at 10:21PM (UTC)
recording-oracle ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 5, 2023 at 10:21PM (UTC)

@vercel vercel bot temporarily deployed to Preview – job-launcher-server March 5, 2023 22:17 Inactive
@leric7 leric7 requested a review from posix4e March 5, 2023 22:17
@vercel vercel bot temporarily deployed to Preview – fortune-reputation-oracle-mumbai March 5, 2023 22:18 Inactive
@vercel vercel bot temporarily deployed to Preview – job-launcher-client March 5, 2023 22:18 Inactive
@vercel vercel bot temporarily deployed to Preview – recording-oracle March 5, 2023 22:19 Inactive
@vercel vercel bot temporarily deployed to Preview – exchange-oracle March 5, 2023 22:20 Inactive
@vercel vercel bot temporarily deployed to Preview – eth-kvstore-gui March 5, 2023 22:21 Inactive
@leric7 leric7 merged commit 26adee0 into main Mar 6, 2023
leric7 added a commit that referenced this pull request Mar 6, 2023
@leric7 leric7 deleted the python-sdk-local-test branch March 8, 2023 00:25
eugenvoronov added a commit that referenced this pull request May 22, 2023
* Added Exchange Oracle basic template

* Added basic templates for all services with migrations and typeorm entities

* Change the provider of the fiat section

* Clean logs

* use minio for python sdk doctest (#281)

* fix duplicated dependency

* Added oracles, added new packages, updated orm config

* Job launcher/fiat payments (#333)

* Added payment service logic. Added job services base template

* Finalized payment module implementation. Added fortune and cvat job types

* Fixed payment module dto. Fixed jwt public guard

* Updated required node version to 18.x

* Job launcher/job types (#336)

* Added logic to process both types of job, Fortune and CVAT

* Implemented support for both types of jobs Fortune and CVAT

* Updated job migrations. Updated dto types. Tested job creation logic

* Job launcher/job types (#337)

* Added logic to process both types of job, Fortune and CVAT

* Implemented support for both types of jobs Fortune and CVAT

* Updated job migrations. Updated dto types. Tested job creation logic

* Added untracked files

* Implemented encryption module (#339)

* Change enums to specific folder and add currency service

* Job launcher/encrypt manifest (#341)

* Updated storage service. Updated create job and manifest encryption logic

* Updated readme

* Removed .github and assets folder

* Job launcher/encrypt manifest (#341)

* Updated storage service. Updated create job and manifest encryption logic

* Updated readme

* Removed .github and assets folder

* Delete contracts folder and create currency controller

* Change function name

* Add docker compose with minio and postgres services

* Update fortune v2 project name

* Added webhook incomming logic processing (#355)

* Added webhook incomming logic processing. Updated payment scheme

* Added pending incoming webhook processing with cron

* Added user balance logic (#357)

* Recording oracle/webhook (#358)

* Added basic notification service for oracle

* Fixed typos

* Added base webhook module (#359)

* Added oracle client (#360)

* Refactored job launcher

* Removed oracles services

* Updated dto

* Refactored service

* Moved mogrations

* Added cockie parser

* Resolved PRs comments

---------

Co-authored-by: Francisco López <francislopez977@gmail.com>
Co-authored-by: CryptoRush <98655210+leric7@users.noreply.github.com>
Co-authored-by: Eric Lee <eric@hmt.ai>
Co-authored-by: Francisco López <50665615+flopez7@users.noreply.github.com>
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