Skip to content

Commit

Permalink
Add fix for non-default worker pool size
Browse files Browse the repository at this point in the history
  • Loading branch information
sneal committed May 3, 2023
1 parent a59e110 commit 8ec2304
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
1 change: 1 addition & 0 deletions pkg/migrate/foundation.go
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,7 @@ func NewFoundationMigratorFromConfig(c config.Config) (*FoundationMigrator, erro
l.Debug("Creating foundation migrator")
fm := NewFoundationMigrator(clientPool, boshClient, vmMigrator, srcAZsToClusters, out)
fm.AdditionalVMs = ConfigToAdditionalVMs(c)
fm.WorkerCount = c.WorkerPoolSize
return fm, nil
}

Expand Down
8 changes: 7 additions & 1 deletion pkg/migrate/foundation_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,14 @@ func baseConfig() config.Config {
}

func TestNewFoundationMigratorFromConfig(t *testing.T) {
_, err := migrate.NewFoundationMigratorFromConfig(baseConfig())
fm, err := migrate.NewFoundationMigratorFromConfig(baseConfig())
require.NoError(t, err)
require.NotNil(t, fm)
require.Equal(t, 1, fm.WorkerCount)
require.Equal(t, []bosh.VM{{
Name: "additional-vm1",
AZ: "az1",
}}, fm.AdditionalVMs)
}

func TestConfigToSourceClustersByAZ(t *testing.T) {
Expand Down

0 comments on commit 8ec2304

Please sign in to comment.