AutoM8 Question - Picking up Switch Interfaces it should not be

[cksrealm]

Query is returning interfaces that are in a Down/Down state and not just ones in an Up/Up state as I expect.

[Osiris]

Can you log this on GitHub? It's a bug the inside query is missing a group by hsc.host_id.

[cksrealm]

Sure thing

[TheWitness]

Saw the ticket and it's not missing the GROUP BY. I guess the question is, when did you last Re-Index the devices? The data it's pulling to do the automation is from the Re-Index Cache and not the live device info. Could this be the issue as I did not find any logic errors in my review. I think showing the SQL is a great idea, but not before the 1.2.25 release.

[Rno]

I have to say this part look wrong to me:

Code: Select all

WHERE ( a.`ifOperStatus` LIKE '%Up%' AND a.`ifAdminStatus` LIKE '%Up%' ( AND a.`ifAlias` LIKE '%//%' OR a.`ifAlias` LIKE '%Wireless%' OR a.`ifName` LIKE '%Stack%' ))

First the OpenStatus AND the Admin Status is UP, not like. I my code I use the option Match and that part work.
Second when you do the Alias Test, the AND should be outside the parenthesis, not as first field inside of it.
So add a item AND after the position 2, then you start the ( Alias test without the and

Rough copy of my code (sorry not good at print screen with GitHub)
This Autom( is to create graph on link that are UP, where the ifAlias is SE- or SRE, etc
For my it's a trunk port to another switch.

Code: Select all

Item#1	1	 	ifType	contains	ethernetCsmacd		
Item#2	4	AND	ifOperStatus	matches	Up		
Item#3	5	AND					
Item#4	8	(					
Item#5	9	(	ifAlias	begins with	SE-		
Item#6	11	OR	ifAlias	begins with	SRE-		
Item#7	12	)					
Item#8	13	OR	ifHighSpeed	is greater than or equal	10000		
Item#9	14	AND	ifAlias	does not contain	Stackwise		
Item#10	15	AND	ifAlias	does not contain	XXX		
Item#11	16	AND	ifAlias	is not empty			
Item#12	17	)					

[Rno]

Ho by the way what's the goal of testing the adminStatus if you test the operStatus ?

You will never have a operStatus UP with an adminStatus DOWN.
So just testing the operStatus on UP is enough.

[TheWitness]

So, I did not write this, I inherited it instead. It's a bug for sure.