20137: Fix incorrect keyword in RailsAPI config reload loop
authorBrett Smith <brett.smith@curii.com>
Mon, 20 Feb 2023 19:46:41 +0000 (14:46 -0500)
committerBrett Smith <brett.smith@curii.com>
Mon, 20 Feb 2023 19:46:41 +0000 (14:46 -0500)
commite03ebe77c257e44645c64c8cd71e5e7c115991ef
tree01c4acee4e129d40ff4f22eea432f9ec72c22ed6
parent94dcdefbf8fe264daa28c5b15f68b304c683e390
20137: Fix incorrect keyword in RailsAPI config reload loop

The reason rails_restart_test.go has been racey is because the code
under test is actually buggy. If the configuration reload loop ever
encounters a condition where it wants to restart (probably
`hash == hash_lastload`), it tries to call a nonexistent method
`continue`, the thread crashes, the configuration never gets reloaded,
and the test fails.

Use the intended Ruby keyword `next` instead.
Closes #20137.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
services/api/config/initializers/reload_config.rb