…ry to write

…n the http endpoint

…osts to simplify the logic

… missing lock

…ng variable

Co-authored-by: Tim Gross <[email protected]>

Also hide the variable items

… nil pointer

…ant be used because of the missing lock ID

…rappers

…responsibility is left for teh user

* fix: move the rpc metrics up on the renew lock to include auth errors

* func: add acquire and release lock functions on fsm

* func: fix the query for missing lock to avoid nil points in case of a missing lock

* func: add happy path test for acquire and release flows on non existing variable

* fix: solve rebase with rpc conflicts

* func: add testing for release lock

* style: typo

* fix: remove debug file

* Update nomad/fsm.go

Co-authored-by: Tim Gross <[email protected]>

* func: change the return when lock ID is wrong from error to conflict
Also hide the variable items

* func: add lock ID verification for variable upserts

* fix: check for empty lock instead of empty lock ID on setCas to avoid nil pointer

* fix: separate the table update for the lock release, the var update cant be used because of the missing lock ID

* func: add tests for delete CAS with lock ID check

* func: move lock ID check to the inner most tx function, not the CAS wrappers

* fix: add lock not nil check on release to avoid nill pointers

* style: function rename

* func: remove the lock ID check from the CAS delete

* func: allow for users to overide the variable data when locking, the responsibility is left for teh user

---------

Co-authored-by: Tim Gross <[email protected]>

…k check for var apply

…id overwriting data when lock release

…onse

…d add testing

…oid panics

…void panics

Co-authored-by: Tim Gross <[email protected]>

Co-authored-by: Tim Gross <[email protected]>