bluerobotics / BlueOS

The open source platform for ROV, USV, robotic system operation, development, and expansion.
https://blueos.cloud/docs/
Other
154 stars 74 forks source link

core: frontend: download extension logs #2962

Open ES-Alexander opened 2 days ago

ES-Alexander commented 2 days ago

Current behaviour

Downloading system logs is typically only for BlueOS-core. When troubleshooting it can be relevant to have logs from other software running.

Expected or desired behaviour

Add an "include extension logs" checkbox to the System Log Files section, so the download button includes logs from the extensions directory (perhaps specifically any folder that's extensions/*/logs/).

Prerequisites

rotu commented 1 day ago

I don't expect extensions to be able to log to file, especially if the extension fails to start unexpectedly. Maybe we should first add a way to download the existing docker logs with a link on the extensions manager page per #1611.

I had to resort to telling a user how to manually use the kraken API:

Navigate your web browser to http://blueos.local:9134/v2.0/list_containers and look for something that contains the words “sonarview” and the “name” field. It should be something like (maybe exactly) "/extension-nicknothomsonarview1100".

If that’s what it is, then go to the following URL (if the name is different, adjust this URL accordingly):

http://blueos-dan.local:9134/v2.0/log?container_name=/extension-nicknothomsonarview1100

JoaoMario109 commented 19 hours ago

Reopened it since the linked PR (#2963) only added downloading logs for individual extensions, not downloading all of them as an option when downloading the system logs.

ES-Alexander commented 18 hours ago

I don't expect extensions to be able to log to file

Per #1640, extensions are expected to have file system access, and can put persistent logs there if they want to.

Maybe we should first add a way to download the existing docker logs with a link on the extensions manager page

That's what @JoaoMario109's PR just added :-)