zrhoffman commented on a change in pull request #6531: URL: https://github.com/apache/trafficcontrol/pull/6531#discussion_r803171795
########## File path: traffic_router/ultimate-test-harness/main_test.go ########## @@ -0,0 +1,160 @@ +package ultimate_test_harness + +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, + * software distributed under the License is distributed on an + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY + * KIND, either express or implied. See the License for the + * specific language governing permissions and limitations + * under the License. + */ + +import ( + "flag" + "fmt" + "math/rand" + "net/url" + "os" + "testing" + "time" + + "github.com/apache/trafficcontrol/lib/go-log" + "github.com/apache/trafficcontrol/lib/go-tc" + client "github.com/apache/trafficcontrol/traffic_ops/v4-client" +) + +type Benchmark struct { + RequestsPerSecondThreshold int + BenchmarkSeconds int + ThreadCount int + DSType tc.Type + TrafficRouters []TRDetails + CoverageZoneLocation string +} + +var IPv4Only *bool +var IPv6Only *bool +var CDNName *string +var DeliveryServiceName *string +var TrafficRouterName *string +var UseCoverageZone *bool +var CoverageZoneLocation *string +var HTTPRequestsPerSecondThreshold *int +var DNSRequestsPerSecondThreshold *int +var BenchTime *int +var ThreadCount *int + +func init() { + rand.Seed(time.Now().UnixNano()) + IPv4Only = flag.Bool("4", false, "test IPv4 addresses only") + IPv6Only = flag.Bool("6", false, "test IPv4 addresses only") + CDNName = flag.String("cdn", "", "the name of a CDN to search for Delivery Services") + DeliveryServiceName = flag.String("ds", "", "the name (XMLID) of a Delivery Service to use for tests") + TrafficRouterName = flag.String("hostname", "", "the hostname of a Traffic Router to use") + UseCoverageZone = flag.Bool("coverage_zone", false, "whether to use an IP address from the Traffic Router's Coverage Zone File") + CoverageZoneLocation = flag.String("coverage_zone_location", "", "the Coverage Zone location to use (implies coverage_zone=true)") + HTTPRequestsPerSecondThreshold = flag.Int("http_requests_threshold", 8000, "the minimum number of HTTP requests per second a Traffic Router must successfully respond to") + DNSRequestsPerSecondThreshold = flag.Int("dns_requests_threshold", 25000, "the minimum number of DNS requests per second a Traffic Router must successfully respond to") + BenchTime = flag.Int("benchmark_time", 10, "the duration of each load test in seconds") + ThreadCount = flag.Int("thread_count", 12, "the number of threads to use for each test") + + log.Init(os.Stderr, os.Stderr, os.Stderr, os.Stderr, os.Stderr) +} + +func GetTrafficRouterDetails(t *testing.T) []TRDetails { + var err error + var trafficRouterDetails []TRDetails + + ipAddressMaps := map[tc.CDNName]IPAddressMap{} + + trafficRouters, err := getTrafficRouters(*TrafficRouterName, tc.CDNName(*CDNName)) + if err != nil { + t.Fatalf("could not get Traffic Routers: %s", err.Error()) + } + + for _, trafficRouter := range trafficRouters { + var ipAddresses []string + for _, serverInterface := range trafficRouter.Interfaces { + if !serverInterface.Monitor { + log.Warnf("skipping server interface %s of Traffic Router %s because it is unmonitored", serverInterface.Name, *trafficRouter.HostName) + continue + } + ipv4, ipv6 := serverInterface.GetDefaultAddress() + if ipv4 != "" && !*IPv6Only { + ipAddresses = append(ipAddresses, ipv4) + } + if ipv6 != "" && !*IPv4Only { + ipAddresses = append(ipAddresses, "["+ipv6+"]") + } + } + if len(ipAddresses) < 1 { + log.Warnf("need at least 1 monitored service address on an interface of Traffic Router '%s' to use it for benchmarks, but %d such addresses were found", *trafficRouter.HostName, len(ipAddresses)) + continue + } + dsTypeName := tc.DSTypeHTTP + httpDSes := getDSes(t, *trafficRouter.CDNID, dsTypeName, tc.DeliveryServiceName(*DeliveryServiceName)) + if len(httpDSes) < 1 { + log.Warnf("at least 1 Delivery Service with type '%s' is required to run HTTP load tests on Traffic Router '%s', but %d were found", dsTypeName, *trafficRouter.HostName, len(httpDSes)) + } + dsURL, err := url.Parse(httpDSes[0].ExampleURLs[0]) Review comment: > This will panic if httpDSes is empty, and since that line is just a `warn`, it won't exit. `continue`ing if `httpDSes` is empty in 29bb35cd10 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
