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

docs: update test setup guide #778

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

redlarva
Copy link
Collaborator

@redlarva redlarva commented Nov 2, 2023

Description

Commit Message

docs: update test setup guide

see the guidelines for commit messages.

Changelog Entry

version: <log entry>

Checklist

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines
  • I have added version bump label on PR.

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@redlarva redlarva linked an issue Nov 2, 2023 that may be closed by this pull request
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 2, 2023
@redlarva redlarva added documentation Improvements or additions to documentation and removed documentation Improvements or additions to documentation labels Nov 2, 2023
@FidelVe
Copy link
Collaborator

FidelVe commented Nov 2, 2023

thanks for working on this, ill review it and let you know if I have any comments on it.

@FidelVe
Copy link
Collaborator

FidelVe commented Nov 2, 2023

I followed the updated steps, here are the issues im finding.

Running the step docker build -f Dockerfile.deprecated -t archway . --build-arg arch=aarch64 throws the following error:

#0 39.34 /usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
#0 39.34 /usr/lib/gcc/x86_64-alpine-linux-musl/12.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: skipping incompatible /lib/../lib/libwasmvm_muslc.a when searching for -lwasmvm_muslc
#0 39.34 /usr/lib/gcc/x86_64-alpine-linux-musl/12.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: skipping incompatible /lib/libwasmvm_muslc.a when searching for -lwasmvm_muslc
#0 39.34 /usr/lib/gcc/x86_64-alpine-linux-musl/12.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lwasmvm_muslc: No such file or directory
#0 39.34 collect2: error: ld returned 1 exit status
#0 39.34
#0 39.54 make: *** [Makefile:98: build] Error 2
------
Dockerfile.deprecated:30
--------------------
  28 |
  29 |     # force it to use static lib (from above) not standard libgo_cosmwasm.so file
  30 | >>> RUN LEDGER_ENABLED=false BUILD_TAGS=muslc LINK_STATICALLY=true make build
  31 |     RUN echo "Ensuring binary is statically linked ..." \
  32 |       && (file /code/build/archwayd | grep "statically linked")
--------------------
ERROR: failed to solve: process "/bin/sh -c LEDGER_ENABLED=false BUILD_TAGS=muslc LINK_STATICALLY=true make build" did not complete successfully: exit code: 2

Continued running steps and executed this command:

./scripts/execute-test.sh --build-ibc --build-xcall --use-docker --test e2e

Result returned error

--- FAIL: TestE2ETestSuite (197.85s)
--- FAIL: TestE2ETestSuite/TestE2E_all (197.84s)
--- FAIL: TestE2ETestSuite/TestE2E_all/test_xcall (0.72s)
FAIL
FAIL    github.com/icon-project/ibc-integration/test/e2e    197.910s
FAIL

@redlarva
Copy link
Collaborator Author

redlarva commented Nov 7, 2023

I followed the updated steps, here are the issues im finding.

Running the step docker build -f Dockerfile.deprecated -t archway . --build-arg arch=aarch64 throws the following error:

#0 39.34 /usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
#0 39.34 /usr/lib/gcc/x86_64-alpine-linux-musl/12.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: skipping incompatible /lib/../lib/libwasmvm_muslc.a when searching for -lwasmvm_muslc
#0 39.34 /usr/lib/gcc/x86_64-alpine-linux-musl/12.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: skipping incompatible /lib/libwasmvm_muslc.a when searching for -lwasmvm_muslc
#0 39.34 /usr/lib/gcc/x86_64-alpine-linux-musl/12.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lwasmvm_muslc: No such file or directory
#0 39.34 collect2: error: ld returned 1 exit status
#0 39.34
#0 39.54 make: *** [Makefile:98: build] Error 2
------
Dockerfile.deprecated:30
--------------------
  28 |
  29 |     # force it to use static lib (from above) not standard libgo_cosmwasm.so file
  30 | >>> RUN LEDGER_ENABLED=false BUILD_TAGS=muslc LINK_STATICALLY=true make build
  31 |     RUN echo "Ensuring binary is statically linked ..." \
  32 |       && (file /code/build/archwayd | grep "statically linked")
--------------------
ERROR: failed to solve: process "/bin/sh -c LEDGER_ENABLED=false BUILD_TAGS=muslc LINK_STATICALLY=true make build" did not complete successfully: exit code: 2

Continued running steps and executed this command:

./scripts/execute-test.sh --build-ibc --build-xcall --use-docker --test e2e

Result returned error

--- FAIL: TestE2ETestSuite (197.85s)
--- FAIL: TestE2ETestSuite/TestE2E_all (197.84s)
--- FAIL: TestE2ETestSuite/TestE2E_all/test_xcall (0.72s)
FAIL
FAIL    github.com/icon-project/ibc-integration/test/e2e    197.910s
FAIL

It seems you've encountered some difficulties in creating the Archway image. let me check it.

@redlarva redlarva force-pushed the feature/improvements-for-the-e2e-integration-test-setup-tutorial branch from dedee2c to 3552cbd Compare December 1, 2023 06:33
@github-actions github-actions bot added the test Testing update for Unit test, Integration test and E2E testing label Dec 1, 2023
@pragyanshrestha-ibriz pragyanshrestha-ibriz requested review from bcsainju and removed request for izyak March 20, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation test Testing update for Unit test, Integration test and E2E testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improvements for the e2e-integration-test-setup tutorial.
2 participants