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

feat: enhance simulation test README #2001

Merged
merged 6 commits into from
Nov 25, 2020
Merged

Conversation

kilrau
Copy link
Contributor

@kilrau kilrau commented Nov 23, 2020

Closes #1995

@kilrau kilrau requested a review from a user November 24, 2020 09:08
Copy link
Collaborator

@sangaman sangaman left a comment

Choose a reason for hiding this comment

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

A couple of comments, also one minor style nit for readability purposes would be to have a line of whitespace between the text and ``` blocks of code.

If you still have issues, make sure docker swarm is initialized and you have typescript installed:
```
docker swarm init
npm install -g typescript
Copy link
Collaborator

Choose a reason for hiding this comment

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

I don't have typescript installed globally but afaik that hasn't caused me any issues, is there a reason this would rely on the global typescript installation rather than the typescript dependency within this xud project?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure, added it because of #1995 (comment) and I believe it fixed a ts error I got once.

Copy link
Contributor

Choose a reason for hiding this comment

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

I later verified that indeed it's not required if you run npm install (which you must do anyway).
See here: #1995 (comment)

cd xud
npm run test:sim:clean
npm run test:sim:clean:indra
rm -rf ./test/simulation/temp
Copy link
Collaborator

Choose a reason for hiding this comment

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

The only things in the temp folder is logs and the indra folder (which is cleaned in the above step), so I'm not sure whether we need to include both steps here, but I suppose it doesn't hurt.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved to the second part.

@sangaman sangaman added the documentation Adding or improving documentation and code comments label Nov 24, 2020
@kilrau
Copy link
Contributor Author

kilrau commented Nov 24, 2020

But more importantly: can you get the tests to pass locally now? @sangaman Honestly, the time Moshe and I invested here was mainly to find steps and a environment (versions) for tests to pass for you.

@erkarl @michael1011 @moshababo and I can get tests to pass reliably by now.

@kilrau
Copy link
Contributor Author

kilrau commented Nov 25, 2020

Well I guess Daniel will let us know if tests still don't work for him.

@kilrau kilrau merged commit 9d71552 into master Nov 25, 2020
@ghost ghost deleted the feat/simulationtest-readme branch November 25, 2020 10:01
@sangaman
Copy link
Collaborator

Well I guess Daniel will let us know if tests still don't work for him.

Still no, unfortunately, tried all the steps here and am getting the same old:

2020/11/27 19:18:52 cannot start xud network: timeout waiting for TLS cert file to be created after 20 seconds: /app/temp/xuddatadir-Dave/tls.cert
exit status 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Adding or improving documentation and code comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Find a stable setup to run simulation tests locally
3 participants