Skip to content

Commit

Permalink
Start controller-runtime manager in single binary (#3547)
Browse files Browse the repository at this point in the history
* starting controller-runtime manager if either webhook or propeller is enabled

Signed-off-by: Daniel Rammer <[email protected]>

* using correct import

Signed-off-by: Daniel Rammer <[email protected]>

* bumping flytepropeller

Signed-off-by: Daniel Rammer <[email protected]>

---------

Signed-off-by: Daniel Rammer <[email protected]>
  • Loading branch information
hamersaw authored Apr 7, 2023
1 parent bc521da commit db9162c
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 3 deletions.
10 changes: 10 additions & 0 deletions cmd/single/start.go
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,16 @@ func startPropeller(ctx context.Context, cfg Propeller) error {
})
}

if !cfg.DisableWebhook || !cfg.Disabled {
g.Go(func() error {
err := propellerEntrypoint.StartControllerManager(childCtx, mgr)
if err != nil {
logger.Fatalf(childCtx, "Failed to start controller manager. Error: %v", err)
}
return err
})
}

return g.Wait()
}

Expand Down
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ go 1.19
require (
github.com/flyteorg/datacatalog v1.0.44
github.com/flyteorg/flyteadmin v1.1.88
github.com/flyteorg/flytepropeller v1.1.76
github.com/flyteorg/flytepropeller v1.1.77
github.com/flyteorg/flytestdlib v1.0.16
github.com/golang/glog v1.0.0
github.com/spf13/cobra v1.4.0
Expand Down
4 changes: 2 additions & 2 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -369,8 +369,8 @@ github.com/flyteorg/flyteidl v1.3.14 h1:o5M0g/r6pXTPu5PEurbYxbQmuOu3hqqsaI2M6uvK
github.com/flyteorg/flyteidl v1.3.14/go.mod h1:Pkt2skI1LiHs/2ZoekBnyPhuGOFMiuul6HHcKGZBsbM=
github.com/flyteorg/flyteplugins v1.0.45 h1:I/N4ehOxX6ln8DivyZ9gayp/UYiBcqoizBbG1hfwIXM=
github.com/flyteorg/flyteplugins v1.0.45/go.mod h1:ztsonku5fKwyxcIg1k69PTiBVjRI6d3nK5DnC+iwx08=
github.com/flyteorg/flytepropeller v1.1.76 h1:EANN/ZH7QE2CIGGargrRZXr1uC09mbjzb4z6N51M1KM=
github.com/flyteorg/flytepropeller v1.1.76/go.mod h1:jAMN8++sdzEwVQ72ulEBlw+lW7rICDhGgTZcV1xdu2Y=
github.com/flyteorg/flytepropeller v1.1.77 h1:n7wi79pAar9zRDhFfL4of0xXHgk6ih1O99gmxK7jp5Q=
github.com/flyteorg/flytepropeller v1.1.77/go.mod h1:jAMN8++sdzEwVQ72ulEBlw+lW7rICDhGgTZcV1xdu2Y=
github.com/flyteorg/flytestdlib v1.0.16 h1:Nv468DHVZMdhzA1WD3a/ef5M59xXIbxdUN4bLX86KWU=
github.com/flyteorg/flytestdlib v1.0.16/go.mod h1:ghw/cjY0sEWIIbyCtcJnL/Gt7ZS7gf9SUi0CCPhbz3s=
github.com/flyteorg/stow v0.3.6 h1:jt50ciM14qhKBaIrB+ppXXY+SXB59FNREFgTJqCyqIk=
Expand Down

0 comments on commit db9162c

Please sign in to comment.