actually still need legacyhandler

lyft · Aug 13, 2024 · 5d92995 · 5d92995
1 parent eb5a0d4
commit 5d92995
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/server/legacy/lyft/gateway/events_controller.go b/server/legacy/lyft/gateway/events_controller.go
@@ -60,9 +60,13 @@ func NewVCSEventsController(
 	clientCreator githubapp.ClientCreator,
 	defaultTFVersion string,
 ) *VCSEventsController {
+	legacyHandler := &gateway_handlers.LegacyPullHandler{
+		Logger:           logger,
+		VCSStatusUpdater: vcsStatusUpdater,
+	}
 	prSignaler := &pr.WorkflowSignaler{TemporalClient: temporalClient, DefaultTFVersion: defaultTFVersion}
 	prRequirementChecker := requirement.NewPRAggregate(globalCfg)
-	modifiedPullHandler := gateway_handlers.NewModifiedPullHandler(logger, asyncScheduler, rootConfigBuilder, globalCfg, prRequirementChecker, prSignaler)
+	modifiedPullHandler := gateway_handlers.NewModifiedPullHandler(logger, asyncScheduler, rootConfigBuilder, globalCfg, prRequirementChecker, prSignaler, legacyHandler)
 	closedPullHandler := &gateway_handlers.ClosedPullRequestHandler{
 		Logger:          logger,
 		PRCloseSignaler: prSignaler,

diff --git a/server/neptune/gateway/event/modified_pull_request_handler.go b/server/neptune/gateway/event/modified_pull_request_handler.go
@@ -19,6 +19,10 @@ import (
 	"github.com/runatlantis/atlantis/server/models"
 )
 
+type legacyHandler interface {
+	Handle(ctx context.Context, request *http.BufferedRequest, event PullRequest, allRoots []*valid.MergedProjectCfg) error
+}
+
 type prSignaler interface {
 	SignalWithStartWorkflow(ctx context.Context, rootCfgs []*valid.MergedProjectCfg, prRequest pr.Request) (client.WorkflowRun, error)
 }
@@ -29,6 +33,7 @@ type ModifiedPullHandler struct {
 	RootConfigBuilder  rootConfigBuilder
 	GlobalCfg          valid.GlobalCfg
 	RequirementChecker requirementChecker
+	LegacyHandler      legacyHandler
 	PRSignaler         prSignaler
 }
 
@@ -41,13 +46,14 @@ type PullRequest struct {
 	InstallationToken int64
 }
 
-func NewModifiedPullHandler(logger logging.Logger, scheduler scheduler, rootConfigBuilder rootConfigBuilder, globalCfg valid.GlobalCfg, requirementChecker requirementChecker, prSignaler prSignaler) *ModifiedPullHandler {
+func NewModifiedPullHandler(logger logging.Logger, scheduler scheduler, rootConfigBuilder rootConfigBuilder, globalCfg valid.GlobalCfg, requirementChecker requirementChecker, prSignaler prSignaler, legacyHandler legacyHandler) *ModifiedPullHandler {
 	return &ModifiedPullHandler{
 		Logger:             logger,
 		Scheduler:          scheduler,
 		RootConfigBuilder:  rootConfigBuilder,
 		GlobalCfg:          globalCfg,
 		RequirementChecker: requirementChecker,
+		LegacyHandler:      legacyHandler,
 		PRSignaler:         prSignaler,
 	}
 }
@@ -97,6 +103,7 @@ func (p *ModifiedPullHandler) handle(ctx context.Context, request *http.Buffered
 	}
 
 	fxns := []func(ctx context.Context, request *http.BufferedRequest, event PullRequest, allRoots []*valid.MergedProjectCfg) error{
+		p.LegacyHandler.Handle,
 		p.handlePlatformMode,
 	}
 	var combinedErrors *multierror.Error