Skip to content

Make robot names be in lowercase when uploading logs

Marcos Huck requested to merge fix/robot_names_url into develop

Related tasks

Context

The worker refactor introduced a new naming convention for robot names when uploading logs to S3. They were previously named in lowercase, now they keep the same format as when the simulation was requested, the subt app is making users download robot logs in lowercase format, that's why they can't download logs, as they're differently named now.

For example:

  • Original naming scheme: 94e0-fc-teambase-commsbridge.tar.gz
  • New naming scheme: 94e0-fc-TEAMBASE-commsbridge.tar.gz

The trade-off here is:

  1. Restore lowercase convention, giving access to previous simulation logs (before worker refactor), but we'll need to rename .tar.gz files for all recently created simulations.
  2. Enforcing the new convention to keep the same name as in the request, but we'll loose access to logs before the worker refactor was introduced, the good thing is that teams will gain immediate access to recent logs.

Change

This MR makes the GetCommsBridgeLogsFilename return the robot name in lowercase.

Other information

IMPORTANT: Recently created simulation robot logs need to match the original naming scheme.

Additional documentation

Edited by Marcos Huck

Merge request reports