Add Go 1.24 to the grpc, http2 and go tls trace BPF tests

Signed-off-by: Dom Del Nano <ddelnano@gmail.com>

Author: ddelnano

bazel/pl_build_system.bzl

@@ -23,8 +23,8 @@ load("@rules_cc//cc:defs.bzl", "cc_binary", "cc_library", "cc_test")
 load("@rules_python//python:defs.bzl", "py_test")
 load("//bazel:toolchain_transitions.bzl", "qemu_interactive_runner")
 
-pl_boringcrypto_go_sdk = ["1.23.5"]
-pl_supported_go_sdk_versions = ["1.18", "1.19", "1.20", "1.21", "1.22", "1.23"]
+pl_boringcrypto_go_sdk = ["1.23.9"]
+pl_supported_go_sdk_versions = ["1.18", "1.19", "1.20", "1.21", "1.22", "1.23", "1.24"]
 
 # The last version in this list corresponds to the boringcrypto go sdk version.
 pl_all_supported_go_sdk_versions = pl_supported_go_sdk_versions + pl_boringcrypto_go_sdk

src/stirling/source_connectors/socket_tracer/BUILD.bazel

@@ -310,13 +310,15 @@ pl_cc_bpf_test(
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_client:golang_1_21_grpc_client",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_client:golang_1_22_grpc_client",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_client:golang_1_23_grpc_client",
+        "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_client:golang_1_24_grpc_client",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_client:golang_boringcrypto_grpc_client",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_18_grpc_server_with_certs",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_19_grpc_server_with_certs",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_20_grpc_server_with_certs",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_21_grpc_server_with_certs",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_22_grpc_server_with_certs",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_23_grpc_server_with_certs",
+        "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_1_24_grpc_server_with_certs",
         "//src/stirling/source_connectors/socket_tracer/protocols/http2/testing/go_grpc_server:golang_boringcrypto_grpc_server_with_certs",
     ],
     flaky = True,
@@ -364,6 +366,8 @@ pl_cc_bpf_test(
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_22_grpc_server_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_23_grpc_client_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_23_grpc_server_container",
+        "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_24_grpc_client_container",
+        "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_24_grpc_server_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_boringcrypto_grpc_client_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_boringcrypto_grpc_server_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:product_catalog_client_container",
@@ -586,6 +590,8 @@ pl_cc_bpf_test(
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_22_tls_server_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_23_tls_client_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_23_tls_server_container",
+        "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_24_tls_client_container",
+        "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_1_24_tls_server_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_boringcrypto_tls_client_container",
         "//src/stirling/source_connectors/socket_tracer/testing/container_images:go_boringcrypto_tls_server_container",
         "//src/stirling/testing:cc_library",

src/stirling/source_connectors/socket_tracer/go_tls_trace_bpf_test.cc

@@ -32,6 +32,8 @@
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_22_tls_server_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_23_tls_client_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_23_tls_server_container.h"
+#include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_tls_client_container.h"
+#include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_tls_server_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_boringcrypto_tls_client_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_boringcrypto_tls_server_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/protocol_checkers.h"
@@ -101,6 +103,11 @@ struct Go1_23TLSClientServerContainers {
   using GoTLSClientContainer = ::px::stirling::testing::Go1_23_TLSClientContainer;
 };
 
+struct Go1_24TLSClientServerContainers {
+  using GoTLSServerContainer = ::px::stirling::testing::Go1_24_TLSServerContainer;
+  using GoTLSClientContainer = ::px::stirling::testing::Go1_24_TLSClientContainer;
+};
+
 struct GoBoringCryptoTLSClientServerContainers {
   using GoTLSServerContainer = ::px::stirling::testing::GoBoringCryptoTLSServerContainer;
   using GoTLSClientContainer = ::px::stirling::testing::GoBoringCryptoTLSClientContainer;
@@ -109,7 +116,7 @@ struct GoBoringCryptoTLSClientServerContainers {
 typedef ::testing::Types<GoBoringCryptoTLSClientServerContainers, Go1_18TLSClientServerContainers,
                          Go1_19TLSClientServerContainers, Go1_20TLSClientServerContainers,
                          Go1_21TLSClientServerContainers, Go1_22TLSClientServerContainers,
-                         Go1_23TLSClientServerContainers>
+                         Go1_23TLSClientServerContainers, Go1_24TLSClientServerContainers>
     GoVersions;
 TYPED_TEST_SUITE(GoTLSTraceTest, GoVersions);
 

src/stirling/source_connectors/socket_tracer/grpc_trace_bpf_test.cc

@@ -196,6 +196,7 @@ INSTANTIATE_TEST_SUITE_P(SecurityModeTest, GRPCTraceTest,
                              TestParams{"1_21", true, true}, TestParams{"1_21", true, false},
                              TestParams{"1_22", true, true}, TestParams{"1_22", true, false},
                              TestParams{"1_23", true, true}, TestParams{"1_23", true, false},
+                             TestParams{"1_24", true, true}, TestParams{"1_24", true, false},
                              TestParams{"boringcrypto", true, true}));
 
 class PyGRPCTraceTest : public testing::SocketTraceBPFTestFixture</* TClientSideTracing */ false> {

src/stirling/source_connectors/socket_tracer/http2_trace_bpf_test.cc

@@ -33,6 +33,8 @@
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_22_grpc_server_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_23_grpc_client_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_23_grpc_server_container.h"
+#include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_grpc_client_container.h"
+#include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_grpc_server_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_boringcrypto_grpc_client_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/go_boringcrypto_grpc_server_container.h"
 #include "src/stirling/source_connectors/socket_tracer/testing/container_images/product_catalog_client_container.h"
@@ -105,6 +107,11 @@ struct Go1_23GRPCClientServerContainers {
   using ClientContainer = ::px::stirling::testing::Go1_23_GRPCClientContainer;
 };
 
+struct Go1_24GRPCClientServerContainers {
+  using ServerContainer = ::px::stirling::testing::Go1_24_GRPCServerContainer;
+  using ClientContainer = ::px::stirling::testing::Go1_24_GRPCClientContainer;
+};
+
 struct GoBoringCryptoGRPCClientServerContainers {
   using ServerContainer = ::px::stirling::testing::GoBoringCryptoGRPCServerContainer;
   using ClientContainer = ::px::stirling::testing::GoBoringCryptoGRPCClientContainer;
@@ -113,7 +120,7 @@ struct GoBoringCryptoGRPCClientServerContainers {
 typedef ::testing::Types<GoBoringCryptoGRPCClientServerContainers, Go1_18GRPCClientServerContainers,
                          Go1_19GRPCClientServerContainers, Go1_20GRPCClientServerContainers,
                          Go1_21GRPCClientServerContainers, Go1_22GRPCClientServerContainers,
-                         Go1_23GRPCClientServerContainers>
+                         Go1_23GRPCClientServerContainers, Go1_24GRPCClientServerContainers>
     GoVersions;
 TYPED_TEST_SUITE(HTTP2TraceTest, GoVersions);
 

src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_grpc_client_container.h

@@ -0,0 +1,45 @@
+/*
+ * Copyright 2018- The Pixie Authors.
+ *
+ * Licensed 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.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#pragma once
+
+#include <string>
+
+#include "src/common/testing/test_environment.h"
+#include "src/common/testing/test_utils/container_runner.h"
+
+namespace px {
+namespace stirling {
+namespace testing {
+
+class Go1_24_GRPCClientContainer : public ContainerRunner {
+ public:
+  Go1_24_GRPCClientContainer()
+      : ContainerRunner(::px::testing::BazelRunfilePath(kBazelImageTar), kContainerNamePrefix,
+                        kReadyMessage) {}
+
+ private:
+  static constexpr std::string_view kBazelImageTar =
+      "src/stirling/testing/demo_apps/go_grpc_tls_pl/client/golang_1_24_grpc_tls_client.tar";
+  static constexpr std::string_view kContainerNamePrefix = "grpc_client";
+  static constexpr std::string_view kReadyMessage = "";
+};
+
+}  // namespace testing
+}  // namespace stirling
+}  // namespace px

src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_grpc_server_container.h

@@ -0,0 +1,46 @@
+/*
+ * Copyright 2018- The Pixie Authors.
+ *
+ * Licensed 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.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#pragma once
+
+#include <string>
+
+#include "src/common/testing/test_environment.h"
+#include "src/common/testing/test_utils/container_runner.h"
+
+namespace px {
+namespace stirling {
+namespace testing {
+
+class Go1_24_GRPCServerContainer : public ContainerRunner {
+ public:
+  Go1_24_GRPCServerContainer()
+      : ContainerRunner(::px::testing::BazelRunfilePath(kBazelImageTar), kContainerNamePrefix,
+                        kReadyMessage) {}
+
+  static constexpr std::string_view kBazelImageTar =
+      "src/stirling/testing/demo_apps/go_grpc_tls_pl/server/golang_1_24_grpc_tls_server.tar";
+
+ private:
+  static constexpr std::string_view kContainerNamePrefix = "grpc_server";
+  static constexpr std::string_view kReadyMessage = "Starting HTTP/2 server";
+};
+
+}  // namespace testing
+}  // namespace stirling
+}  // namespace px

src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_tls_client_container.h

@@ -0,0 +1,45 @@
+/*
+ * Copyright 2018- The Pixie Authors.
+ *
+ * Licensed 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.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#pragma once
+
+#include <string>
+
+#include "src/common/testing/test_environment.h"
+#include "src/common/testing/test_utils/container_runner.h"
+
+namespace px {
+namespace stirling {
+namespace testing {
+
+class Go1_24_TLSClientContainer : public ContainerRunner {
+ public:
+  Go1_24_TLSClientContainer()
+      : ContainerRunner(::px::testing::BazelRunfilePath(kBazelImageTar), kContainerNamePrefix,
+                        kReadyMessage) {}
+
+ private:
+  static constexpr std::string_view kBazelImageTar =
+      "src/stirling/testing/demo_apps/go_https/client/golang_1_24_https_client.tar";
+  static constexpr std::string_view kContainerNamePrefix = "https_client";
+  static constexpr std::string_view kReadyMessage = R"({"status":"ok"})";
+};
+
+}  // namespace testing
+}  // namespace stirling
+}  // namespace px

src/stirling/source_connectors/socket_tracer/testing/container_images/go_1_24_tls_server_container.h

@@ -0,0 +1,46 @@
+/*
+ * Copyright 2018- The Pixie Authors.
+ *
+ * Licensed 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.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#pragma once
+
+#include <string>
+
+#include "src/common/testing/test_environment.h"
+#include "src/common/testing/test_utils/container_runner.h"
+
+namespace px {
+namespace stirling {
+namespace testing {
+
+class Go1_24_TLSServerContainer : public ContainerRunner {
+ public:
+  Go1_24_TLSServerContainer()
+      : ContainerRunner(::px::testing::BazelRunfilePath(kBazelImageTar), kContainerNamePrefix,
+                        kReadyMessage) {}
+
+  static constexpr std::string_view kBazelImageTar =
+      "src/stirling/testing/demo_apps/go_https/server/golang_1_24_https_server.tar";
+
+ private:
+  static constexpr std::string_view kContainerNamePrefix = "https_server";
+  static constexpr std::string_view kReadyMessage = "Starting HTTPS service";
+};
+
+}  // namespace testing
+}  // namespace stirling
+}  // namespace px