Fix for older linux kernels and blocking read of /proc/* files in virtualization plugin

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Unknown Unknown
  • Resolution: Unresolved
  • Affects Version/s: 6.14.0
  • Fix Version/s: None
  • Component/s: Kernel
  • Labels:
    None
  • Environment:

    Red Hat Enterprise Linux Server release 5.6 (Tikanga)

    1. uname -r
      2.6.18-238.28.1.el5

Description

Due to a bug in kernel prior 2.6.30, blocking read of /proc/* files in ruby cause shef to hang forever when loading plugins. See https://github.com/newrelic/rpm/pull/47, http://tickets.opscode.com/browse/OHAI-196

Pull request: https://github.com/opscode/ohai/pull/78

Issue Links

relates to

Bug - A problem which impairs or prevents the functions of the product. OHAI-196 IO.select hangs forever reading device files on older linux kernels

  • Blocker - Blocks development and/or testing work, production could not run.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Dmytro Ilchenko added a comment -

Updated patch in pull request, please verify and merge.

Show
Dmytro Ilchenko added a comment - Updated patch in pull request, please verify and merge.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

Dmytro, in the future hit 'resolve' when you have a patch and we'll review the code at our regular code review meetings. I've got it this time.

Show
Bryan McLellan [Opscode] added a comment - Dmytro, in the future hit 'resolve' when you have a patch and we'll review the code at our regular code review meetings. I've got it this time.
Hide
Permalink
Dmytro Ilchenko added a comment -

Got it. Thanks!

Show
Dmytro Ilchenko added a comment - Got it. Thanks!
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

The function in this commit is pretty much a direct copy of Jon Guymon's code in newrelic. I realize it is a small chunk of code, but we really should get Jon Guymon to complete a CLA and agree to including his code in the project.

https://github.com/idtest/ohai/commit/2e8c33cfbc9a364180fa456aa2ef65923649030b#diff-0
https://github.com/newrelic/rpm/commit/f670baef01d0116bf021d5d9be35025ddd596d78#diff-0

I've sent Jon and Dmytro an email to see if we can resolve this quickly.

Show
Bryan McLellan [Opscode] added a comment - The function in this commit is pretty much a direct copy of Jon Guymon's code in newrelic. I realize it is a small chunk of code, but we really should get Jon Guymon to complete a CLA and agree to including his code in the project. https://github.com/idtest/ohai/commit/2e8c33cfbc9a364180fa456aa2ef65923649030b#diff-0 https://github.com/newrelic/rpm/commit/f670baef01d0116bf021d5d9be35025ddd596d78#diff-0 I've sent Jon and Dmytro an email to see if we can resolve this quickly.
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

I did not receive a reply to my email to Jon Guymon and Dmytro Ilchenko.

It looks like rpm is not an open-source project and it seems unlikely at this point they're going to step up.

Thus, I'm reopening this until a patch is provided that does not use their copyrighted code.

Show
Bryan McLellan [Opscode] added a comment - I did not receive a reply to my email to Jon Guymon and Dmytro Ilchenko. It looks like rpm is not an open-source project and it seems unlikely at this point they're going to step up. Thus, I'm reopening this until a patch is provided that does not use their copyrighted code.

People

  • Assignee:
    Bryan McLellan [Opscode]
    Reporter:
    Dmytro Ilchenko
Vote (0)
Watch (1)

Dates

  • Created:
    Updated: