fix: add missing default service imports on reload

Signed-off-by: Waldemar Quevedo <wally@nats.io>

server/reload_test.go

@@ -2719,15 +2719,40 @@ func TestConfigReloadAccountWithNoChanges(t *testing.T) {
 		}
 	}
 	`))
-	s, _ := RunServerWithConfig(conf)
+	s, opts := RunServerWithConfig(conf)
 	defer s.Shutdown()
+
+	ncA, err := nats.Connect(fmt.Sprintf("nats://a:@%s:%d", opts.Host, opts.Port))
+	if err != nil {
+		t.Fatalf("Error on connect: %v", err)
+	}
+	defer ncA.Close()
+
+	// Confirm service imports are ok.
+	resp, err := ncA.Request("$SYS.REQ.ACCOUNT.PING.CONNZ", nil, time.Second)
+	if err != nil {
+		t.Error(err)
+	}
+	if resp == nil || !strings.Contains(string(resp.Data), `"num_connections":1`) {
+		t.Fatal("unexpected data in connz response")
+	}
+
 	before := s.NumSubscriptions()
 	s.Reload()
 	after := s.NumSubscriptions()
 	if before != after {
 		t.Errorf("Number of subscriptions changed after reload: %d -> %d", before, after)
 	}
 
+	// Confirm this still works...
+	resp, err = ncA.Request("$SYS.REQ.ACCOUNT.PING.CONNZ", nil, time.Second)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if resp == nil || !strings.Contains(string(resp.Data), `"num_connections":1`) {
+		t.Fatal("unexpected data in connz response")
+	}
+
 	before = s.NumSubscriptions()
 	s.Reload()
 	after = s.NumSubscriptions()

server/server.go

@@ -900,6 +900,9 @@ func (s *Server) configureAccounts(reloading bool) (map[string]struct{}, error)
 			c.processUnsub(sid)
 		}
 		acc.addAllServiceImportSubs()
+		s.mu.Unlock()
+		s.registerSystemImports(acc)
+		s.mu.Lock()
 	}
 
 	// Set the system account if it was configured.