Fix is_master_only options. Now it runs on master correctly.

When `is_master_only` was true, expirationd worked only on replica. It was not correct.
Now it works only on master when `is_master_only` true.

(cherry picked from commit a411f3e16b81bd8f2e9fc7af073c95768f1020e5)

Author: Satbek

CHANGELOG.md

@@ -5,14 +5,16 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
-## [Unreleased]
+## Unreleased
 
 ### Added
 
 ### Changed
 
 ### Fixed
 
+- Fix `is_master_only` option. Now it runs on master correctly.
+
 ## 1.6.0 - 2024-03-25
 
 The release introduces a role for Tarantool 3.0.

roles/expirationd.lua

@@ -248,7 +248,7 @@ local function load_task(task_conf, task_name)
 
     fiber.name(role_name .. ":" .. task_name)
 
-    local skip = task_conf.is_master_only and not box.info.ro
+    local skip = task_conf.is_master_only and box.info.ro
     if skip then
         return
     end

test/unit/tarantool_role_test.lua

@@ -1,4 +1,5 @@
 local expirationd = require('expirationd')
+local fiber = require('fiber')
 local t = require('luatest')
 local helpers = require('test.helper')
 
@@ -551,6 +552,9 @@ end
 function g.test_apply_config_skip_is_master_only(cg)
     local task_name = "apply_config_test"
 
+    t.assert_equals(box.info.ro, false)
+    box.cfg{read_only=true}
+
     cg.role.apply({
         [task_name] = {
             space = g.space.id,
@@ -559,7 +563,10 @@ function g.test_apply_config_skip_is_master_only(cg)
         },
     }, {is_master = false})
 
+    fiber.yield()
     helpers.retrying({}, function()
         t.assert_equals(#expirationd.tasks(), 0)
     end)
+
+    box.cfg{read_only=false}
 end