Project

General

Profile

Actions

Bug #5432

closed

Deleted Technique from technique editor are still present in directive tree view

Bug #5432: Deleted Technique from technique editor are still present in directive tree view

Added by John Robert Mendoza over 11 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
3
Assignee:
-
Category:
Web - Technique editor
Target version:
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Getting started - demo | first install | level 1 Techniques
Effort required:
Medium
Priority:
0
Name check:
Fix check:
Regression:

Description

The reference technique library page doesn't get updated and still shows the custom technique I've deleted from ncf editor. Here's how to reproduce the bug.

1. Create a new technique using the ncf technique editor
2. Check the technique library management page and the newly created technique will be shown under the "Meta Technique" section of the Reference Technique Library tree.
3. Head back to the technique editor and delete your custom technique.
4. Refresh the library management page. Observe that the custom technique is still displayed.


Related issues 4 (0 open4 closed)

Has duplicate Rudder - Bug #5829: Deleted Technique from NCF builder are still present in Rudder Technique libraryRejectedActions
Has duplicate Rudder - Bug #7202: Technique Editor and Rudder Techniques Library isn't coherentRejectedActions
Has duplicate Rudder - Bug #13813: User's technique don't disapear after deletingRejectedVincent MEMBRÉActions
Has duplicate Rudder - Bug #6641: When we remove a technique from the editor, it is still available for new directive creationRejectedActions

Updated by John Robert Mendoza over 11 years ago Actions #1

Forgot to mention the target version for this. It's 2.11.2-release-1

Updated by Vincent MEMBRÉ over 11 years ago Actions #2

  • Assignee set to Vincent MEMBRÉ
  • Target version set to 2.11.3

Thank you for reporting John.

In fact the technique is not automatically deleted from Rudder. I will add a way to automate this.

for now, you can manually delete the technique:

cd /var/rudder/configuration-repository
# A 'git status' would show that cfengine files were deleted, but it was not commited, so rm the files
git rm -r ncf/50_techniques/my_technique techniques/ncf_techniques/my_technique
git commit -m "delete my_technique" 
# Reload technique library
curl -s -f -k "https://localhost/rudder/api/techniqueLibrary/reload" 

I will add ncf-hooks that delete the files automatically. and reload Rudder technique library

Updated by Vincent MEMBRÉ over 11 years ago Actions #3

Before running those commands, please delete all Directive that were created based on that Technique

Updated by Matthieu CERDA about 11 years ago Actions #4

  • Target version changed from 2.11.3 to 2.11.4

Updated by Vincent MEMBRÉ about 11 years ago Actions #5

  • Target version changed from 2.11.4 to 2.11.5

Updated by Vincent MEMBRÉ almost 11 years ago Actions #6

  • Target version changed from 2.11.5 to 2.11.6

Updated by Vincent MEMBRÉ almost 11 years ago Actions #7

  • Target version changed from 2.11.6 to 2.11.7

Updated by Vincent MEMBRÉ almost 11 years ago Actions #8

  • Target version changed from 2.11.7 to 2.11.8

Updated by Vincent MEMBRÉ almost 11 years ago Actions #9

  • Target version changed from 2.11.8 to 2.11.9

Updated by Vincent MEMBRÉ over 10 years ago Actions #10

  • Target version changed from 2.11.9 to 2.11.10

Updated by Vincent MEMBRÉ over 10 years ago Actions #11

  • Target version changed from 2.11.10 to 2.11.11

Updated by Benoît PECCATTE over 10 years ago Actions #12

  • Category set to Web - Technique editor

Updated by Vincent MEMBRÉ over 10 years ago Actions #13

  • Target version changed from 2.11.11 to 2.11.12

Updated by Vincent MEMBRÉ over 10 years ago Actions #14

  • Target version changed from 2.11.12 to 2.11.13

Updated by Vincent MEMBRÉ over 10 years ago Actions #15

  • Target version changed from 2.11.13 to 2.11.14

Updated by Vincent MEMBRÉ about 10 years ago Actions #16

  • Target version changed from 2.11.14 to 2.11.15

Updated by Vincent MEMBRÉ about 10 years ago Actions #17

  • Target version changed from 2.11.15 to 2.11.16

Updated by Vincent MEMBRÉ about 10 years ago Actions #18

  • Target version changed from 2.11.16 to 2.11.17

Updated by Vincent MEMBRÉ almost 10 years ago Actions #19

  • Target version changed from 2.11.17 to 2.11.18

Updated by Vincent MEMBRÉ almost 10 years ago Actions #20

  • Target version changed from 2.11.18 to 2.11.19

Updated by Vincent MEMBRÉ almost 10 years ago Actions #21

  • Target version changed from 2.11.19 to 2.11.20

Updated by Vincent MEMBRÉ over 9 years ago Actions #22

  • Target version changed from 2.11.20 to 2.11.21

Updated by Vincent MEMBRÉ over 9 years ago Actions #23

  • Target version changed from 2.11.21 to 2.11.22

Updated by Vincent MEMBRÉ over 9 years ago Actions #24

  • Target version changed from 2.11.22 to 2.11.23

Updated by Vincent MEMBRÉ over 9 years ago Actions #25

  • Target version changed from 2.11.23 to 2.11.24

Updated by Vincent MEMBRÉ over 9 years ago Actions #26

  • Target version changed from 2.11.24 to 308

Updated by Vincent MEMBRÉ about 9 years ago Actions #27

  • Target version changed from 308 to 3.1.14

Updated by Vincent MEMBRÉ about 9 years ago Actions #28

  • Target version changed from 3.1.14 to 3.1.15

Updated by Vincent MEMBRÉ about 9 years ago Actions #29

  • Target version changed from 3.1.15 to 3.1.16

Updated by Vincent MEMBRÉ about 9 years ago Actions #30

  • Target version changed from 3.1.16 to 3.1.17

Updated by Vincent MEMBRÉ about 9 years ago Actions #31

  • Target version changed from 3.1.17 to 3.1.18

Updated by Vincent MEMBRÉ almost 9 years ago Actions #32

  • Target version changed from 3.1.18 to 3.1.19

Updated by François ARMAND over 8 years ago Actions #33

  • User visibility set to Getting started - demo | first install | level 1 Techniques
  • Effort required set to Small
  • Priority set to 0

This is still happening.
When a directive created from the deleted technique is configured on a node, running "rudder agent run" leads to:

Rudder agent 3.1.19.rc1.git201704040733 (CFEngine Core 3.6.5)
Node uuid: root
2017-04-04T12:04:20+0200    error: Bundle 'vim_install' listed in the bundlesequence is not a defined bundle
2017-04-04T12:04:20+0200    error: Fatal CFEngine error: Errors in promise bundles: could not verify bundlesequence
error    Rudder agent was interrupted during execution by a fatal error
         Run with -i to see log messages.

## Summary #####################################################################
error:        1
execution time: 0.65s
################################################################################

But nothing in the UI let you thing that there a problem (no error in generation, after delete, etc). And you can even create again new directive from the deleted technique, without any error!
Moreover, the compliance is only in "no answer", so we need to go the node to understand the problem.

You have to delete all directives from that technique by hand to make rudder agent works again.

Updated by François ARMAND over 8 years ago Actions #34

  • Severity set to Major - prevents use of part of Rudder | no simple workaround

I'm setting the severity to major, because of the potential silent breaking of all nodes with the directive, and a not so easy diagnosis for the problem.

Updated by François ARMAND over 8 years ago Actions #35

  • Has duplicate Bug #5829: Deleted Technique from NCF builder are still present in Rudder Technique library added

Updated by François ARMAND over 8 years ago Actions #36

  • Subject changed from Reference Technique Library Tree after deleting custom NCF directive to Deleted Technique from NCF builder are still present in Rudder Technique library

Updated by Vincent MEMBRÉ over 8 years ago Actions #37

  • Target version changed from 3.1.19 to 3.1.20

Updated by Jonathan CLARKE over 8 years ago Actions #38

  • Assignee deleted (Vincent MEMBRÉ)

Updated by Vincent MEMBRÉ over 8 years ago Actions #39

  • Target version changed from 3.1.20 to 3.1.21

Updated by François ARMAND over 8 years ago Actions #40

  • Severity changed from Major - prevents use of part of Rudder | no simple workaround to Minor - inconvenience | misleading | easy workaround
  • Priority changed from 0 to 36

This must be tested again. In 4.1, it seems that the directive is disabled (or at least not accessible in the UI) when the ncf technique is deleted, and so the policy generation is not broken. (and so, Minor in place of major).

In all cases, we must add some warning regarding the usage of the technique before deleting it.

Updated by Vincent MEMBRÉ over 8 years ago Actions #41

  • Target version changed from 3.1.21 to 3.1.22

Updated by Benoît PECCATTE over 8 years ago Actions #42

  • Priority changed from 36 to 45

Updated by Vincent MEMBRÉ over 8 years ago Actions #43

  • Target version changed from 3.1.22 to 3.1.23

Updated by Vincent MEMBRÉ over 8 years ago Actions #44

  • Target version changed from 3.1.23 to 3.1.24

Updated by Vincent MEMBRÉ about 8 years ago Actions #45

  • Target version changed from 3.1.24 to 3.1.25
  • Priority changed from 45 to 46

Updated by Benoît PECCATTE about 8 years ago Actions #46

  • Priority changed from 46 to 56

Updated by Vincent MEMBRÉ almost 8 years ago Actions #47

  • Target version changed from 3.1.25 to 387
  • Priority changed from 56 to 57

Updated by Vincent MEMBRÉ almost 8 years ago Actions #48

  • Target version changed from 387 to 4.1.10

Updated by Vincent MEMBRÉ almost 8 years ago Actions #49

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 57 to 58

Updated by Vincent MEMBRÉ over 7 years ago Actions #50

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 58 to 59

Updated by Vincent MEMBRÉ over 7 years ago Actions #51

  • Target version changed from 4.1.12 to 4.1.13

Updated by Vincent MEMBRÉ over 7 years ago Actions #52

  • Effort required changed from Small to Medium
  • Priority changed from 59 to 31

Updated by Vincent MEMBRÉ over 7 years ago Actions #53

  • Target version changed from 4.1.13 to 4.1.14

Updated by Benoît PECCATTE over 7 years ago Actions #54

  • Target version changed from 4.1.14 to 4.1.15

Updated by Vincent MEMBRÉ about 7 years ago Actions #55

  • Target version changed from 4.1.15 to 4.1.16

Updated by Vincent MEMBRÉ about 7 years ago Actions #56

  • Target version changed from 4.1.16 to 4.1.17
  • Priority changed from 31 to 32

Updated by François ARMAND about 7 years ago Actions #57

  • Has duplicate Bug #7202: Technique Editor and Rudder Techniques Library isn't coherent added

Updated by François ARMAND about 7 years ago Actions #58

  • Has duplicate Bug #13813: User's technique don't disapear after deleting added

Updated by François ARMAND about 7 years ago Actions #59

  • Subject changed from Deleted Technique from NCF builder are still present in Rudder Technique library to Deleted Technique from technique editor are still present in directive tree view
  • Priority changed from 32 to 0

Updated by François ARMAND about 7 years ago Actions #60

  • Related to Bug #6641: When we remove a technique from the editor, it is still available for new directive creation added

Updated by François ARMAND about 7 years ago Actions #61

  • Related to deleted (Bug #6641: When we remove a technique from the editor, it is still available for new directive creation)

Updated by François ARMAND about 7 years ago Actions #62

  • Has duplicate Bug #6641: When we remove a technique from the editor, it is still available for new directive creation added

Updated by Vincent MEMBRÉ about 7 years ago Actions #63

  • Target version changed from 4.1.17 to 4.1.18

Updated by Vincent MEMBRÉ almost 7 years ago Actions #64

  • Target version changed from 4.1.18 to 4.1.19

Updated by Alexis Mousset almost 7 years ago Actions #65

  • Target version changed from 4.1.19 to 4.1.20

Updated by Alexis Mousset over 6 years ago Actions #66

  • Target version changed from 4.1.20 to 588

Updated by Alexis Mousset over 6 years ago Actions #67

  • Target version changed from 588 to 5.0.13

Updated by Vincent MEMBRÉ over 6 years ago Actions #68

  • Target version changed from 5.0.13 to 5.0.14

Updated by Vincent MEMBRÉ about 6 years ago Actions #69

  • Target version changed from 5.0.14 to 5.0.15

Updated by Vincent MEMBRÉ about 6 years ago Actions #70

  • Target version changed from 5.0.15 to 5.0.16

Updated by Alexis Mousset almost 6 years ago Actions #71

  • Target version changed from 5.0.16 to 5.0.17

Updated by Vincent MEMBRÉ over 5 years ago Actions #72

  • Target version changed from 5.0.17 to 5.0.18

Updated by Vincent MEMBRÉ over 5 years ago Actions #73

  • Target version changed from 5.0.18 to 5.0.19

Updated by Vincent MEMBRÉ over 5 years ago Actions #74

  • Target version changed from 5.0.19 to 5.0.20

Updated by Vincent MEMBRÉ about 5 years ago Actions #75

  • Target version changed from 5.0.20 to 797

Updated by Benoît PECCATTE over 4 years ago Actions #76

  • Target version changed from 797 to 6.1.14

Updated by Vincent MEMBRÉ over 4 years ago Actions #77

  • Target version changed from 6.1.14 to 6.1.15

Updated by Vincent MEMBRÉ over 4 years ago Actions #78

  • Target version changed from 6.1.15 to 6.1.16

Updated by Vincent MEMBRÉ over 4 years ago Actions #79

  • Target version changed from 6.1.16 to 6.1.17

Updated by Vincent MEMBRÉ about 4 years ago Actions #80

  • Target version changed from 6.1.17 to 6.1.18

Updated by Vincent MEMBRÉ almost 4 years ago Actions #81

  • Target version changed from 6.1.18 to 6.1.19

Updated by François ARMAND almost 4 years ago Actions #82

  • Status changed from New to Resolved

As of 7.0 at least, it is not the case anymore.

Actions

Also available in: PDF Atom