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

'NoneType' object has no attribute 'endswith' #15

Closed
kazikfriedrich opened this issue Sep 13, 2017 · 4 comments
Closed

'NoneType' object has no attribute 'endswith' #15

kazikfriedrich opened this issue Sep 13, 2017 · 4 comments
Labels

Comments

@kazikfriedrich
Copy link

Here's the output I'm getting on Fedora 25:

Python 2.7.13

~/out-of-memory $ python oom-investigate.py 
----------------------------------------
      _____ _____ _____ 
     |     |     |     |
     |  |  |  |  | | | |
     |_____|_____|_|_|_|
     Out Of Memory Analyser

Disclaimer:
If system OOMs too viciously, there may be nothing logged!
Do NOT take this script as FACT, investigate further
----------------------------------------
Unsupported OS

Error:
'NoneType' object has no attribute 'endswith'

----------------------------------------
@LukeShirnia
Copy link
Owner

Unsupported OS...but ill take a look at it shortly

@LukeShirnia
Copy link
Owner

LukeShirnia commented Aug 13, 2018

Issue resolved in latest merge.
See the following lines for resolution:
https://github.com/LukeShirnia/out-of-memory/blob/master/oom-investigate.py#L436-L448
and
https://github.com/LukeShirnia/out-of-memory/blob/master/oom-investigate.py#L549-L552
Please let me know if you are still having issues.
Ill close this on Friday if no response.

@kazikfriedrich
Copy link
Author

I don't have the means to test with Fedora 25 any more. Fedora 28 moved to journald and by default doesn't write anything to /var/log/messages, however I can imagine that adding support for journald systems is a major piece of work. Thanks for the fix anyway!

@LukeShirnia
Copy link
Owner

Apologies for the delay in fixing the issue. Thanks for the update.
I have opened up #17 for journalctl compatibility.
I should be more active in issue fixes etc now.

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

No branches or pull requests

2 participants