Skip to content

Commit

Permalink
Merge pull request #48 from Comcast/sns_fix
Browse files Browse the repository at this point in the history
Looks great!
  • Loading branch information
johnabass authored Aug 25, 2017
2 parents 4f7ab9d + db2c57d commit 0681693
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 20 deletions.
4 changes: 2 additions & 2 deletions src/caduceus/caduceus.go
Original file line number Diff line number Diff line change
Expand Up @@ -204,12 +204,12 @@ func caduceus(arguments []string) int {
Host: v.GetString("fqdn") + v.GetString("primary.address"),
}

webhookFactory.Initialize(mux, selfURL, webhookHandler, logger)
webhookFactory.Initialize(mux, selfURL, webhookHandler, logger, nil)

caduceusHealth := &CaduceusHealth{}
var runnable concurrent.Runnable

caduceusHealth.Monitor, runnable = webPA.Prepare(logger, mux)
caduceusHealth.Monitor, runnable = webPA.Prepare(logger, nil, mux)
serverWrapper.caduceusHealth = caduceusHealth

waitGroup, shutdown, err := concurrent.Execute(runnable)
Expand Down
36 changes: 19 additions & 17 deletions src/glide.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion src/glide.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package: .
import:
- package: github.com/Comcast/webpa-common
version: 353718a8846ab788342ab804b68ce39898949bd7
version: 218a5e38b23db86344113b3021b323214d004b81
subpackages:
- concurrent
- handler
Expand Down

0 comments on commit 0681693

Please sign in to comment.