Skip to content

Commit ee2db2d

Browse files
committed
refactor: remove redundant brick validation from app restart process
1 parent 7a8ad44 commit ee2db2d

File tree

2 files changed

+0
-7
lines changed

2 files changed

+0
-7
lines changed

internal/orchestrator/app/validator_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,6 @@ bricks:
151151
appDescriptor, err := ParseDescriptorFile(appYaml)
152152
require.NoError(t, err)
153153

154-
// Validate the bricks
155154
err = ValidateBricks(appDescriptor, bricksIndex)
156155
if tc.expectedError == nil {
157156
assert.NoError(t, err, "Expected no validation errors")

internal/orchestrator/orchestrator.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -453,12 +453,6 @@ func RestartApp(
453453
ctx, cancel := context.WithCancel(ctx)
454454
defer cancel()
455455

456-
err := app.ValidateBricks(appToStart.Descriptor, bricksIndex)
457-
if err != nil {
458-
yield(StreamMessage{error: err})
459-
return
460-
}
461-
462456
runningApp, err := getRunningApp(ctx, docker.Client())
463457
if err != nil {
464458
yield(StreamMessage{error: err})

0 commit comments

Comments
 (0)