Skip to content

fix: updating the testcontainer dep #864

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

Closed
wants to merge 3 commits into from

Conversation

Dudleyneedham
Copy link
Member

@Dudleyneedham Dudleyneedham commented May 17, 2024

#3410

Tag wasn't updated in develop
https://hub.docker.com/r/kiltprotocol/standalone-node/tags?page=&page_size=&ordering=&name=develop

Found out that the branch doesn't use the docker images anymore and direct to testcontainer.

How to test:

Run the docker compose and check to see if working.

Checklist:

  • I have verified that the code works
  • I have verified that the code is easy to understand
    • If not, I have left a well-balanced amount of inline comments
  • I have left the code in a better state
  • I have documented the changes (where applicable)
    • Either PR or Ticket to update the Docs
    • Link the PR/Ticket here

@Dudleyneedham Dudleyneedham requested a review from ntn-x2 May 17, 2024 13:57
@@ -41,7 +41,7 @@ export async function getStartedTestContainer(
process.env.TESTCONTAINERS_NODE_IMG || 'kiltprotocol/standalone-node'
console.log(`using testcontainer with image ${image}`)
const strategies = [
['--dev', '--ws-external', `--ws-port=${WS_PORT}`],
// ['--dev', '--ws-external', `--ws-port=${WS_PORT}`],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trying to figure out why the test container isn't loading. To active the CI and see what is happening and why the container isn't loading.

@Dudleyneedham Dudleyneedham changed the title fix: docker image tag for develop fix: updating the testcontainer dep May 21, 2024
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