thin-edge / thin-edge.io

The open edge framework for lightweight IoT devices
https://thin-edge.io
Apache License 2.0
221 stars 54 forks source link

Rename configuration enable.sudo to sudo.enable #2566

Closed reubenmiller closed 9 months ago

reubenmiller commented 9 months ago

Is your feature improvement request related to a problem? Please describe.

The tedge configuration item, enable.sudo is not consistent with out configuration items like http, c8y, where the first part of the configuration item represents a category and not an action.

enable.sudo=true

For example you just need to look at other configuration items with enable in their names:

# tedge config list | grep enable
c8y.enable.log_upload=true
c8y.enable.config_snapshot=true
c8y.enable.config_update=true
c8y.enable.firmware_update=true
agent.enable.config_update=true
agent.enable.config_snapshot=true
agent.enable.log_upload=true
enable.sudo=true

Describe the solution you'd like

Rename enable.sudo to sudo.enable.

Describe alternatives you've considered

Additional context

reubenmiller commented 9 months ago

Merged. The solution was implemented as suggested in the ticket description.

gligorisaev commented 9 months ago

QA has thoroughly checked the feature and here are the results: