[17.0][IMP] fs_storage: Allow setting check_connection_method in configuration file

[vincent-hatakeyama]

No description provided.

[etobella]

Making this change will loose the data from the instance, as the system will drop the column check_connection_method.

At least we need a migration script

[vincent-hatakeyama]

I’ve checked : the column is not dropped.

The value is lost thought, as server_environment looks for the value elsewhere.

I’m trying to find a way to migrate the value.

[vincent-hatakeyama]

Migration script added.

[etobella]

Have you checked on a database?

[vincent-hatakeyama]

I’ve done the test on 18. The value is migrated correctly.

Before:

template_odoo_18=# select * from fs_storage;
-[ RECORD 1 ]-----------+-------------------------------------
id                      | 1
create_uid              | 1
write_uid               | 2
name                    | Odoo Filesystem Backend
code                    | odoofs
check_connection_method | ls
server_env_defaults     | {"x_protocol_env_default": "odoofs"}
create_date             | 2025-07-02 08:53:43.386356
write_date              | 2025-07-02 09:07:24.933916

After migration:

template_odoo_18=# select * from fs_storage;
-[ RECORD 1 ]-------+------------------------------------------------------------------------------------
id                  | 1
create_uid          | 1
write_uid           | 2
name                | Odoo Filesystem Backend
code                | odoofs
server_env_defaults | {"x_protocol_env_default": "odoofs", "x_check_connection_method_env_default": "ls"}
create_date         | 2025-07-02 08:53:43.386356
write_date          | 2025-07-02 09:07:24.933916

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖