fix: dont warn on error cleaning individual log files

[lukekarrys]

Closes: #4128

This also refactors the logic for cleaning log files to use the ignore
option from glob to not clean current log files instead of relying on
updating the logs-max count.

I also discovered a bug where we weren't cleaning log files written with
the old naming convention, so this fixes that as well.

[wraithgar]

My one tiny suggestion is in a test file and not a big deal.

[npm-robot]

found 4 benchmarks with statistically significant performance improvements

• app-large: modules-only, no-lock
• app-medium: modules-only, no-lock

found 1 benchmarks with statistically significant performance regressions

• app-large: clean

timing results

app-large	clean	lock-only	cache-only	cache-only peer-deps	modules-only	no-lock	no-cache	no-modules	no-clean	no-clean audit
npm@7	54.823 ±0.47	30.141 ±0.25	27.193 ±12.62	20.576 ±0.81	28.462 ±2.05	8.786 ±0.01	2.546 ±0.01	12.396 ±0.05	2.551 ±0.03	3.359 ±0.01
#4134	61.977 ±2.92	32.494 ±0.29	29.167 ±13.60	21.809 ±0.83	3.314 ±0.16	3.256 ±0.02	2.630 ±0.02	12.776 ±0.02	2.615 ±0.02	3.602 ±0.20

app-medium	clean	lock-only	cache-only	cache-only peer-deps	modules-only	no-lock	no-cache	no-modules	no-clean	no-clean audit
npm@7	38.298 ±2.65	23.933 ±0.33	13.802 ±0.15	14.775 ±0.21	18.655 ±1.18	7.997 ±0.04	2.653 ±0.00	9.380 ±0.13	2.488 ±0.06	3.067 ±0.06
#4134	39.496 ±4.09	25.510 ±0.07	14.564 ±0.22	15.398 ±0.13	2.911 ±0.02	2.974 ±0.02	2.625 ±0.04	9.696 ±0.03	2.510 ±0.02	3.217 ±0.18