Skip to content

Commit d1e4394

Browse files
authored
Retention config shouldn't need logstream to be initialized (#398)
It is common for users to create stream and set the retention (and be done with initial config) before sending logs to the log stream. Accordingly we're going to add support in Helm chart to declaratively create logstream and set retention. To allow that setup we need to ensure log stream retention can be set before log stream has events (i.e. initialised) Prerequisite for #397
1 parent 8cd8cce commit d1e4394

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

server/src/handlers/http/logstream.rs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -212,10 +212,6 @@ pub async fn put_retention(
212212
Err(err) => return Err(StreamError::InvalidRetentionConfig(err)),
213213
};
214214

215-
if !STREAM_INFO.stream_initialized(&stream_name)? {
216-
return Err(StreamError::UninitializedLogstream);
217-
}
218-
219215
CONFIG
220216
.storage()
221217
.get_object_store()

0 commit comments

Comments
 (0)