Project

General

Profile

Actions

Bug #4392

closed

Can not delete custom Active techniques category

Added by Alex Tkachenko almost 11 years ago. Updated almost 11 years ago.

Status:
Released
Priority:
2
Category:
Web - Config management
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

This issue was first discovered in 2.8.1 and I kind hoped that it may be addressed in 2.9.2, but unfortunately it is still present.

If I create a custom category, it is added into the LDAP tree as far as I could tell with correct attributes (see below), but when I try to delete it via the web interface, it prints in red "Error when deleting the categoy" (misspelled too) and the category remains. The file /var/log/rudder/webapp/*stderrout.log shows the message:

[2014-01-17 10:38:50] ERROR com.normation.rudder.web.components.TechniqueCategoryEditForm - Error when deleting user lib category with ID ActiveTechniqueCategoryId(b8772d1f-201f-4d9c-b876-49b850ee03cd): The given entry is not of the expected ObjectClass 'techniqueCategory'. Entry details: Entry(dn='techniqueCategoryId=b8772d1f-201f-4d9c-876-49b850ee03cd,techniqueCategoryId=systemSettings,techniqueCategoryId=Active Techniques,ou=Rudder,cn=rudder-configuration', attributes={Attribute(name=techniqueCategoryId, values={'b8772d1f-201f-4d9c-b876-49b850ee03cd'})})

However, if I query the LDAP directly it shows that the object has the requested Object Class:

# /opt/rudder/bin/ldapsearch -LLL -H ldap://localhost -x -W  -D "cn=Manager,cn=rudder-configuration" -b "cn=rudder-configuration" "techniqueCategoryId=b8772d1f*" 
Enter LDAP Password: 
dn: techniqueCategoryId=b8772d1f-201f-4d9c-b876-49b850ee03cd,techniqueCategory
 Id=systemSettings,techniqueCategoryId=Active Techniques,ou=Rudder,cn=rudder-c
 onfiguration
techniqueCategoryId: b8772d1f-201f-4d9c-b876-49b850ee03cd
objectClass: techniqueCategory
objectClass: top
isSystem: FALSE
cn: test

BTW, is it possible to specify the TechniqueCategoryID during creation, so it uses some descriptive name instead of a long string of chars?

I am sorry for not providing any suggested fixes (I usually try to :) but getting through the scala code is out of my league at the moment :)

Actions #1

Updated by François ARMAND almost 11 years ago

  • Category set to Web - Config management
  • Status changed from New to 8
  • Assignee set to François ARMAND
  • Priority changed from N/A to 2
  • Target version set to 2.6.11

Hello Alex,

I'm looking to that.

For the ID, I fear we didn't thought of that option. Our idea was that IDs should have the least possiblities of overlapping (think for example to the use of several Rudder server, with information exchange between them like in dev/preprod/prod, etc), and so we used UUIDs.

The use of the technique name / description is not sufficient ? The problem is when you look in LDAP or in the git, is it that ?

Actions #2

Updated by Alex Tkachenko almost 11 years ago

I guess it would be fine to have an auto-generated ID if I don't have to go to LDAP to delete the category manually, as this is the only time when I have the hard time locating it :) Can not speak for git though, but so far I did not run into necessity to look into that directly.

Actions #3

Updated by François ARMAND almost 11 years ago

  • Status changed from 8 to Pending technical review
  • Assignee changed from François ARMAND to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/rudder/pull/432
Actions #4

Updated by François ARMAND almost 11 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100
Actions #5

Updated by Vincent MEMBRÉ almost 11 years ago

  • Subject changed from Can not delete custom Achtive techniques category to Can not delete custom Active techniques category
Actions #6

Updated by Vincent MEMBRÉ almost 11 years ago

  • Status changed from Pending release to Released

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

Actions

Also available in: Atom PDF