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

local integration stack #738

Closed
wants to merge 7 commits into from
Closed

Conversation

MSalopek
Copy link
Contributor

@MSalopek MSalopek commented Feb 15, 2023

Description

Allows usin local sdk and gaia in integration tests:

STEPS

add gaia and cosmos-sdk to /interchain-security (copy/paste them into /interchain-security top level directory).
Change your gaia and cosmos-sdk as you need.

run:

go run ./tests/integration/... --happy-path-only --use-gaia --local-stack

check logs

docker exec -it interchain-security-instance bash
tail -f /provi/query/logs

OR

Get logs to your host machine

cd interchain-security
mkdir logs
docker cp interchain-security-instance:/provi/query/logs "./logs/query_$(date +%s)"

trace/debug logs

To set logs to trace or debug you must change tests/integration/testnet-scripts/start-chain.sh log levels (applies to all chains in the test suite)
Search for:

    # LOG_LEVEL="--log_level trace" # switch to trace to see panic messages and rich and all debug msgs
    LOG_LEVEL="--log_level info"

AND

# QUERY_LOG_LEVEL="--log_level trace" # switch to trace to see panic messages and rich and all debug msgs
QUERY_LOG_LEVEL="--log_level info"

@shaspitz
Copy link
Contributor

shaspitz commented Feb 15, 2023

Love the idea! My opinion is that we should merge the other two PRs first before developing this branch.

Feedback:

  • imo it's not neccessary to specify both a local gaia and sdk, because the local gaia could reference whatever sdk it wants from it's own go.mod
  • We can automate the process of making users copy/paste code, as is done for the use-local-sdk flag

@MSalopek
Copy link
Contributor Author

Closing since it is no longer relevant.

@MSalopek MSalopek closed this Apr 19, 2023
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