Project

General

Profile

Actions
Copy link

Bug #26023

closed

Technique compilation errors doesn't seems to be reloaded when the technique is deleted

Bug #26023: Technique compilation errors doesn't seems to be reloaded when the technique is deleted

Added by Elaad FURREEDAN about 1 year ago. Updated 12 months ago.

Status:
Released
Priority:
1 (highest)
Assignee:
François ARMAND
Category:
Techniques
Target version:
8.2.4
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
I hate Rudder for that
User visibility:
First impressions of Rudder
Effort required:
Priority:
160
Name check:
To do
Fix check:
Checked
Regression:
No

Description

I try to delete a technique that generate compilation error on status, I delete the directory of the technique, git add, git commit, rudder server reload-techniques but the error is still present.
Even if I try to delete it from Techniques tree page, the error is still here. The error disappear when restarting the webapp.

Related issues 2 (0 open — 2 closed)

Related to Rudder - Bug #26022: Status should not be on error when there is technique compilator error on disable techniquesReleasedFrançois ARMANDActions
Related to Rudder - Bug #25527: List techniques with compilation failure in bar for generation statusReleasedFrançois ARMANDActions

Updated by Elaad FURREEDAN about 1 year ago Actions
Copy link
 #1

  • Priority changed from N/A to To review

Updated by Elaad FURREEDAN about 1 year ago Actions
Copy link
 #2

  • Related to Bug #26022: Status should not be on error when there is technique compilator error on disable techniques added

Updated by Clark ANDRIANASOLO about 1 year ago Actions
Copy link
 #3

  • Related to Bug #25527: List techniques with compilation failure in bar for generation status added

Updated by Elaad FURREEDAN about 1 year ago Actions
Copy link
 #4

  • Subject changed from Technique compilation errors doesn't seems to be reload when the technique is delete from FS to Technique compilation errors doesn't seems to be reloaded when the technique is delete from FS

Updated by Elaad FURREEDAN about 1 year ago Actions
Copy link
 #5

  • Subject changed from Technique compilation errors doesn't seems to be reloaded when the technique is delete from FS to Technique compilation errors doesn't seems to be reloaded when the technique is deleted

Updated by Elaad FURREEDAN about 1 year ago Actions
Copy link
 #6

In fact, even if we delete a technique from the editor directly, the error doesn't go away unless we reboot the webapp

Updated by Clark ANDRIANASOLO about 1 year ago Actions
Copy link
 #7

The data model is a cache but the TechniqueCompilationErrorsActorSync#updateStatus logic does not reset the cache, it only traverse techniques and remove those in success from the cache and adds those in error to the cache. The deleted ones are left in the cache if they were deleted when being in error

Updated by François ARMAND about 1 year ago Actions
Copy link
 #8

  • Assignee set to Clark ANDRIANASOLO

Updated by François ARMAND about 1 year ago Actions
Copy link
 #9

  • Priority changed from To review to 1 (highest)

Updated by Clark ANDRIANASOLO about 1 year ago Actions
Copy link
 #10

  • Status changed from New to In progress

Updated by Clark ANDRIANASOLO about 1 year ago Actions
Copy link
 #11

  • Status changed from In progress to Pending technical review
  • Assignee changed from Clark ANDRIANASOLO to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/6068

Updated by Clark ANDRIANASOLO about 1 year ago Actions
Copy link
 #12

  • Status changed from Pending technical review to Pending release

Updated by Elaad FURREEDAN about 1 year ago Actions
Copy link
 #13

  • Fix check changed from To do to Checked

Updated by Vincent MEMBRÉ 12 months ago Actions
Copy link
 #14

  • Status changed from Pending release to Released
  • Priority changed from 161 to 160

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

Actions
Copy link

Also available in: PDF Atom