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

AttributeError: 'NoneType' object has no attribute 'group' #6175

Closed
Jerry-Fix opened this issue Jul 16, 2013 · 3 comments · Fixed by #6181
Closed

AttributeError: 'NoneType' object has no attribute 'group' #6175

Jerry-Fix opened this issue Jul 16, 2013 · 3 comments · Fixed by #6181
Assignees
Labels
Bug broken, incorrect, or confusing behavior help-wanted Community help is needed to resolve this
Milestone

Comments

@Jerry-Fix
Copy link

I use 0.15.3 for both minion and master, and I want to do some benchmark, so I install sysbench(0.4.12-5.el6) in minion. When I exec in the master it says:

 # salt 'xxx' sysbench.cpu
xxx:
    Traceback (most recent call last):
      File "/usr/lib/python2.6/site-packages/salt/minion.py", line 443, in _thread_return
        ret['return'] = func(*args, **kwargs)
      File "/usr/lib/python2.6/site-packages/salt/modules/sysbench.py", line 78, in cpu
        ret_val[key] = _parser(result)
      File "/usr/lib/python2.6/site-packages/salt/modules/sysbench.py", line 37, in _parser
        total_execution = re.search(_total_execution, result).group(1)
    AttributeError: 'NoneType' object has no attribute 'group'
@basepi
Copy link
Contributor

basepi commented Jul 16, 2013

Thanks for the report. This should be pretty easy to fix.

terminalmage added a commit to terminalmage/salt that referenced this issue Jul 16, 2013
This fixes saltstack#6175 by altering the regex for total execution time.
Additionally, this commit incudes a spelling change, and a modification
to the dicts returned by the _parser() helper function. There is no
reason why the dictionary keys should have trailing whitespace.
@ghost ghost assigned terminalmage Jul 16, 2013
terminalmage added a commit that referenced this issue Jul 16, 2013
This fixes #6175 by altering the regex for total execution time.
Additionally, this commit incudes a spelling change, and a modification
to the dicts returned by the _parser() helper function. There is no
reason why the dictionary keys should have trailing whitespace.
@brianthelion
Copy link

This appears to be rearing its ugly head again in Oxygen with sysbench version 1.0.11.

@terminalmage
Copy link
Contributor

Rather than commenting in an issue which has remained closed for over 5 years, please open a new issue and link to this one. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior help-wanted Community help is needed to resolve this
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants