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

pylint and livestatus broker #549

Closed
vaxvms opened this issue Aug 28, 2012 · 4 comments
Closed

pylint and livestatus broker #549

vaxvms opened this issue Aug 28, 2012 · 4 comments

Comments

@vaxvms
Copy link
Collaborator

vaxvms commented Aug 28, 2012

While looking at pylint output for issue #373, i've found these 2 errors in livestatus :

  • The easy one, we have to remove the columns method, don't we ?
    shinken/modules/livestatus_broker/livestatus_query_metainfo.py:304: [E, LiveStatusQueryMetainfo.columns] An attribute affected in shinken.modules.livestatus_broker.livestatus_query_metainfo line 159 hide this method
  • The not too easy one. It's looks like livestatus haven't been updated with new filters methods
    shinken/modules/livestatus_broker/livestatus_wait_query.py:179: [E, LiveStatusWaitQuery.launch_query] No value passed for parameter 'cs' in function call
@ghost ghost assigned lausser Aug 28, 2012
lausser added a commit that referenced this issue Aug 30, 2012
@naparuba
Copy link
Contributor

This don't seems to be so blocking, so will wait 1.6.

@Frescha
Copy link
Collaborator

Frescha commented Dec 12, 2013

any news?

@naparuba
Copy link
Contributor

naparuba commented Jan 2, 2014

Should be moved on the ls module repo please :)

@Seb-Solon
Copy link
Contributor

Moved to shinken-monitoring/mod-livestatus#32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants