open-balena-api icon indicating copy to clipboard operation
open-balena-api copied to clipboard

patch: Replace legacy config RESIN_OVERRIDE_LOCK

Open vipulgupta2048 opened this issue 1 year ago • 2 comments

Got a user mentioning the confusion of two configuration variables being present to enforce the override lock. Talking to @cywang117, it got clarified that resin_override_lock is legacy and balena_supervisor_override_lock is latest

The Docs is where the confusion started because Docs pulls these values directly from the API: https://docs.balena.io/reference/supervisor/configuration-list/raspberrypi3/

Hence, to fix the docs, we have to correct the variable name in the API and this is the PR doing just that.

Signed-off-by: Vipul Gupta (@vipulgupta2048) [email protected]

vipulgupta2048 avatar Aug 10 '23 06:08 vipulgupta2048

Thinking about this again, if there is a reason we have chosen to keep RESIN_OVERRIDE_LOCK around then I can handle this one on the docs with a simple if condition. LMK

vipulgupta2048 avatar Aug 10 '23 10:08 vipulgupta2048

@vipulgupta2048 it looks like the SUPERVISOR_OVERRIDE_LOCK being in the block list and having the bC-api & the sdk using the RESIN_OVERRIDE_LOCK was intentional, so I think that we should (at least for now) stick with RESIN_OVERRIDE_LOCK See: https://github.com/balena-io/open-balena-api/blob/254475e12c759d029273a76b5eebc35ac731935c/src/features/vars-schema/env-vars.ts#L48-L57 See: https://github.com/balena-io/balena-api/blob/7e6d06fe8e75d7aee5a0f0079e0f21794acc7e14/src/features/devices/routes/lock-override.ts#L6 See: https://github.com/balena-io/balena-sdk/blob/4e54ae8a0daebbf85bd2ba48c9a522a96f5803ce/src/models/device.ts#L90

thgreasi avatar Sep 12 '23 08:09 thgreasi