From: Ward Vandewege <ward@curii.com>
Date: Tue, 9 Feb 2021 20:14:10 +0000 (-0500)
Subject: 17355: keepstore should take the volume's AccessViaHosts ReadOnly flag
X-Git-Tag: 2.2.0~84^2~61
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/7df49ca80b5ed10c9efc644de5b07ff3eb1829c8

17355: keepstore should take the volume's AccessViaHosts ReadOnly flag
       into account when reporting ReadOnly status of a volume on
       startup.

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <ward@curii.com>
---

diff --git a/services/keepstore/volume.go b/services/keepstore/volume.go
index 4d8a0aec7a..26e6b73182 100644
--- a/services/keepstore/volume.go
+++ b/services/keepstore/volume.go
@@ -315,7 +315,7 @@ func makeRRVolumeManager(logger logrus.FieldLogger, cluster *arvados.Cluster, my
 		if err != nil {
 			return nil, fmt.Errorf("error initializing volume %s: %s", uuid, err)
 		}
-		logger.Printf("started volume %s (%s), ReadOnly=%v", uuid, vol, cfgvol.ReadOnly)
+		logger.Printf("started volume %s (%s), ReadOnly=%v", uuid, vol, cfgvol.ReadOnly || va.ReadOnly)
 
 		sc := cfgvol.StorageClasses
 		if len(sc) == 0 {