Bug #12282
closedIn technique editor, technique with missing generic method need a warning icon
Description
We can need to use bundle which are not in the generic method library. It is not an error to have such a bundle, but it must be differenciated visualy ("warning: user, that method does not exists in provided ncf lib, you must provide it or it will break at run time").
It could be something like that at technique level:
And an other icon should be added on generic method in technique:
Note: I'm not sure we want it to be a warning or an error.
Files
Updated by Vincent MEMBRÉ over 6 years ago
- Target version changed from 4.1.11 to 4.1.12
Updated by Vincent MEMBRÉ over 6 years ago
- Target version changed from 4.1.12 to 4.1.13
Updated by Benoît PECCATTE over 6 years ago
- Target version changed from 4.1.13 to 411
Updated by Benoît PECCATTE over 6 years ago
- Target version changed from 411 to 4.1.13
Updated by Vincent MEMBRÉ over 6 years ago
- Target version changed from 4.1.13 to 4.1.14
Updated by Benoît PECCATTE over 6 years ago
- Target version changed from 4.1.14 to 4.1.15
Updated by François ARMAND about 6 years ago
- Subject changed from In technique editor, non existing generic method are not differenciated from the one in lib to In technique editor, technique broken (because missing generic method) are not highlighted
- Severity set to Minor - inconvenience | misleading | easy workaround
- User visibility set to Infrequent - complex configurations | third party integrations
- Priority changed from 0 to 20
Updated by Vincent MEMBRÉ about 6 years ago
- Target version changed from 4.1.15 to 4.1.16
Updated by Vincent MEMBRÉ about 6 years ago
- Target version changed from 4.1.16 to 4.1.17
- Priority changed from 20 to 19
Updated by François ARMAND about 6 years ago
- Effort required set to Small
- Priority changed from 19 to 34
Updated by Vincent MEMBRÉ almost 6 years ago
- Target version changed from 4.1.17 to 4.1.18
- Priority changed from 34 to 33
Updated by Vincent MEMBRÉ almost 6 years ago
- Target version changed from 4.1.18 to 4.1.19
- Priority changed from 33 to 32
Updated by Alexis Mousset almost 6 years ago
- Target version changed from 4.1.19 to 4.1.20
Updated by François ARMAND over 5 years ago
- Target version changed from 4.1.20 to 4.1.21
- Priority changed from 32 to 31
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 4.1.21 to 4.1.22
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 4.1.22 to 4.1.23
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 4.1.23 to 4.1.24
Updated by Vincent MEMBRÉ over 5 years ago
- Target version changed from 4.1.24 to 588
Updated by Nicolas CHARLES over 5 years ago
- Target version changed from 588 to 5.0.13
most likely still the case in 5.0
Updated by Vincent MEMBRÉ about 5 years ago
- Target version changed from 5.0.13 to 5.0.14
Updated by Vincent MEMBRÉ about 5 years ago
- Target version changed from 5.0.14 to 5.0.15
Updated by Vincent MEMBRÉ almost 5 years ago
- Target version changed from 5.0.15 to 5.0.16
Updated by Alexis Mousset almost 5 years ago
- Target version changed from 5.0.16 to 5.0.17
Updated by Vincent MEMBRÉ over 4 years ago
- Target version changed from 5.0.17 to 5.0.18
Updated by Vincent MEMBRÉ over 4 years ago
- Target version changed from 5.0.18 to 5.0.19
Updated by Vincent MEMBRÉ about 4 years ago
- Target version changed from 5.0.19 to 5.0.20
Updated by Vincent MEMBRÉ about 4 years ago
- Target version changed from 5.0.20 to 797
Updated by Benoît PECCATTE over 3 years ago
- Target version changed from 797 to 6.1.14
- Priority changed from 31 to 32
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.1.14 to 6.1.15
Updated by Vincent MEMBRÉ over 3 years ago
- Target version changed from 6.1.15 to 6.1.16
Updated by Vincent MEMBRÉ about 3 years ago
- Target version changed from 6.1.16 to 6.1.17
- Priority changed from 32 to 33
Updated by Vincent MEMBRÉ about 3 years ago
- Target version changed from 6.1.17 to 6.1.18
Updated by Vincent MEMBRÉ almost 3 years ago
- Target version changed from 6.1.18 to 6.1.19
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.1.19 to 6.1.20
- Priority changed from 33 to 34
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.1.20 to 6.1.21
- Priority changed from 34 to 35
Updated by Vincent MEMBRÉ over 2 years ago
- Target version changed from 6.1.21 to old 6.1 issues to relocate
Updated by François ARMAND over 1 year ago
- File clipboard-202303021705-ugm5c.png clipboard-202303021705-ugm5c.png added
- File clipboard-202303021708-reus3.png clipboard-202303021708-reus3.png added
- Subject changed from In technique editor, technique broken (because missing generic method) are not highlighted to In technique editor, technique with missing generic method need a warning icon
- Description updated (diff)
- Target version changed from old 6.1 issues to relocate to 7.2.5
- Priority changed from 35 to 0
- Regression set to No
Updated by François ARMAND over 1 year ago
- UX impact set to I dislike using that feature
- Priority changed from 0 to 61
The case is extremelly infrequent, but encoutering it make it even more complicated to understand since there is nothing telling us that we are out of our normal way.
Updated by Vincent MEMBRÉ over 1 year ago
- Target version changed from 7.2.5 to 7.2.6
Updated by Elaad FURREEDAN over 1 year ago
- Status changed from New to In progress
- Assignee set to Elaad FURREEDAN
Updated by Elaad FURREEDAN over 1 year ago
- Status changed from In progress to Pending technical review
- Assignee changed from Elaad FURREEDAN to François ARMAND
- Pull Request set to https://github.com/Normation/rudder/pull/4737
Updated by François ARMAND over 1 year ago
Perfect!
Updated by Anonymous over 1 year ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder|d2bf8a5bf23a46cee50042b89963b12bafdae420.
Updated by Vincent MEMBRÉ over 1 year ago
- Status changed from Pending release to Released
This bug has been fixed in Rudder 7.2.6 and 7.3.1 which were released today.