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

doc: update RAEDME of dspot-web mojo #941

Merged
merged 106 commits into from
Feb 18, 2020

Conversation

borisbaldassari
Copy link
Contributor

Refactor dspot-web to include dspot-mojo, a web interface for dspot + descartes demonstration.

@monperrus
Copy link
Member

monperrus commented Feb 11, 2020

thanks for the pr.

the diff seems incorrect (too many changes + conflict), merge/rebase and recreate the PR?

@borisbaldassari
Copy link
Contributor Author

Hey Martin,
Ok, I don't know what happened. Will do that end of next week (we have our Crossminer review Tuesday) and will let you know.

@monperrus
Copy link
Member

monperrus commented Feb 14, 2020 via email

@borisbaldassari
Copy link
Contributor Author

borisbaldassari commented Feb 15, 2020 via email

@coveralls
Copy link

coveralls commented Feb 15, 2020

Pull Request Test Coverage Report for Build 2496

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 22 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.4%) to 83.167%

Files with Coverage Reduction New Missed Lines %
dspot/src/main/java/eu/stamp_project/dspot/common/test_framework/AbstractTestFrameworkDecorator.java 3 28.57%
dspot/src/main/java/eu/stamp_project/dspot/common/test_framework/TestFramework.java 3 88.37%
dspot/src/main/java/eu/stamp_project/dspot/amplifier/SimpleInputAmplDistributor.java 16 75.76%
Totals Coverage Status
Change from base Build 2492: -0.4%
Covered Lines: 4921
Relevant Lines: 5917

💛 - Coveralls

@monperrus
Copy link
Member

Thanks.

On the demo machine:

root@dovh1$ hypnotoad
bash: hypnotoad: command not found
debian@dovh1:~$ hypnotoad
bash: hypnotoad: command not found

Where is hypnotoad?

@borisbaldassari
Copy link
Contributor Author

borisbaldassari commented Feb 17, 2020 via email

@monperrus
Copy link
Member

OK, I can see it when I su dspot, thanks.

Next step, where is script/dspot_web on the machine?

@borisbaldassari
Copy link
Contributor Author

borisbaldassari commented Feb 17, 2020 via email

@monperrus monperrus closed this Feb 17, 2020
@monperrus monperrus reopened this Feb 17, 2020
@monperrus
Copy link
Member

~/dspot/dspot-web/v2-mojo/script/dspot_web yes! modified the README in this PR.

About the worker: how to stop it? how to check that it's up and running?'

@borisbaldassari
Copy link
Contributor Author

The easiest way to check if the worker is running is to go to the admin UI at [1].
[1] https://dspot-demo.stamp-project.eu/admin/minion

From the sysadmin perspective, the worker is running inside a screen on the dspot server. To see it just run screen -r to re-attache the screen and you should see the worker running (it's a foreground process so there should be lines printed on stdout).

@monperrus
Copy link
Member

Thanks, updated with this info b8fbe74

https://dspot-demo.stamp-project.eu/admin/minion says that it's working.

However, on the server, screen -r gives nothing, either as user dspot or as root.

Any idea?

@borisbaldassari
Copy link
Contributor Author

borisbaldassari commented Feb 18, 2020

However, on the server, screen -r gives nothing, either as user dspot or as root.

It should be run as user dspot, not root. And it should definitely work. It may be because I was connected and the screen process was indeed attached. In that case simply run screen -dr in order to detach it before re-attaching it.
I disconnected myself in any case so a simple screen -r should work now.

@monperrus monperrus changed the title Merge dspot-web mojo doc: update RAEDME of dspot-web mojo Feb 18, 2020
@monperrus monperrus merged commit 14cbb69 into STAMP-project:master Feb 18, 2020
@monperrus
Copy link
Member

OK, thanks a lot Boris for the last doc update.

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.

3 participants