mirror of
https://github.com/mattermost/mattermost.git
synced 2026-09-24 16:05:00 +08:00
[MM-67677] Adjust max autotranslations workers to 64 (#35463)
This commit is contained in:
+1
-1
@@ -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",
|
||||
|
||||
@@ -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)
|
||||
}
|
||||
|
||||
|
||||
@@ -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",
|
||||
|
||||
Reference in New Issue
Block a user