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

Topology does not show running nodes? #5

Open
russellday opened this issue Aug 3, 2016 · 2 comments
Open

Topology does not show running nodes? #5

russellday opened this issue Aug 3, 2016 · 2 comments

Comments

@russellday
Copy link

russellday commented Aug 3, 2016

Wondering if you have any troubleshooting suggestions? The only modification I made is to config.json to use port 80. I started ignite using ignite.sh examples/config/example-cache.xml. Any suggestions would be appreciated. Running a curl on http://localhost:8080/ignite?cmd=version returns the correct Ignite version so it appears the rest api is running.

2016-08-03T18:26:26.682Z - info: Configuration loaded successfully from /home/ec2-user/aiex/config.json rootPath=/api, port=80, zmqSocketAddress=tcp://127.0.0.1:5555
2016-08-03T18:26:27.209Z - info: Web Server Ready on port: 80
2016-08-03T18:26:51.457Z - info:
2016-08-03T18:26:51.501Z - info: Error: getaddrinfo ENOTFOUND
at errnoException (dns.js:37:11)
at Object.onanswer as oncomplete
2016-08-03T18:27:04.512Z - info: key=abc, val=123
2016-08-03T18:27:04.515Z - info: Error: getaddrinfo ENOTFOUND
at errnoException (dns.js:37:11)
at Object.onanswer as oncomplete

@russellday
Copy link
Author

I found the issue. I believe there is a bug in the logical Or statement below (..modules/ignite/rest-service.js):

var rootUrl = "http://" + process.env.IGNITE_REST + "/ignite?cmd=" || "http://localhost:8080/ignite?cmd=";

if IGNITE_REST is undefined it evaluates to "http://undefined/ignite?cmd=".

I modified as shown below.

var r = process.env.IGNITE_REST;
var rootUrl = r !== undefined ? "http://" + process.env.IGNITE_REST + "/ignite?cmd=" : "http://localhost:8080/ignite?cmd=";

@artbasov
Copy link

@russellday This solves the issue for me as well. Mind issue PR ? Though project seems to stall.

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

No branches or pull requests

2 participants