Bug #3517: An error is displayed in the WebUI and the logs when removing a node because of an inefficient current action - Rudder - Issue Tracker

Custom queries

8.1 beta
Bugs triage
My TR
Need technical review
Open bugs (all)
Open bugs (UI - UX)
Open issues 8.0
Open UX defects (all)
Opened by users

Actions

Copy link

Bug #3517

closed

An error is displayed in the WebUI and the logs when removing a node because of an inefficient current action

Added by Nicolas PERRON almost 11 years ago. Updated over 10 years ago.

Status:

Released

Priority:

Assignee:

François ARMAND

Category:

Web - Nodes & inventories

Target version:

2.4.7

Pull Request:

https://github.com/Normation/rudder/p...

Severity:

UX impact:

User visibility:

Effort required:

Priority:

Name check:

Fix check:

Regression:

Description

Removing a Node which has not resolvable name lead to display errors like in the picture above:

Files

BugDeleteNodeWithoutResolveName.png (52.8 KB) BugDeleteNodeWithoutResolveName.png

Nicolas PERRON, 2013-04-24 11:26

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Nicolas PERRON almost 11 years ago

And in /var/log/rudder/webapp/2013_04_24.stderrout.log:

[...]
11:17:26.683 [qtp760805843-76] DEBUG com.normation.rudder.services.servers.RemoveNodeServiceImpl - Trying to clear all Nodes Configuration
11:17:26.685 [qtp760805843-76] DEBUG com.normation.rudder.services.servers.NodeConfigurationServiceImpl - Concurrent acces for method getStackTrace
11:17:26.778 [qtp760805843-76] ERROR net.liftweb.http.LiftRules - Exception being returned to browser when processing Req(List(F1163938454405HUZJAC), Map(F1163938454405HUZJAC -> List(true)), ParsePath(List(ajax_request, F1163938454370QOYXHH, index),,true,true), /rudder, PostRequest, Full(application/x-www-form-urlencoded; charset=UTF-8))
java.nio.channels.OverlappingFileLockException: null
    at sun.nio.ch.FileChannelImpl$SharedFileLockTable.checkList(FileChannelImpl.java:1166) ~[na:1.6.0_29]
    at sun.nio.ch.FileChannelImpl$SharedFileLockTable.add(FileChannelImpl.java:1068) ~[na:1.6.0_29]
    at sun.nio.ch.FileChannelImpl.tryLock(FileChannelImpl.java:868) ~[na:1.6.0_29]
    at java.nio.channels.FileChannel.tryLock(FileChannel.java:962) ~[na:1.6.0_29]
    at com.normation.rudder.services.servers.NodeConfigurationServiceImpl.inLock(NodeConfigurationServiceImpl.scala:105) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.NodeConfigurationServiceImpl.deleteAllNodeConfigurations(NodeConfigurationServiceImpl.scala:385) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl.com$normation$rudder$services$servers$RemoveNodeServiceImpl$$deleteAllNodesConfiguration(RemoveNodeService.scala:130) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$com$normation$rudder$services$servers$RemoveNodeServiceImpl$$atomicDelete$1$$anonfun$apply$4.apply(RemoveNodeService.scala:103) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$com$normation$rudder$services$servers$RemoveNodeServiceImpl$$atomicDelete$1$$anonfun$apply$4.apply(RemoveNodeService.scala:102) ~[rudder-core-2.5.2.jar:na]
    at net.liftweb.common.Full.flatMap(Box.scala:493) ~[lift-common_2.9.1-2.4.jar:2.4]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$com$normation$rudder$services$servers$RemoveNodeServiceImpl$$atomicDelete$1.apply(RemoveNodeService.scala:102) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$com$normation$rudder$services$servers$RemoveNodeServiceImpl$$atomicDelete$1.apply(RemoveNodeService.scala:101) ~[rudder-core-2.5.2.jar:na]
    at net.liftweb.common.Full.flatMap(Box.scala:493) ~[lift-common_2.9.1-2.4.jar:2.4]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl.com$normation$rudder$services$servers$RemoveNodeServiceImpl$$atomicDelete(RemoveNodeService.scala:101) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$removeNode$2$$anonfun$apply$1.apply(RemoveNodeService.scala:75) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$removeNode$2$$anonfun$apply$1.apply(RemoveNodeService.scala:75) ~[rudder-core-2.5.2.jar:na]
    at com.normation.utils.ScalaLock$$anonfun$apply$1.apply(Lock.scala:45) ~[utils-2.5.2.jar:na]
    at com.normation.utils.ScalaLock$$anonfun$apply$1.apply(Lock.scala:45) ~[utils-2.5.2.jar:na]
    at com.normation.utils.ScalaLock$class.map(Lock.scala:35) ~[utils-2.5.2.jar:na]
    at com.normation.utils.ScalaLock$$anon$1.map(Lock.scala:58) ~[utils-2.5.2.jar:na]
    at com.normation.utils.ScalaLock$class.apply(Lock.scala:45) ~[utils-2.5.2.jar:na]
    at com.normation.utils.ScalaLock$$anon$1.apply(Lock.scala:58) ~[utils-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$removeNode$2.apply(RemoveNodeService.scala:75) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl$$anonfun$removeNode$2.apply(RemoveNodeService.scala:73) ~[rudder-core-2.5.2.jar:na]
    at net.liftweb.common.Full.flatMap(Box.scala:493) ~[lift-common_2.9.1-2.4.jar:2.4]
    at com.normation.rudder.services.servers.RemoveNodeServiceImpl.removeNode(RemoveNodeService.scala:73) ~[rudder-core-2.5.2.jar:na]
    at com.normation.rudder.web.services.DisplayNode$.com$normation$rudder$web$services$DisplayNode$$removeNode(DisplayNode.scala:708) ~[classes/:na]
    at com.normation.rudder.web.services.DisplayNode$$anonfun$93.apply(DisplayNode.scala:691) ~[classes/:na]
    at com.normation.rudder.web.services.DisplayNode$$anonfun$93.apply(DisplayNode.scala:691) ~[classes/:na]
    at net.liftweb.http.S$NFuncHolder$$anonfun$apply$5.apply(S.scala:190) ~[lift-webkit_2.9.1-2.4.jar:2.4]
    at net.liftweb.http.S$NFuncHolder$$anonfun$apply$5.apply(S.scala:190) ~[lift-webkit_2.9.1-2.4.jar:2.4]
11:17:27.924 [qtp760805843-78] DEBUG com.normation.rudder.services.servers.RemoveNodeServiceImpl - Trying to remove node f2a84868-222d-4008-a5f1-3352271ed6dd from the LDAP
11:17:27.946 [qtp760805843-78] ERROR com.normation.rudder.web.services.DisplayNode - Could not remove node f2a84868-222d-4008-a5f1-3352271ed6dd from Rudder <- Node with ID 'f2a84868-222d-4008-a5f1-3352271ed6dd' was not found
[...]

Actions

Copy link

Updated by Nicolas PERRON almost 11 years ago

Target version changed from 2.5.3 to 2.5.4

Actions

Copy link

Updated by Nicolas CHARLES almost 11 years ago

Nicolas

I can reproduce this issue; I can't see how the resolvability of the hostname could influence anything

The stacktrace seems to point to a locking problem... do you have more info on the configuration ?

Actions

Copy link

Updated by Nicolas PERRON almost 11 years ago

I'm not sure about the link between the resolvability of the hostname and the issue.

This server had several nodes to manage then I'd added a new node which came from vagrant on my own machine. This node could contact the server but its hostname was not known into the server. It's the only obvious difference I've seen between this node and the others. It could be wrong.

Actions

Copy link

Updated by François ARMAND almost 11 years ago

That error seems to also have been reported by dnns:

16:53 < dnns> i deleted a node from rudder (2.5 that is)
16:54 < dnns> and it gave an error
16:54 < dnns> but.. the node is deleted
16:55 < fanf> dnns: partial success ?
16:55 < dnns> i don't know, the node is deleted, but there was an error
16:55 < fanf> dnns: more seriously, did you keep the error ?
16:56 < dnns> no but the error gave no info at all
16:56 < fanf> :/
17:04 < Vince_MacBuche> There is nothing in /var/log/rudder/webapp/2013_05_23.webapp.log which could be 
                        linked to that error ?
17:07 < dnns> no can't find any
17:16 < ncharles_> dnns: is it like http://www.rudder-project.org/redmine/issues/3420 ?
17:17 < dnns> yes
17:18 < dnns> but i can re-generate rules still
17:20 < ncharles_> so more like http://www.rudder-project.org/redmine/issues/3517 ?
17:34 < dnns> ncharles_: yes!

Actions

Copy link

Updated by Nicolas PERRON almost 11 years ago

Target version changed from 2.5.4 to 2.5.5

Actions

Copy link

Updated by Vincent MEMBRÉ almost 11 years ago

Category set to Web - Nodes & inventories
Status changed from New to In progress
Assignee set to Vincent MEMBRÉ
Priority changed from N/A to 1

This does not seem to be an hostname error, In fact an exception is launched, when trying to acquire a Lock.

Which fails the remove node (after removing it from groups and directives)

It should wait for the lock file tu be acquired.

Actions

Copy link

Updated by Vincent MEMBRÉ almost 11 years ago

Status changed from In progress to Pending technical review
Assignee changed from Vincent MEMBRÉ to François ARMAND
Target version changed from 2.5.5 to 2.4.7
Pull Request set to https://github.com/Normation/rudder/pull/228

This happens on 2.4 too, The exception is not handled too.

The pull request here is a simple workaround, a far more longer and deeper change should be done about node configurations are handled in Rudder.

Pull request : https://github.com/Normation/rudder/pull/228

Actions

Copy link

Updated by Vincent MEMBRÉ almost 11 years ago

Status changed from Pending technical review to Pending release
% Done changed from 0 to 100

Applied in changeset c4d5dfc0ae6c67051705f2ec6ae9234139fac7a7.

Actions

Copy link

#10

Updated by Anonymous almost 11 years ago

Applied in changeset af3144363338bcf094df90d965f0dae86a312eed.

Actions

Copy link

#11

Updated by Nicolas PERRON over 10 years ago

Subject changed from Removing a node which is hostname is not resolvable lead to an error in the WebUI and the logs but remove the node to An error is displayed in the WebUI and the logs when removing a node because of an inefficient current action

Actions

Copy link

#12

Updated by Nicolas PERRON over 10 years ago

Status changed from Pending release to Released

Actions

Copy link

#13

Updated by Nicolas PERRON over 10 years ago

This bug has been fixed in Rudder 2.4.7, which was released today.
Check out:

The release announcement: http://www.rudder-project.org/pipermail/rudder-announce/2013-July/000041.html
The full ChangeLog: http://www.rudder-project.org/foswiki/bin/view/System/Documentation:ChangeLog24
Download information: http://www.rudder-project.org/foswiki/Download/

Actions

Copy link

Also available in: Atom PDF

Project

General

Profile

Rudder

Custom queries

Bug #3517

An error is displayed in the WebUI and the logs when removing a node because of an inefficient current action

Updated by Nicolas PERRON almost 11 years ago

Updated by Nicolas PERRON almost 11 years ago

Updated by Nicolas CHARLES almost 11 years ago

Updated by Nicolas PERRON almost 11 years ago

Updated by François ARMAND almost 11 years ago

Updated by Nicolas PERRON almost 11 years ago

Updated by Vincent MEMBRÉ almost 11 years ago

Updated by Vincent MEMBRÉ almost 11 years ago

Updated by Vincent MEMBRÉ almost 11 years ago

Updated by Anonymous almost 11 years ago

Updated by Nicolas PERRON over 10 years ago

Updated by Nicolas PERRON over 10 years ago

Updated by Nicolas PERRON over 10 years ago