Bug 798006
Summary: | Drift detection hangs on reading named pipes on filesystem | |||
---|---|---|---|---|
Product: | [JBoss] JBoss Operations Network | Reporter: | Mike Foley <mfoley> | |
Component: | Drift | Assignee: | Jay Shaughnessy <jshaughn> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Mike Foley <mfoley> | |
Severity: | high | Docs Contact: | ||
Priority: | unspecified | |||
Version: | JON 3.0.0 | CC: | ccrouch, hrupp, loleary, mmahoney, myarboro | |
Target Milestone: | ER01 | |||
Target Release: | JON 3.3.0 | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 880815 (view as bug list) | Environment: | ||
Last Closed: | 2014-12-11 14:00:05 UTC | Type: | --- | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | 880815 | |||
Bug Blocks: |
Description
Mike Foley
2012-02-27 19:38:32 UTC
this is an edge case that can be avoided with includes/excludes ... in the unlikely situation that someone wants to monitor a directory with these unusual file types preliminary discussion with developer indicated the fix could be a bit heavy-handed to open every file to see if it is actually readable (btw, trying to read this named pipe file is what causes drift to hang) re inclusion into JON 3.01 (we are past code freeze now, and in the endgame) -is it a regression? NO -is it a customer issue? NO -is it urgent? -is it a main line primary use-case? NO ergo, recommend a push of target release to jon 3.1 or later. documenting that i tested drift on directories with symbolic links ... and that worked. JON 3.1 medium priority, per triage meeting (crouch, foley, loleary) I think the solution to this, would be to change th following code in DriftDetector.doDirectoryScan(): if (!file.canRead()) { if (log.isDebugEnabled()) { log.debug("Skipping " + file.getPath() + " since we do not have read access."); } return; } to: if (!file.isFile()) { if (log.isDebugEnabled()) { log.debug("Skipping " + file.getPath() + " since it is not a regular file."); } return; } Since File.isFile() should only return true for regular files or symlinks to regular files, which I believe are the only things we care about. I'm assigning this to Jay, since he worked on the related bug 789454, so he can comment. commit 7e9ce1d0a3953248a7dcc66b2025447b1cf1a019 Author: Jay Shaughnessy <jshaughn> Date: Mon Apr 21 16:37:35 2014 -0400 Protect against drift detection on not "normal" files. Moving to ON_QA as available to test with brew build of DR01: https://brewweb.devel.redhat.com//buildinfo?buildID=373993 failed qe. the drift feature is broken in jon 3.3 |