Improve node idle detection, including adding NODE_KEEP

app/Jobs/AddressIdle.php

@@ -176,41 +176,28 @@ class AddressIdle implements ShouldQueue
 		$age = Carbon::now()->subDays($days)->endOfDay();
 
 		return Address::select([
-			'a.id',
-			'a.system_id',
-			'a.zone_id',
-			'addresses.region_id',
-			'a.host_id',
-			'a.node_id',
-			'a.point_id',
+			'addresses.id',
+			'system_id',
+			'zone_id',
+			'region_id',
+			'host_id',
+			'node_id',
+			'point_id',
 			'addresses.active',
-			'addresses.hub_id',
-			'addresses.role',
+			'hub_id',
+			'role',
 			'addresses.updated_at',
-			DB::raw('sum(a.uncollected_echomail) as uncollected_echomail'),
-			DB::raw('sum(a.uncollected_netmail) as uncollected_netmail'),
-			DB::raw('sum(a.uncollected_files) as uncollected_files')
 		])
-			->from(
-				Address::UncollectedEchomailTotal()
-					->union(Address::UncollectedNetmailTotal())
-					->union(Address::UncollectedFilesTotal()),'a')
+			->activeFTN()
 			->where('systems.active',TRUE)
-			->where('addresses.active',TRUE)
-			->where('zones.active',TRUE)
-			->where('domains.active',TRUE)
-			->where(fn($query)=>$query->where('a.point_id',0)->orWhereNull('a.point_id'))
-			->whereNotIn('a.id',our_address()->pluck('id'))
+			->where(fn($query)=>$query->where('point_id',0)->orWhereNull('point_id'))
+			->whereIn('addresses.id',our_nodes($do)->pluck('id'))
 			->when($ao,fn($query)=>$query->where('addresses.id',$ao->id))
-			->where('last_session','<',$age)
-			->where('domains.id',$do->id)
-			->whereRaw(sprintf('((role IS NULL) OR ((role & %d) > 0))',$flags))
-			->join('addresses',['addresses.id'=>'a.id'])
-			->join('systems',['systems.id'=>'a.system_id'])
-			->join('zones',['zones.id'=>'addresses.zone_id'])
-			->join('domains',['domains.id'=>'zones.domain_id'])
+			->where(fn($q)=>$q->where('last_session','<',$age)->orWhereNull('last_session'))
+			->whereRaw(sprintf('((role IS NULL) OR (role=0) OR ((role & %d) > 0))',$flags))
+			->whereRaw(sprintf('((role & %d) = 0)',Address::NODE_KEEP))
+			->join('systems',['systems.id'=>'addresses.system_id'])
 			->ftnOrder()
-			->groupBy('a.system_id','a.id','a.zone_id','addresses.region_id','a.host_id','a.node_id','a.point_id','addresses.hub_id','addresses.role','addresses.active','addresses.updated_at')
 			->with(['system','zone.domain'])
 			->dontCache()
 			->get();