graylog2-server icon indicating copy to clipboard operation
graylog2-server copied to clipboard

Erroneous Index Set prefix conflict

Open williamtrelawny opened this issue 5 years ago • 6 comments

Expected Behavior

Index Set prefixes should not conflict if they are in their entirety distinct strings, even if they start with some of the same characters.

Current Behavior

Have existing Index Set with prefix "cisco". Create a new Index Set with prefix "ciscoasa". API complains that new Index Set prefix conflicts with existing prefix:

{"type":"ApiError","message":"Index prefix "ciscoasa" would conflict with existing index set prefix "cisco""}HTTP/1.1 100 Continue

Possible Solution

Not sure exactly how the matching test goes for Index Set prefixes, but it appears to be based on a certain length at the start of the string. I have another instance where there are two Index Set prefixes that start with the same 2 characters but deviate after the 3rd, so there must be some matching logic that fails once the beginnings of the strings match further down the string.

Or perhaps this fails because the entirety of the "cisco" prefix matches against the other prefix "ciscoasa", which is causing the error.

Either way, I consider this a failure of the matching system.

Steps to Reproduce (for bugs)

  1. Create Index Set with prefix "cisco"
  2. Attempt to create Index Set with prefix "ciscoasa"... API error from above.

Context

Trying to create separate indices for various Cisco devices- ASA, Firepower, ISE, etc., and want the prefixes to reflect the different device types.

Your Environment

  • Graylog Version: 3.0.2
  • Elasticsearch Version: 6.7.1
  • MongoDB Version: 4.0.8
  • Operating System: OEL-7.5
  • Browser version: n/a

williamtrelawny avatar Feb 27 '20 15:02 williamtrelawny

[z#843198]

jalogisch avatar Feb 28 '20 12:02 jalogisch

https://github.com/Graylog2/graylog2-server/blob/b5e5f2740e469515c7515e04fb35b61a08d2983f/graylog2-server/src/main/java/org/graylog2/indexer/IndexSetValidator.java#L36-L50

According to the Code the example should work at all.

jalogisch avatar Feb 28 '20 15:02 jalogisch

@jalogisch @florianpopp still happening in 5.2.1:

{
  "type": "ApiError",
  "message": "Index prefix \"index_naming_test\" would conflict with existing index set prefix \"index_naming\""
}

Littlericket avatar Nov 17 '23 15:11 Littlericket

I have the same problem in 5.2.1.

I'm also surprised there aren't more people who have the problem, or maybe they do but don't share it and just use cryptic indexes so there's no conflict.

bloodhunterd avatar Dec 11 '23 09:12 bloodhunterd

Was in progress for opening a new Bug on this one also, but found this report. Like you iam suprised that this is so long open but like you assume, i just switched to other name generation.

But in general it would be a lot easier for me if this was fixed because using natural named prefixes is a lot more human.

/hasturo

hasturo avatar Feb 16 '24 11:02 hasturo

It's not a blocking bug, but it's always annoying to first find out what exactly the problem is and then think about what a good work-around could be.

bloodhunterd avatar Feb 16 '24 11:02 bloodhunterd