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.

Sign up for GitHub

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

Jump to bottom

wait for jmx up does not work #8183

Closed
soyacz opened this issue Jul 29, 2024 · 1 comment · Fixed by #8190
Closed

wait for jmx up does not work #8183

soyacz opened this issue Jul 29, 2024 · 1 comment · Fixed by #8190
Assignees
@soyacz

Comments

@soyacz
Copy link
Contributor

soyacz commented Jul 29, 2024

#8046 introduced regression for waiting JMX port up: verification is fixed to self.cql_address which is not valid for JMX.

Test: artifacts-ubuntu2204-nonroot-test
Test id: 47773f63-db39-43f3-ae65-34d1b865aeed
Test name: enterprise-2024.2/artifacts-offline-install/artifacts-ubuntu2204-nonroot-test
Test config file(s):

Logs and commands
  • Restore Monitor Stack command: $ hydra investigate show-monitor 47773f63-db39-43f3-ae65-34d1b865aeed
  • Restore monitor on AWS instance using Jenkins job
  • Show all stored logs command: $ hydra investigate show-logs 47773f63-db39-43f3-ae65-34d1b865aeed

Logs:

Jenkins job URL
Argus
@soyacz
Copy link
Contributor Author

soyacz commented Jul 29, 2024

can't be easily fixed with setting host to localhost as won't work with current approach for port open verification (it is verified from sct_runner which has no access to 'localhost' which jmx port runs on)

soyacz added a commit to soyacz/scylla-cluster-tests that referenced this issue Jul 29, 2024
@soyacz
fix(jmx): fix wait for jmx up
a89912b 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: scylladb#8183
@soyacz soyacz mentioned this issue Jul 29, 2024
Merged
3 tasks
soyacz added a commit to soyacz/scylla-cluster-tests that referenced this issue Jul 29, 2024
@soyacz
fix(jmx): fix wait for jmx up
8cdb9da 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: scylladb#8183
@fruch fruch closed this as completed in abc641b Jul 30, 2024
mergify bot pushed a commit that referenced this issue Jul 30, 2024
@soyacz @mergify
fix(jmx): fix wait for jmx up
027ac43 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
mergify bot pushed a commit that referenced this issue Jul 30, 2024
@soyacz @mergify
fix(jmx): fix wait for jmx up
2d490ac 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
mergify bot pushed a commit that referenced this issue Jul 30, 2024
@soyacz @mergify
fix(jmx): fix wait for jmx up
6751fd1 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)

# Conflicts:
#	sdcm/cluster.py
This was referenced Jul 30, 2024
Merged
Merged
Merged
mergify bot pushed a commit that referenced this issue Jul 30, 2024
@soyacz @mergify
fix(jmx): fix wait for jmx up
4a6d36d 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
mergify bot pushed a commit that referenced this issue Jul 30, 2024
@soyacz @mergify
fix(jmx): fix wait for jmx up
d0472e3 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
This was referenced Jul 30, 2024
Merged
Merged
fruch pushed a commit that referenced this issue Jul 30, 2024
@soyacz @fruch
fix(jmx): fix wait for jmx up
21b25c5 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
fruch pushed a commit that referenced this issue Jul 30, 2024
@soyacz @fruch
fix(jmx): fix wait for jmx up
3b5d8fe 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
fruch pushed a commit that referenced this issue Jul 30, 2024
@soyacz @fruch
fix(jmx): fix wait for jmx up
9ae2b82 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
fruch pushed a commit that referenced this issue Jul 30, 2024
@soyacz @fruch
fix(jmx): fix wait for jmx up
e063908 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
soyacz added a commit to soyacz/scylla-cluster-tests that referenced this issue Jul 31, 2024
@soyacz
fix(jmx): fix wait for jmx up
ce356ab 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: scylladb#8183
(cherry picked from commit abc641b)
soyacz added a commit that referenced this issue Jul 31, 2024
@soyacz
fix(jmx): fix wait for jmx up
d70eead 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: #8183
(cherry picked from commit abc641b)
Lakshmipathi pushed a commit to Lakshmipathi/scylla-cluster-tests that referenced this issue Aug 21, 2024
@soyacz
fix(jmx): fix wait for jmx up
dc69d8c 
As we now verify ports being open from sct runner, we cannot use it for
jmx as it listens on localhost.

Fix by verifying JMX being up by verification of log message in
scylla-jmx service status.

fixes: scylladb#8183
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@soyacz soyacz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(jmx): fix wait for jmx up soyacz/scylla-cluster-tests
1 participant
@soyacz