ETH-585 docker images build and test in CI #616

samt1803 · 2023-08-29T08:15:53Z

added smoketest that queries thegraph for all the exampledata from the chain-preload, if this works, the images both work and work together, meaning all addresses also match

This changes the CI integration test

…e chain-preload, if this works, the images both work and work together, meaning all addresses also match

…d" step

… inside the docker it needs to run on its own

* master: doc: push to Docker Hub instructions release(network-contracts): 5.0.0

… versions inside docker after installing the correct version first

linear · 2023-09-01T07:17:19Z

ETH-585 Fastchain+subgraph docker images integration test in CI

add a test in CI that

builds the images (fastchain image, subgraph-deploy image)
starts them up as containers and then
does a smoketest against them
maybe publishes them to dockerhub? (needs to build multiarch images for that, maybe only do this on merge to master)

…hub-contracts

packages/network-contracts/test/integration/contractsSubgraphSmoketest.test.ts

packages/network-contracts/package.json

.github/workflows/ci.yaml

* master: (22 commits) release(config): @streamr/config 4.0.4 streamr 1.0 mumbai deployment addresses update Modularize operator (#629) doc: add clean step to push-to-docker-hub instructions release(network-contracts): 6.0.1 fix: remove accidentally duplicated function Eth 584 erc677 in operator factory (#627) added hardhat-contract-sizer to contract build process ETH-579: Check Operator contract function guards (#628) deoc: added addresses preloaded with DATA to fastchain readme release(config): @streamr/config 4.0.3 new addresses for mumbai streamr 1.0 deployment release(network-contracts): 6.0.0 Rename getEarningsFromSponsorships -> getSponsorships (#626) add mumbai streamr 1.0 testnet contract addresses Protocol fee collection (ETH-572) (#625) Sorted operator tests under describe blocks (#621) fix: potential re-entrancy issue (#620) build: bump version release(config): @streamr/config 4.0.2 ... # Conflicts: # package-lock.json # packages/network-contracts/package.json

samt1803 · 2023-09-14T09:03:12Z

It is now blocked by the ENS PR not being merged yes, so the integration test here fails because ENS sync is not yet adapted to the fastchain env.
I am taking ENS integration out from this PR, and will re-enable it on the other ENS PR (streamr-dev/streamr-docker-dev#215)

…o fastchain, will re-enable it in that PR

…uild_smoke * origin/master: fix: merge hazard from flagWithMetadata PR cleanup: reorder setParam function, uint256 -> uint in Policies feat: make slashing fraction configurable (#624) rename flagWithMetadata -> flag, and remove the previous implementation (#630)

* address_change_hotfix: contract adresses changed, also in subgraoph

packages/hub-contracts/package.json

…lthcheck * origin/master: (32 commits) release(network-contracts): v6.1.0 release(config): @streamr/config 4.0.5 ETH-585 docker images build and test in CI (#616) Eth 596 subgraph fixeschanges (#631) fix: merge hazard from flagWithMetadata PR cleanup: reorder setParam function, uint256 -> uint in Policies feat: make slashing fraction configurable (#624) rename flagWithMetadata -> flag, and remove the previous implementation (#630) release(config): @streamr/config 4.0.4 streamr 1.0 mumbai deployment addresses update Modularize operator (#629) doc: add clean step to push-to-docker-hub instructions release(network-contracts): 6.0.1 fix: remove accidentally duplicated function Eth 584 erc677 in operator factory (#627) added hardhat-contract-sizer to contract build process ETH-579: Check Operator contract function guards (#628) deoc: added addresses preloaded with DATA to fastchain readme release(config): @streamr/config 4.0.3 new addresses for mumbai streamr 1.0 deployment ...

* ens-sync-script-healthcheck: (39 commits) codereview changerequest changes release(network-contracts): v6.1.0 release(config): @streamr/config 4.0.5 ETH-585 docker images build and test in CI (#616) Eth 596 subgraph fixeschanges (#631) fix: merge hazard from flagWithMetadata PR cleanup: reorder setParam function, uint256 -> uint in Policies feat: make slashing fraction configurable (#624) rename flagWithMetadata -> flag, and remove the previous implementation (#630) release(config): @streamr/config 4.0.4 streamr 1.0 mumbai deployment addresses update Modularize operator (#629) doc: add clean step to push-to-docker-hub instructions release(network-contracts): 6.0.1 fix: remove accidentally duplicated function Eth 584 erc677 in operator factory (#627) added hardhat-contract-sizer to contract build process ETH-579: Check Operator contract function guards (#628) deoc: added addresses preloaded with DATA to fastchain readme release(config): @streamr/config 4.0.3 ... # Conflicts: # .github/workflows/ci.yaml # packages/network-contracts/package.json

* master: Add a test that parallels an Operator plugin test in network repo (#633) updated ens-sync-script to work with fastchain (#615) Sponsorship fix: don't let _reduceStakeBy revert when over-slashing (#632) doc: update build and publish images description doc: publish instructions release(network-contracts): v6.1.0 release(config): @streamr/config 4.0.5 ETH-585 docker images build and test in CI (#616) Eth 596 subgraph fixeschanges (#631) fix: merge hazard from flagWithMetadata PR cleanup: reorder setParam function, uint256 -> uint in Policies feat: make slashing fraction configurable (#624) rename flagWithMetadata -> flag, and remove the previous implementation (#630) release(config): @streamr/config 4.0.4 streamr 1.0 mumbai deployment addresses update # Conflicts: # package-lock.json # packages/network-subgraphs/subgraph.yaml

added smoketest that queries thegraph for all the exampledata from th…

28a64f4

…e chain-preload, if this works, the images both work and work together, meaning all addresses also match

samt1803 marked this pull request as draft August 29, 2023 08:17

samt1803 added 2 commits August 29, 2023 13:18

debugging ci workflow

df1daba

update package lock

8388089

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from 185fa1a to 762dd47 Compare August 31, 2023 07:25

fix installing local packages overwriting package.json entries

ed0d395

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from 762dd47 to ed0d395 Compare August 31, 2023 07:34

samt1803 added 3 commits August 31, 2023 10:45

fix subgraph docker image build

691b498

add graph-cli to buildprocess

f8bac52

skip subgraph build on host

4136893

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from b7ac106 to 4136893 Compare August 31, 2023 08:18

samt1803 added 2 commits August 31, 2023 12:07

build packages in CI step, don't build fastchainimage on global "buil…

bc89c69

…d" step

update package.json and package-lock, decouple subrepo from monorepo,…

38c11fa

… inside the docker it needs to run on its own

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from 1787de2 to 38c11fa Compare August 31, 2023 11:02

samt1803 added 2 commits September 1, 2023 10:11

Merge branch 'master' into ETH-585_CI_docker_build_smoke

0cbf7b9

* master: doc: push to Docker Hub instructions release(network-contracts): 5.0.0

remove fixed version of streamr packages, otherwise it pulls outdated…

d9e3af9

… versions inside docker after installing the correct version first

samt1803 changed the title ~~added smoketest that queries thegraph for all the exampledata from …~~ ETH-585 docker images build and test in CI Sep 1, 2023

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from 6aaa0ce to 476b587 Compare September 1, 2023 12:45

completely revamp fastchain image creation, clean up dependencies of …

099a437

…hub-contracts

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from 476b587 to 099a437 Compare September 1, 2023 13:09

samt1803 added 2 commits September 1, 2023 16:21

test graph entities with static and also dynamic ids

37ea38e

cleanup

cea22ec

samt1803 requested review from jtakalai and samigabor September 1, 2023 14:08

samt1803 marked this pull request as ready for review September 1, 2023 14:08

jtakalai suggested changes Sep 4, 2023

View reviewed changes

packages/network-contracts/test/integration/contractsSubgraphSmoketest.test.ts Outdated Show resolved Hide resolved

packages/network-contracts/package.json Outdated Show resolved Hide resolved

.github/workflows/ci.yaml Outdated Show resolved Hide resolved

samt1803 added 3 commits September 14, 2023 10:45

make test green even if ids of example data entities change

a269b4e

review change requests

57f9abd

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch 2 times, most recently from 41c48fb to 44502eb Compare September 14, 2023 08:24

force ci to install linux x64 amd gnu specific package

05a0f1a

samt1803 force-pushed the ETH-585_CI_docker_build_smoke branch from 44502eb to 05a0f1a Compare September 14, 2023 08:28

jtakalai self-requested a review September 14, 2023 09:27

samt1803 added 5 commits September 14, 2023 14:01

disable ens integration test until the PR is merged, that adapts it t…

cb598ba

…o fastchain, will re-enable it in that PR

make contract size too big an error

2a8ab85

contract adresses changed, also in subgraoph

bf6f9d2

Merge branch 'address_change_hotfix' into ETH-585_CI_docker_build_smoke

dbc90b5

* address_change_hotfix: contract adresses changed, also in subgraoph

jtakalai suggested changes Sep 14, 2023

View reviewed changes

packages/hub-contracts/package.json Show resolved Hide resolved

packages/hub-contracts/package.json Outdated Show resolved Hide resolved

moved some dependencies to devdepoendencies

8235816

samt1803 requested a review from jtakalai September 14, 2023 13:24

jtakalai approved these changes Sep 14, 2023

View reviewed changes

samt1803 merged commit 44afdad into master Sep 14, 2023

samt1803 deleted the ETH-585_CI_docker_build_smoke branch September 14, 2023 14:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

ETH-585 docker images build and test in CI #616

ETH-585 docker images build and test in CI #616

Uh oh!

samt1803 commented Aug 29, 2023 •

edited

Loading

Uh oh!

linear bot commented Sep 1, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

samt1803 commented Sep 14, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ETH-585 docker images build and test in CI #616

ETH-585 docker images build and test in CI #616

Uh oh!

Conversation

samt1803 commented Aug 29, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

linear bot commented Sep 1, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

samt1803 commented Sep 14, 2023

Uh oh!

Uh oh!

Uh oh!

Uh oh!

samt1803 commented Aug 29, 2023 •

edited

Loading