cluster: support serde for allocate_id_reply

Signed-off-by: Noah Watkins <noah@redpanda.com>
redpanda-data · Jun 7, 2022 · a411ebe · a411ebe
1 parent 5445156
commit a411ebe
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/v/cluster/tests/serialization_rt_test.cc b/src/v/cluster/tests/serialization_rt_test.cc
@@ -524,4 +524,7 @@ SEASTAR_THREAD_TEST_CASE(serde_reflection_roundtrip) {
 
     roundtrip_test(
       cluster::allocate_id_request(model::timeout_clock::duration(234234)));
+
+    roundtrip_test(
+      cluster::allocate_id_reply(23433, cluster::errc::invalid_node_operation));
 }
diff --git a/src/v/cluster/types.h b/src/v/cluster/types.h
@@ -54,13 +54,18 @@ struct allocate_id_request
     auto serde_fields() { return std::tie(timeout); }
 };
 
-struct allocate_id_reply {
+struct allocate_id_reply
+  : serde::envelope<allocate_id_reply, serde::version<0>> {
     int64_t id;
     errc ec;
 
+    allocate_id_reply() noexcept = default;
+
     allocate_id_reply(int64_t id, errc ec)
       : id(id)
       , ec(ec) {}
+
+    auto serde_fields() { return std::tie(id, ec); }
 };
 
 enum class tx_errc {