Skip to content

Commit b9359a3

Browse files
athanatosleonidc
authored andcommitted
include/ceph_features: add NVMEOFHA feature bit
Normally, we'd just use the SERVER_SQUID or SERVER_T flags instead of using an extra feature bit. However, the nvmeof ha monitor paxos service has had a more complex development journey. There are users interested in using the nvmeof ha feature in squid, but it didn't make the cutoff for backporting it. There's an upstream nvmeof-squid branch in the ceph.git repository with the patches backported for anyone interested in building it. However, that means that users of our normal stable releases will see the feature added to the monitor one release after anyone who chooses to use the nvmeof-squid branch. We could disallow upgrades from nvmeof-squid to T, but by adding a feature bit here we make such a restriction unnecessary. Signed-off-by: Samuel Just <[email protected]>
1 parent 337269d commit b9359a3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/include/ceph_features.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,7 @@ DEFINE_CEPH_FEATURE(41, 1, CRUSH_TUNABLES3) // 3.15
144144
DEFINE_CEPH_FEATURE(41, 1, OSD_PRIMARY_AFFINITY) // overlap
145145
DEFINE_CEPH_FEATURE(42, 1, MSGR_KEEPALIVE2) // 4.3 (for consistency)
146146
DEFINE_CEPH_FEATURE(43, 1, OSD_POOLRESEND) // 4.13
147-
DEFINE_CEPH_FEATURE_RETIRED(44, 1, ERASURE_CODE_PLUGINS_V2, MIMIC, OCTOPUS)
148-
// available
147+
DEFINE_CEPH_FEATURE(44, 2, NVMEOFHA)
149148
DEFINE_CEPH_FEATURE_RETIRED(45, 1, OSD_SET_ALLOC_HINT, JEWEL, LUMINOUS)
150149
// available
151150
DEFINE_CEPH_FEATURE(46, 1, OSD_FADVISE_FLAGS)
@@ -224,6 +223,7 @@ DEFINE_CEPH_FEATURE_RETIRED(63, 1, RESERVED_BROKEN, LUMINOUS, QUINCY) // client-
224223
CEPH_FEATURE_OSD_PRIMARY_AFFINITY | \
225224
CEPH_FEATURE_MSGR_KEEPALIVE2 | \
226225
CEPH_FEATURE_OSD_POOLRESEND | \
226+
CEPH_FEATUREMASK_NVMEOFHA | \
227227
CEPH_FEATURE_OSD_FADVISE_FLAGS | \
228228
CEPH_FEATURE_MDS_QUOTA | \
229229
CEPH_FEATURE_CRUSH_V4 | \

0 commit comments

Comments
 (0)