From 5f648406b0f2e322851fce36b062a298cb3d6736 Mon Sep 17 00:00:00 2001 From: David Teigland Date: Thu, 2 Aug 2018 11:26:59 -0500 Subject: [PATCH] mirrors: fix read_only_volume_list If a mirror LV is listed in read_only_volume_list, it would still be activated rw. The activation would initially be readonly, but the monitoring function would immediately change it to rw. This was a regression from commit fade45b1d14c mirror: improve table update The monitoring function needs to copy the read_only setting into the new set of mirror activation options it uses. --- lib/activate/activate.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/activate/activate.c b/lib/activate/activate.c index 56ec73241..70cce30cb 100644 --- a/lib/activate/activate.c +++ b/lib/activate/activate.c @@ -1851,6 +1851,8 @@ int monitor_dev_for_events(struct cmd_context *cmd, const struct logical_volume if (!laopts) laopts = &zlaopts; + else + mirr_laopts.read_only = laopts->read_only; /* skip dmeventd code altogether */ if (dmeventd_monitor_mode() == DMEVENTD_MONITOR_IGNORE)