From e296a314bb93a318b66aec81353776b7d95aa04a Mon Sep 17 00:00:00 2001 From: Ben Cooke Date: Wed, 4 Mar 2026 16:29:40 -0500 Subject: [PATCH] [MM-67677] Adjust max autotranslations workers to 64 (#35463) --- server/i18n/en.json | 2 +- server/public/model/config.go | 4 ++-- server/public/model/config_test.go | 27 +++++++++++++++++++++++++++ 3 files changed, 30 insertions(+), 3 deletions(-) diff --git a/server/i18n/en.json b/server/i18n/en.json index efcca7a8dac..ec29cf95ec1 100644 --- a/server/i18n/en.json +++ b/server/i18n/en.json @@ -10030,7 +10030,7 @@ }, { "id": "model.config.is_valid.autotranslation.workers.app_error", - "translation": "Workers must be between 1 and 32." + "translation": "Workers must be between 1 and 64." }, { "id": "model.config.is_valid.cache_type.app_error", diff --git a/server/public/model/config.go b/server/public/model/config.go index c0682628fdf..fcc5876ce53 100644 --- a/server/public/model/config.go +++ b/server/public/model/config.go @@ -4850,8 +4850,8 @@ func (s *AutoTranslationSettings) isValid() *AppError { return NewAppError("Config.IsValid", "model.config.is_valid.autotranslation.timeout.app_error", nil, "", http.StatusBadRequest) } - // Validate workers if set (must be between 1 and 32) - if s.Workers != nil && (*s.Workers < 1 || *s.Workers > 32) { + // Validate workers if set (must be between 1 and 64) + if s.Workers != nil && (*s.Workers < 1 || *s.Workers > 64) { return NewAppError("Config.IsValid", "model.config.is_valid.autotranslation.workers.app_error", nil, "", http.StatusBadRequest) } diff --git a/server/public/model/config_test.go b/server/public/model/config_test.go index aba9f273386..112e38d30f0 100644 --- a/server/public/model/config_test.go +++ b/server/public/model/config_test.go @@ -2787,6 +2787,33 @@ func TestAutoTranslationSettingsIsValid(t *testing.T) { }, expectError: false, }, + { + name: "valid workers at 48", + settings: AutoTranslationSettings{ + Enable: NewPointer(true), + Provider: NewPointer("libretranslate"), + Workers: NewPointer(48), + LibreTranslate: &LibreTranslateProviderSettings{ + URL: NewPointer("https://lt.example.com"), + APIKey: NewPointer("optional-key"), + }, + }, + expectError: false, + }, + { + name: "invalid workers above 64", + errorId: "model.config.is_valid.autotranslation.workers.app_error", + settings: AutoTranslationSettings{ + Enable: NewPointer(true), + Provider: NewPointer("libretranslate"), + Workers: NewPointer(65), + LibreTranslate: &LibreTranslateProviderSettings{ + URL: NewPointer("https://lt.example.com"), + APIKey: NewPointer("optional-key"), + }, + }, + expectError: true, + }, // TODO: Enable Agents provider in future release // { // name: "valid agents settings",