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

added dockerfile, workflow and entrypoint #1359

Closed
wants to merge 27 commits into from
Closed

added dockerfile, workflow and entrypoint #1359

wants to merge 27 commits into from

Conversation

liviuf-ancas
Copy link
Contributor

Reasoning

  • we need a public docker image that contains api-service to use in other projects too.

Proposed Changes

  • added dockerfile, docker entrypoint script

How to test

cfaur09 and others added 27 commits September 10, 2024 10:14
@cfaur09 @tanghel
fetch token price from mx-assets priceSource.url (#1290)
224baee 
* fetch token price from mx-assets priceSource.url

* update extractData methods

* use apiService.get to fetch result data

* update list of providers

* update key

* review & adjustments

* removed v2 from allEsdtTokens

---------

Co-authored-by: tanghel <tanghel@live.com>
@tanghel @GuticaStefan
@MogageNicolae @gabrielmatei
Nodes and tokens fetch from external api (#1306)
024e877 
* updated devnet config

* Signed commit test

* Signed commit restore changes

* Signed commit test

* Signed message restore changes

* Added tokens fetch from external API

* Added nodes fetch from external API

* Add ApiService as provider in tests

* Add nodes/tokens fetch features in config files

* Add unit tests for external api

* Divide requests for nodes and tokens in smaller requests containing maximum 1000 elements and bug fixes

* Wrong function call when fetching nodes fixed

* refactoring

* simplified node / token fetching

* Fixed tests bug

---------

Co-authored-by: GuticaStefan <stefan.gutica@gmail.com>
Co-authored-by: Nicolae Mogage <mogagenicolae@yahoo.com>
Co-authored-by: Gabriel Matei <gabrielvmatei@gmail.com>
@tanghel
default features disabled (#1323)
7e8f220
@cfaur09
add applications unit tests (#1321)
c004e34 
* Create applications.spec.ts

* Update schema.gql

* Update applications.spec.ts
@cfaur09
update API to latest sdk-nestjs version 3.7.4 (#1322)
33557b4
@GuticaStefan
Change rabbitmq docker image version from 3.5 to 3.9 (#1325)
999b7ac
@MogageNicolae @tanghel
@GuticaStefan @cfaur09
Integration of K6 into API Github workflow (#1309)
93859c4 
* Add workflow load-tests

* Add initialization of the project

* Fix port when waiting for API

* Add config for API

* Add folder content for debug

* Add folder content for debug before api start

* Add debug for config file

* Remove config.yaml from gitignore

* Add config.yaml file in dist

* Move config copy after build

* Add docker services

* Remove wrong added -

* Change rabbitmq version

* Add background API start

* Add docker installation in workflow

* Changed docker compose version use to start services

* Changed docker compose version

* Add docker compose global

* Change docker compose plugin version

* Change docker compose plugin version

* Update trend names

* Add cache preload

* Add 1 minute duration

* Add preload for all tokens/nodes

* Add preload for all tokens/nodes

* test

* Change table

* Change wrong closing tag for table

* Refactor generateTable function

* Add more endpoints

* Fix wrong name for Trends

* Add empty line at end of file

* Fix wrong trend calls

* Changed actions/upload-artifacts from v2 to v3

* changed actions/download-artifact from v2 to v3

* Clear docker images caching in workflow

* removed unncessary clear docker images cache

* Update load-tests.yml

* Update load-test.yml

* updated preload.js path

* update path for preload.js

* test

* disable preload cache

* revert ref base

* run action for PRs to main or development only

* update load-tests.yml

---------

Co-authored-by: tanghel <tanghel@live.com>
Co-authored-by: GuticaStefan <stefan.gutica@gmail.com>
Co-authored-by: cfaur09 <catalinfaurpaul@gmail.com>
@GuticaStefan @cfaur09
Fetch-providers-from-external-API (#1326)
cce46d1 
* added feature to fetch providers from external api instead of computing them everytime the api-service starts

* removed unnecessary return type

* added unit test for fetch providers from external api

* linter fix

* add more unit tests for providers service

---------

Co-authored-by: cfaur09 <catalinfaurpaul@gmail.com>
@cfaur09
Update nodes.spec.ts (#1330)
b2ed900
@GuticaStefan
tokens fetch from external api unit tests (#1331)
28dce31 
* upgrade tokens unit tests WIP

* add unit test for tokens

* fix lint

* remove console log
@tanghel
Filter NFTs by full tag (#1332)
2844314
@tanghel
removed erl_crash.dump file (#1337)
60ec4ae
@tanghel @cfaur09
Handle activation epoch where staking v4 flag is checked (#1338)
125e1e1 
* handle activation epoch where staking v4 flag is checked

* update specs

---------

Co-authored-by: cfaur09 <catalinfaurpaul@gmail.com>
@cfaur09 @tanghel
remove support for indexer v 3 (#1324)
c78cb2d 
* remove indexerV3 + specs

* fix specs

* remove support for indexerV5

* remove token.keyword

---------

Co-authored-by: tanghel <tanghel@live.com>
@cfaur09 @tanghel
applications assets (#1340)
acfaafa 
* add applications assets

* code formatting

---------

Co-authored-by: tanghel <tanghel@live.com>
@cfaur09
add support for token priceSource filter (#1342)
f69c275
@cfaur09 @tanghel
add mex token charts (#1334)
d5cb860 
* add mex token charts

* Create mex.token.charts.spec.ts

* check if given token has pair

* Update mex.token.charts.spec.ts

* user after instead of start query param

* add caching

* Update mex.token.charts.spec.ts

* fixes after review

* Update mex.token.charts.spec.ts

* fix format

* fixes after review

* small adjustments

---------

Co-authored-by: tanghel <tanghel@live.com>
@cfaur09
update deprecated pairs query (#1335)
abd7725 
* update deprecated pairs query

* add tradesCount24h

* add mex pair status and apply filter by state
@cfaur09
integrate-latest-sdk-nestjs-version (#1343)
4e16e63
@bogdan-rosianu
fix nft filters (#1344)
ca238af 
* fix nft filters

* fix failing unit tests
@cfaur09 @tanghel
relayed v3 features (#1299)
e409dd6 
* add innerTransactions

* add result status field

* add relayer field + filter

* remove relayerAddr from query

* update tests

* filter by relayer address

* add account transfer relayer

* remove relayerAddr

* fixes after review

* extract relayed version from ES

* revert changes

* add receiverUsername in innerTransaction

* fix specs

* Update mex.token.charts.spec.ts

---------

Co-authored-by: tanghel <tanghel@live.com>
@cfaur09 @tanghel
contracts deploys endpoints (#1336)
081bd5a 
* replace account contracts with account deploys

* add account contracts

* add AccountContract Entity instead of any

* Update mex.token.charts.spec.ts

---------

Co-authored-by: tanghel <tanghel@live.com>
@cfaur09
add support for relayer v3 (#1346)
ca1109a 
* add support for isRelayed in global trasfers endpoint

* small identation fix

* apply relayer field for transaction details

* add support for isRelayed filter
@cfaur09
add support for wildSearchcard accounts (#1347)
dce8f8b
@tanghel
Basic support for innerTx in transfers endpoint (#1351)
64040d5
@cfaur09 @tanghel
feat/sovereign (#1341)
595359b 
* add cacheDuration

* add support for NftSubType

* Update elastic.indexer.helper.ts

* Update nft.filter.ts

* Update nft.controller.ts

* add filter.type on collection elastic

* add subType even for SFT

* fix mex.token.charts.spec.ts

* Update nft.controller.ts

* revert changes

* add subType filter

* add collections subType filter

* add address collection roles subType

* add subType filter for roles/collections count

* add accounts/nfts subType filter

* add accounts/collection subType Filter

* improved support for subType filtering

* commented out field decoration for inner transactions in context of graphgql

* fix array subType filter

* Update elastic.indexer.helper.ts

* remove empty line

---------

Co-authored-by: tanghel <tanghel@live.com>
added dockerfile, workflow and entrypoint
ca01ed8
@github-actions GitHub Actions
Copy link

k6 load testing comparison.
Base Commit Hash: 595359b
Target Commit Hash: ca01ed8

Metric Base Target Diff
AvgMax9095AvgMax9095AvgMax9095
Blocks63.111254.0165.3078.3964.911257.4174.8190.30+2.86% 🔴+0.27% 🔴+14.56% 🔴+15.20% 🔴
Nodes57.13819.9564.6574.2258.611374.1871.4881.44+2.59% 🔴+67.59% 🔴+10.57% 🔴+9.72% 🔴
Mex55.93342.7564.5175.1656.55193.7971.1181.33+1.11% 🔴-43.46% ✅+10.23% 🔴+8.20% 🔴
Transactions89.592662.0871.2381.6593.232972.1579.9892.85+4.06% 🔴+11.65% 🔴+12.28% 🔴+13.71% 🔴
Accounts56.96658.3464.6074.2058.06919.8270.6582.09+1.94% 🔴+39.72% 🔴+9.36% 🔴+10.64% 🔴
Tokens56.80702.8864.5775.1757.39600.1570.8381.82+1.04% 🔴-14.62% ✅+9.69% 🔴+8.84% 🔴
Pool56.58585.7464.5274.6957.29618.2770.8181.58+1.25% 🔴+5.55% 🔴+9.75% 🔴+9.22% 🔴
Test Run Duration60005.5860004.89

Legend: Avg - Average Response Time, Max - Maximum Response Time, 90 - 90th Percentile, 95 - 95th Percentile
All times are in milliseconds.

cfaur09
cfaur09 previously approved these changes Oct 25, 2024
View reviewed changes
@dragos-rebegea dragos-rebegea changed the base branch from development to main October 28, 2024 15:16
@dragos-rebegea dragos-rebegea dismissed cfaur09’s stale review October 28, 2024 15:16

The base branch was changed.

@dragos-rebegea dragos-rebegea deleted the add-dockerfile-workflow-scripts-dev branch October 29, 2024 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cfaur09 cfaur09 cfaur09 left review comments

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

Successfully merging this pull request may close these issues.
7 participants
@liviuf-ancas @cfaur09 @dragos-rebegea @tanghel @GuticaStefan @MogageNicolae @bogdan-rosianu