Merge pull request #108 from fly-apps/bug-fix

Remove flag that's no longer being used

Author: davissp14

cmd/event_handler/main.go

@@ -17,10 +17,6 @@ const eventLogFile = "/data/event.log"
 func main() {
 	event := flag.String("event", "", "event type")
 	nodeID := flag.Int("node-id", 0, "the node id")
-	// This might not actually always be the new primary. %p from repmgr is variably the new or
-	// old primary. In the events that we subscribe to it's always either empty or the new primary.
-	// In the future if we subscribe to repmgrd_failover_promote, then we would have to change this
-	// name.
 	success := flag.String("success", "", "success (1) failure (0)")
 	details := flag.String("details", "", "details")
 	flag.Parse()

internal/flypg/repmgr.go

@@ -144,7 +144,7 @@ func (r *RepMgr) setDefaults() {
 		"use_replication_slots":        "yes",
 		"promote_command":              fmt.Sprintf("'repmgr standby promote -f %s --log-to-file'", r.ConfigPath),
 		"follow_command":               fmt.Sprintf("'repmgr standby follow -f %s --log-to-file --upstream-node-id=%%n'", r.ConfigPath),
-		"event_notification_command":   fmt.Sprintf("'/usr/local/bin/event_handler -node-id %%n -event %%e -success %%s -details \"%%d\" -new-node-id \\'%%p\\''"),
+		"event_notification_command":   fmt.Sprintf("'/usr/local/bin/event_handler -node-id %%n -event %%e -success %%s -details \"%%d\"'"),
 		"event_notifications":          "'child_node_disconnect,child_node_reconnect,child_node_new_connect'",
 		"location":                     r.Region,
 		"primary_visibility_consensus": true,