conprof: disable profiling tiflash (#184) (#187)

close #185

component/conprof/http/api.go

@@ -123,7 +123,8 @@ func getProfileEstimateSize(component topology.Component) int {
 	case topology.ComponentTiKV:
 		return 200 * 1024 // profile size
 	case topology.ComponentTiFlash:
-		return 200 * 1024 // profile size
+		// TODO: remove this after TiFlash fix the profile bug.
+		return 0
 	}
 	return defaultProfileSize
 }

component/conprof/scrape/manager.go

@@ -188,6 +188,10 @@ func (m *Manager) startScrape(ctx context.Context, component topology.Component,
 	if !continueProfilingCfg.Enable {
 		return nil
 	}
+	// TODO: remove this after TiFlash fix the profile bug.
+	if component.Name == topology.ComponentTiFlash {
+		return nil
+	}
 	profilingConfig := m.getProfilingConfig(component)
 	httpCfg := m.config.Security.GetHTTPClientConfig()
 	addr := fmt.Sprintf("%v:%v", component.IP, component.Port)

component/conprof/scrape/manager_test.go

@@ -90,6 +90,8 @@ func TestManager(t *testing.T) {
 					break
 				}
 			}
+			// TODO: remove this after support tiflash
+			require.True(t, list.Target.Component != topology.ComponentTiFlash)
 			require.True(t, found, fmt.Sprintf("%#v", list))
 			for _, ts := range list.TsList {
 				require.True(t, ts >= param.Begin && ts <= param.End)
@@ -130,7 +132,8 @@ func TestManager(t *testing.T) {
 
 	// test for GetCurrentScrapeComponents
 	comp := manager.GetCurrentScrapeComponents()
-	require.Equal(t, len(comp), len(components))
+	// TODO: update this after support tiflash
+	require.Equal(t, len(comp), len(components)-1)
 
 	// test for topology changed.
 	mockServer2 := testutil.CreateMockProfileServer(t)