Project

General

Profile

Bug #6327

Change the error message: There are directives based on techniques with different versions applied to the same node, please correct the version for the following directive(s): checkGenericFileContent

Added by Dennis Cabooter over 3 years ago. Updated about 1 month ago.

Status:
New
Priority:
2
Assignee:
-
Category:
Web - Config management
Target version:
Target version (plugin):
Severity:
Minor - inconvenience | misleading | easy workaround
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Very Small
Pull Request:
Priority:
52

Description

Error message when regenerating policies:

There are directives based on techniques with different versions applied to the same node, please correct the version for the following directive(s): checkGenericFileContent

This is not true! As you can see on the attached screenshot I migrated only directives applied to desktops. I'm sure this are the only checkGenericFileContent directives to be applied to desktops. Please change the message so that it makes sense (you need to have all checkGenericFileContent directives to be the same) or change things so that they do what they say.

policy_update.png (250 KB) policy_update.png Dennis Cabooter, 2015-03-03 14:12

History

#1 Updated by Dennis Cabooter over 3 years ago

  • File deleted (technique_deprecated.png)

#2 Updated by Dennis Cabooter over 3 years ago

  • File tech_logs.png added

#3 Updated by Dennis Cabooter over 3 years ago

  • File deleted (tech_logs.png)

#5 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 3.0.2 to 3.0.3

#6 Updated by François ARMAND over 3 years ago

Hi Denis, this is interesting.

The check is actually done node by node, and only check that on any given node, all the directive are derived from the same version of technique.

For example, you could have:
- node1 => [directive1_0 from tech1-v1 ; directive1_1 from tech1-v1]
- node2 => [directive2_0 from tech1-v2 ]

This is the only (HUGE - we are aware of it) limitation on technique version.

I'm going to try to reproduce. If you have a way to reproduce, that would be very helpful.

In all case, we have to make that message clearer, and add log to help analyse the problem.

#7 Updated by Dennis Cabooter over 3 years ago

Say you have 4 directives, dir1, dir2 dir3 and dir4:

dir1 and dir2 => group: servers
dir3 and dir4 => group: desktops

Then you migrate only dir1 and dir2 and you'll get the error. Basicly you need to migrate all directives of the same type or the error occurs.

#8 Updated by François ARMAND over 3 years ago

  • Assignee set to François ARMAND

OK, that should not be the case, you should be able to migrate them independently.
I'm going to try to reproduce. Did you noticed that on version before 3.0 ?

#9 Updated by François ARMAND over 3 years ago

I just tested with Rudder (3.0.1) and it possible to have a directive (checkGenericFileContent in my case) with:

- version 6 applied to group 1 (with node 1 into it),
- version 7 applied to group 2 (with node 2 into it).

So perhaps you have some overlap in the groups...

Could you check that, it would be very helpful: I don't want to overlook a bug because I wasn't able to reproduce it.

But in all case, the error message should say explicitly what is the problem ("node 1 has the following rules applying diffent version of technique XXX: [rule1, rule2, ...]; node 2 etc).

#10 Updated by François ARMAND over 3 years ago

  • Subject changed from There are directives based on techniques with different versions applied to the same node, please correct the version for the following directive(s): checkGenericFileContent to Change the error message: There are directives based on techniques with different versions applied to the same node, please correct the version for the following directive(s): checkGenericFileContent
  • Priority changed from 1 to 2
  • Target version changed from 3.0.3 to 2.10.12
  • Reproduced set to No

I'm updating that ticket to the message problem. When we have better begguging information, we would be able to open a better qualified ticket for the other problem.

#11 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.12 to 2.10.13

#12 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.13 to 2.10.14

#13 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.14 to 2.10.15

#14 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.10.15 to 2.10.16

#15 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.10.16 to 2.10.17

#16 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.17 to 2.10.18

#17 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.18 to 2.10.19

#18 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.19 to 2.10.20

#19 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.20 to 277

#20 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 277 to 2.11.18

#21 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.18 to 2.11.19

#22 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.19 to 2.11.20

#23 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.11.20 to 2.11.21

#24 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.11.21 to 2.11.22

#25 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.11.22 to 2.11.23

#26 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 2.11.23 to 2.11.24

#27 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 2.11.24 to 308

#28 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 308 to 3.1.14

#29 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.14 to 3.1.15

#30 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.15 to 3.1.16

#31 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.16 to 3.1.17

#32 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.17 to 3.1.18

#33 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.18 to 3.1.19

#34 Updated by Jonathan CLARKE about 1 year ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings

#35 Updated by Jonathan CLARKE about 1 year ago

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

#36 Updated by Benoît PECCATTE about 1 year ago

  • Priority set to 14

#37 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.19 to 3.1.20

#38 Updated by Jonathan CLARKE about 1 year ago

  • Assignee deleted (François ARMAND)

#39 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.20 to 3.1.21

#40 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.21 to 3.1.22

#41 Updated by Benoît PECCATTE 12 months ago

  • Priority changed from 14 to 27

#42 Updated by Benoît PECCATTE 12 months ago

  • Effort required set to Very Small
  • Priority changed from 27 to 41

#43 Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 3.1.22 to 3.1.23

#44 Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 3.1.23 to 3.1.24

#45 Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 3.1.24 to 3.1.25

#46 Updated by Benoît PECCATTE 7 months ago

  • Priority changed from 41 to 51

#47 Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 3.1.25 to 387

#48 Updated by Vincent MEMBRÉ 6 months ago

  • Target version changed from 387 to 4.1.10

#49 Updated by Vincent MEMBRÉ 4 months ago

  • Target version changed from 4.1.10 to 4.1.11

#50 Updated by Vincent MEMBRÉ 2 months ago

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 51 to 52

#51 Updated by Vincent MEMBRÉ about 1 month ago

  • Target version changed from 4.1.12 to 4.1.13

Also available in: Atom PDF