Improve Handling for unavailable targets about autoscan HOT 3 CLOSED

Note: I have no experience with go, but I do have experience with other languages, so while I am confident in my ability to find and provide a solution I am not comfortable committing anything due to my lack of experience with the go language, however this issue hasn't been resolved and it feels like a simple solution.

targetCounter := 0
for {

if targetCounter == 4 {
    // sleep indefinitely
    select{}
} else {
    counter += 1
    time.Sleep(30 * time.Second)
    continue
}

I would much prefer this (and the others) to be at least a minute maybe 3 as suggested by OP

from autoscan.

The main reason why Autoscan halts any scanning when a target is down is because we want to prevent two things:

1. Targets not unnecessarily scanning the same path multiple times.
2. Targets not being "left out" because they're unavailable at the time.

@UnknownWitcher the indefinite sleep is not necessarily related to this issue. Autoscan will only reach that state when the target is throwing a fatal error, which can only occur if there are setup-related problems (and thus only an adjustment to the config can make a difference).

Autoscan's current behaviour is to check Target health every 15 seconds. By specifically checking for health (instead of simply sending other scans), no duplicate scans occur. If all targets are found to be healthy, scanning will continue!

Now, I hear you wonder:

Can't Autoscan simply keep track of which scans have been successfully processed by certain targets? So if a target is unavailable, only that target will receive scans at a later date?

We did think about this while designing Autoscan, but by linking scans to targets, the database becomes dependent on the config. We try to avoid this as much as possible as it can cause issues when someone's config changes.

from autoscan.

Hey @m-rots thanks for getting back to me, that's completely understandable and yes I did have that thought lol, anyway, I ended up running the container with "depends_on - plex" to avoid this issue.

from autoscan.