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

test_metrics() from fabric8's test.sh fails with centos/openjdk-8-centos7 #62

Closed
vorburger opened this issue Feb 13, 2019 · 5 comments
Closed

Comments

@vorburger
Copy link
Contributor

For #51 I've played with a first test, and it failed when it tried to GET /metrics ...

I was just wondering if this was expected in this image?

@vorburger
Copy link
Contributor Author

oh... you use /jolokia instead of /metrics in this image, and run it on 8778 instead of 9779 huh?

But even after fixing that, the curl in test_metrics() in our test.sh still fails, now with curl: (52) Empty reply from server (instead of the initial curl: (56) Recv failure: Connection reset by peer due to wrong URL & port) ...

@jmtd
Copy link
Member

jmtd commented Feb 13, 2019

Is this testing against centos/openjdk-11-centos7 ? I'm unsure of the status of jolokia (and hawkular) in the 11-flavour images. There's at least these two known problems: https://issues.jboss.org/browse/CLOUD-3018 (⇒ jolokia/jolokia#394) and https://issues.jboss.org/browse/CLOUD-3054

@vorburger
Copy link
Contributor Author

Is this testing against centos/openjdk-11-centos7 ?

no, with a (locally built) centos/openjdk-8-centos7 .. (I haven't been able to build 11 yet, due to #59.)

do the tests here test the metrics, similar to test.sh does over in our other project? (but NB #63)

@jmtd
Copy link
Member

jmtd commented Feb 13, 2019

There appears to be very limited testing at the moment. we test jolokia made it into the image, and what version, and that we get the jolokia banner at startup, and not much else.

@jmtd jmtd changed the title /metrics ? test_metrics() from fabric8's test.sh fails with centos/openjdk-8-centos7 May 3, 2019
@jmtd
Copy link
Member

jmtd commented May 3, 2019

I think this is because test.sh assumes http but we default to https

If you wanted to confirm you could try starting the container with the environment variable AB_JOLOKIA_HTTPS=false

@jmtd jmtd closed this as completed May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants