Merge pull request redpanda-data#12498 from andrwng/archival-warn-to-…

…error archival: promote some guardrail warnings to error
joejulian · Aug 10, 2023 · a42f3e5 · a42f3e5
2 parents e77545e + cea1aee
commit a42f3e5
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/v/cluster/archival_metadata_stm.cc b/src/v/cluster/archival_metadata_stm.cc
@@ -1002,7 +1002,7 @@ void archival_metadata_stm::apply_add_segment(const segment& segment) {
     if (!disable_safe_add && !_manifest->safe_segment_meta_to_add(meta)) {
         auto last = _manifest->last_segment();
         vlog(
-          _logger.warn,
+          _logger.error,
           "Can't add segment: {}, previous segment: {}",
           meta,
           last);
@@ -1029,7 +1029,7 @@ void archival_metadata_stm::apply_add_segment(const segment& segment) {
             // the hole.
 
             vlog(
-              _logger.warn,
+              _logger.error,
               "hole in the remote offset range detected! previous last "
               "offset: "
               "{}, new segment base offset: {}",
@@ -1135,7 +1135,7 @@ void archival_metadata_stm::apply_spillover(const spillover_cmd& so) {
           get_start_offset(),
           get_last_offset());
     } else {
-        vlog(_logger.warn, "Can't apply spillover_cmd: {}", so.manifest_meta);
+        vlog(_logger.error, "Can't apply spillover_cmd: {}", so.manifest_meta);
     }
 }
 
@@ -1175,7 +1175,7 @@ archival_metadata_stm::get_segments_to_cleanup() const {
             // manifest in S3 so if we will delete it the data will be lost.
             if (m_name == s_name) {
                 vlog(
-                  _logger.warn,
+                  _logger.error,
                   "The replaced segment name {} collides with the segment {} "
                   "in the manifest. It will be removed to prevent the data "
                   "loss.",