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

scylla with custom release string fails #7153

Closed
avikivity opened this issue Jan 27, 2024 · 2 comments
Closed

scylla with custom release string fails #7153

avikivity opened this issue Jan 27, 2024 · 2 comments
Assignees

Comments

@avikivity
Copy link
Member

20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR > Exception in setUp. Will call tearDown < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR > Exception in setUp. Will call tearDown
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR > Traceback (most recent call last):
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >   File "/home/ubuntu/scylla-cluster-tests/sdcm/tester.py", line 161, in wrapper
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >     return method(*args, **kwargs)
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >   File "/home/ubuntu/scylla-cluster-tests/sdcm/utils/decorators.py", line 119, in inner
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >     res = func(*args, **kwargs)
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >   File "/home/ubuntu/scylla-cluster-tests/sdcm/tester.py", line 827, in setUp
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >     self.create_test_stats()
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >   File "/home/ubuntu/scylla-cluster-tests/sdcm/db_stats.py", line 657, in create_test_stats
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >     self._stats['versions'] = self.get_scylla_versions()
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >   File "/home/ubuntu/scylla-cluster-tests/sdcm/db_stats.py", line 581, in get_scylla_versions
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR >     versions['scylla-server']['build_mode'] = node.scylla_build_mode
20:54:21  < t:2024-01-27 18:54:20,932 f:tester.py       l:168  c:sdcm.tester          p:ERROR > KeyError: 'scylla-server'
20:54:21  < t:2024-01-27 18:54:20,934 f:file_logger.py  l:101  c:sdcm.sct_events.file_logger p:ERROR > 2024-01-27 18:54:20.931: 

I suspect the regex in get_scylla_versions is too tight.

@avikivity avikivity removed their assignment Jan 28, 2024
@fruch
Copy link
Contributor

fruch commented Mar 4, 2024

this code isn't safe regardless of which version string it is.

@avikivity can you share the version ? so whom would work fixing it can try it out ?

@fruch fruch self-assigned this May 5, 2024
@fruch
Copy link
Contributor

fruch commented Jul 25, 2024

this specific issue was handled in

f6e1627

it was a mistake, and the code wasn't safe enough, assuming optional things always happen.

@fruch fruch closed this as not planned Won't fix, can't repro, duplicate, stale Jul 25, 2024
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