armbianmonitor -u: rationalize paste server retrying, use ANSI dmesg

[rpardini]

armbianmonitor -u: rationalize paste server retrying, use ANSI dmesg

• armbianmonitor: shellfmt, no changes
• armbianmonitor: squash shellcheck warnings around the main getopts case statement
  • b and B were never parsed
  • an * case is added to handle invalid options
• armbianmonitor -u: rationalize paste server retrying, use ANSI dmesg
  • use PIPESTATUS to check all steps for failure
  • avoid blaming paste server if data collection failed

[EvilOlaf]

Looping through paste servers doesn't seem to work. At least it doesn't catch the current error to try the next server.

[rpardini]

Looping through paste servers doesn't seem to work. At least it doesn't catch the current error to try the next server.

Could you share logs? (eg from bash -x /usr/bin/armbianmonitor -u)

Here I've a machine with a network misconfiguration, and that loops over both servers (and both fail "correctly") so wondering what could it be.

[EvilOlaf]

"Unfortunately" TheLinuxBug fixed the error (disk ran out of space) before thinking about grabbing such logs and now cannot test anymore. Apologize

[rpardini]

the error (disk ran out of space)

Hmm, in this case the bug is probably on the server side -- it should reply with a 500-level HTTP code, which would cause curl to "correctly" fail and the script to try the next server.

[rpardini]

Hmm, the server seems to at least try to spit out 500 error when this problem occurs: https://github.com/rpardini/ansi-hastebin/blob/fefe5edda54f5cafc113554a3a28d93cb5a9b013/lib/document_handler.js#L161-L162

Seems that curl doesnt' exit with an error if it receives a 500. Hmm. Will send a patch. Thanks for reporting @EvilOlaf

[rpardini]

#7487