Project

General

Profile

Actions

Bug #15178

closed

Deleting a technique in the technique editor just after its creation fails

Added by Alexis Mousset over 5 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Very Small
Priority:
82
Name check:
Reviewed
Fix check:
Checked
Regression:

Description

It fails with:

Cause: [Errno 2] No such file or directory: '/var/rudder/configuration-repository/ncf/50_techniques/undefined'
Traceback (most recent call last):
  File "/usr/share/ncf/api/ncf_api_flask_app/views.py", line 176, in delete_technique
    ncf.delete_technique(bundle_name,path)
  File "/usr/share/ncf/tools/ncf.py", line 743, in delete_technique
    shutil.rmtree(filename)
  File "/usr/lib/python2.7/shutil.py", line 239, in rmtree
    onerror(os.listdir, path, sys.exc_info())
  File "/usr/lib/python2.7/shutil.py", line 237, in rmtree
    names = os.listdir(path)
OSError: [Errno 2] No such file or directory: '/var/rudder/configuration-repository/ncf/50_techniques/undefined'

It should just delete it in the browser instead


Related issues 1 (0 open1 closed)

Related to Rudder - User story #5482: Enable delete/clone button just after we create a cloneResolvedVincent MEMBRÉActions
Actions #1

Updated by Alexis Mousset over 5 years ago

  • Description updated (diff)
Actions #2

Updated by Alexis Mousset over 5 years ago

  • Related to User story #5482: Enable delete/clone button just after we create a clone added
Actions #3

Updated by Vincent MEMBRÉ over 5 years ago

  • Subject changed from Delete crashes when clicked on a new tehcnique not saved yet to In technique editor, deleting a new technique crashes

There should be no api call when deleting a new technique just delete it in UI

Actions #4

Updated by François ARMAND over 5 years ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Getting started - demo | first install | Technique editor and level 1 Techniques
  • Priority changed from 0 to 70
Actions #5

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.13 to 5.0.14
  • Priority changed from 70 to 69
Actions #6

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
  • Priority changed from 69 to 68
Actions #7

Updated by François ARMAND about 5 years ago

  • Effort required set to Very Small
  • Priority changed from 68 to 94
Actions #8

Updated by Raphael GAUTHIER about 5 years ago

  • Status changed from New to In progress
  • Assignee set to Raphael GAUTHIER
Actions #9

Updated by Raphael GAUTHIER about 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Raphael GAUTHIER to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/ncf/pull/1100
Actions #10

Updated by Raphael GAUTHIER about 5 years ago

  • Status changed from Pending technical review to Pending release
Actions #11

Updated by Vincent MEMBRÉ about 5 years ago

  • Priority changed from 94 to 93
Actions #12

Updated by Alexis Mousset about 5 years ago

  • Subject changed from In technique editor, deleting a new technique crashes to Deleting a technique in the technique editor just after its creation fails
  • Name check changed from To do to Reviewed
Actions #13

Updated by Alexis Mousset about 5 years ago

  • Fix check changed from To do to Checked
Actions #14

Updated by Vincent MEMBRÉ about 5 years ago

  • Status changed from Pending release to Released
  • Priority changed from 93 to 92

This bug has been fixed in Rudder 5.0.15 which was released today.

Actions #15

Updated by Alexis Mousset over 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - UI/UX to Web - Technique editor
  • Priority changed from 92 to 82
Actions

Also available in: Atom PDF