Upgrade rules_scala, scala and finagle versions

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

Author: ddelnano

WORKSPACE

@@ -23,6 +23,10 @@ load("@bazel_features//:deps.bzl", "bazel_features_deps")
 
 bazel_features_deps()
 
+load("@rules_cc//cc:extensions.bzl", "compatibility_proxy_repo")
+
+compatibility_proxy_repo()
+
 # Order is important. Try to go from most basic/primitive to higher level packages.
 # - go_rules_dependencies
 # - protobuf_deps
@@ -49,6 +53,10 @@ go_register_toolchains()
 # gazelle:repository_macro go_deps.bzl%pl_go_dependencies
 pl_go_dependencies()
 
+load("@rules_java//java:rules_java_deps.bzl", "rules_java_dependencies")
+
+rules_java_dependencies()
+
 load("@com_google_protobuf//:protobuf_deps.bzl", "protobuf_deps")
 
 protobuf_deps()
@@ -57,17 +65,34 @@ load("@com_github_grpc_grpc//bazel:grpc_deps.bzl", "grpc_deps")
 
 grpc_deps()
 
-load("@io_bazel_rules_scala//:scala_config.bzl", "scala_config")
+load("@io_bazel_rules_scala//scala:deps.bzl", "rules_scala_dependencies")
 
-scala_version = "2.13.6"
+rules_scala_dependencies()
 
-scala_config(scala_version = scala_version)
+load("@io_bazel_rules_scala//:scala_config.bzl", "scala_config")
 
-load("@io_bazel_rules_scala//scala:scala.bzl", "scala_repositories")
+scala_version = "2.13.16"
 
-scala_repositories()
+scala_config(scala_version = scala_version)
 
-load("@io_bazel_rules_scala//scala:toolchains.bzl", "scala_register_toolchains")
+load("@io_bazel_rules_scala//scala:toolchains.bzl", "scala_register_toolchains", "scala_toolchains")
+
+scala_toolchains(
+    fetch_sources = True,
+    twitter_scrooge = {
+        "libthrift": "@thrift_deps//:org_apache_thrift_libthrift",
+        # Use scrooge_core_with_finagle to include finagle on the compile classpath
+        # for generated thrift service code. Must use @px// prefix to reference
+        # the main workspace from within the generated @rules_scala_toolchains repo.
+        "scrooge_core": "@px//src/stirling/source_connectors/socket_tracer/testing/containers/thriftmux:scrooge_core_with_finagle",
+        "scrooge_generator": "@thrift_deps//:com_twitter_scrooge_generator_2_13",
+        "util_core": "@thrift_deps//:com_twitter_util_core_2_13",
+        "util_logging": "@thrift_deps//:com_twitter_util_logging_2_13",
+        "javax_annotation_api": "@thrift_deps//:javax_annotation_javax_annotation_api",
+        "mustache": "@thrift_deps//:com_github_spullara_mustache_java_compiler",
+        "scopt": "@thrift_deps//:com_github_scopt_scopt_2_13",
+    },
+)
 
 scala_register_toolchains()
 
@@ -187,26 +212,6 @@ load("@thrift_deps//:defs.bzl", thrift_pinned_maven_install = "pinned_maven_inst
 
 thrift_pinned_maven_install()
 
-# twitter_scrooge will use incompatible versions of @scrooge_jars and @thrift_jars.
-# These bind statements ensure that the correct versions of finagle libthrift, scrooge core
-# and scrooge generator are used to ensure successful compilation.
-# See https://github.com/bazelbuild/rules_scala/issues/592 and
-# https://github.com/bazelbuild/rules_scala/pull/847 for more details.
-bind(
-    name = "io_bazel_rules_scala/dependency/thrift/scrooge_core",
-    actual = "//src/stirling/source_connectors/socket_tracer/testing/containers/thriftmux:scrooge_jars",
-)
-
-bind(
-    name = "io_bazel_rules_scala/dependency/thrift/scrooge_generator",
-    actual = "//src/stirling/source_connectors/socket_tracer/testing/containers/thriftmux:scrooge_jars",
-)
-
-bind(
-    name = "io_bazel_rules_scala/dependency/thrift/libthrift",
-    actual = "//src/stirling/source_connectors/socket_tracer/testing/containers/thriftmux:thrift_jars",
-)
-
 # gazelle:repo bazel_gazelle
 # Gazelle depes need to be loaded last to make sure they don't override our dependencies.
 # The first one wins when it comes to package declaration.

bazel/repository_locations.bzl

@@ -431,11 +431,11 @@ REPOSITORY_LOCATIONS = dict(
         ],
     ),
     io_bazel_rules_scala = dict(
-        sha256 = "6e9191363357d30b144e7306fec74deea2c7f1de63f3ed32028838116c239e8a",
+        sha256 = "6a900a39bf643c5c9308c8200f9a61e27588dc54a67aa08fcfc798f25add4f7b",
         urls = [
-            "https://github.com/bazelbuild/rules_scala/archive/4ba3780fcba8d26980daff4639abc6f18517308b.tar.gz",
+            "https://github.com/bazelbuild/rules_scala/releases/download/v7.1.0/rules_scala-v7.1.0.tar.gz",
         ],
-        strip_prefix = "rules_scala-4ba3780fcba8d26980daff4639abc6f18517308b",
+        strip_prefix = "rules_scala-7.1.0",
     ),
     org_tensorflow = dict(
         sha256 = "a640d1f97be316a09301dfc9347e3d929ad4d9a2336e3ca23c32c93b0ff7e5d0",
@@ -449,9 +449,9 @@ REPOSITORY_LOCATIONS = dict(
         manual_license_name = "libc/musl",
     ),
     rules_cc = dict(
-        sha256 = "abc605dd850f813bb37004b77db20106a19311a96b2da1c92b789da529d28fe1",
-        strip_prefix = "rules_cc-0.0.17",
-        urls = ["https://github.com/bazelbuild/rules_cc/releases/download/0.0.17/rules_cc-0.0.17.tar.gz"],
+        sha256 = "a2fdfde2ab9b2176bd6a33afca14458039023edb1dd2e73e6823810809df4027",
+        strip_prefix = "rules_cc-0.2.14",
+        urls = ["https://github.com/bazelbuild/rules_cc/releases/download/0.2.14/rules_cc-0.2.14.tar.gz"],
     ),
     rules_foreign_cc = dict(
         sha256 = "6041f1374ff32ba711564374ad8e007aef77f71561a7ce784123b9b4b88614fc",

bazel/thrift.bzl

@@ -14,25 +14,29 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 
-load("@io_bazel_rules_scala//twitter_scrooge:twitter_scrooge.bzl", "twitter_scrooge")
 load("@rules_jvm_external//:defs.bzl", "maven_install")
 
 def thrift_deps(scala_version):
-    twitter_scrooge()
+    # twitter_scrooge is now set up via scala_toolchains(twitter_scrooge = {...}) in WORKSPACE
 
-    finagle_version = "22.7.0"
+    finagle_version = "24.2.0"
     scala_minor_version = ".".join(scala_version.split(".")[:2])
 
     maven_install(
         name = "thrift_deps",
         artifacts = [
+            # Finagle dependencies (needed for generated service code)
             "com.twitter:finagle-thriftmux_%s:%s" % (scala_minor_version, finagle_version),
+            "com.twitter:finagle-thrift_%s:%s" % (scala_minor_version, finagle_version),
             "com.twitter:finagle-mux_%s:%s" % (scala_minor_version, finagle_version),
             "com.twitter:finagle-core_%s:%s" % (scala_minor_version, finagle_version),
+            "com.twitter:finagle-http_%s:%s" % (scala_minor_version, finagle_version),
+            # Scrooge dependencies (needed for code generation toolchain)
             "com.twitter:scrooge-core_%s:%s" % (scala_minor_version, finagle_version),
             "com.twitter:scrooge-generator_%s:%s" % (scala_minor_version, finagle_version),
-            "com.twitter:finagle-http_%s:%s" % (scala_minor_version, finagle_version),
-            "org.apache.thrift:libthrift:0.10.0",
+            "com.twitter:util-core_%s:%s" % (scala_minor_version, finagle_version),
+            "com.twitter:util-logging_%s:%s" % (scala_minor_version, finagle_version),
+            # Logging
             "org.slf4j:slf4j-api:1.7.36",
             "ch.qos.logback:logback-core:1.2.10",
             "ch.qos.logback:logback-classic:1.2.10",

src/stirling/source_connectors/socket_tracer/testing/containers/thriftmux/BUILD.bazel

@@ -67,6 +67,19 @@ thrift_library(
     srcs = glob(["**/*.thrift"]),
 )
 
+# Scrooge core with finagle dependencies bundled.
+# This is used as the scrooge_core for the toolchain, so that the generated
+# service code has access to finagle classes at compile time.
+scala_library(
+    name = "scrooge_core_with_finagle",
+    visibility = ["//visibility:public"],
+    exports = [
+        "@thrift_deps//:com_twitter_finagle_core_2_13",
+        "@thrift_deps//:com_twitter_finagle_thrift_2_13",
+        "@thrift_deps//:com_twitter_scrooge_core_2_13",
+    ],
+)
+
 scala_library(
     name = "scrooge_jars",
     visibility = ["//visibility:public"],
@@ -81,6 +94,31 @@ scala_library(
     ],
 )
 
+# Runtime dependencies for generated scrooge code (for downstream consumers)
+scala_library(
+    name = "scrooge_runtime_deps",
+    visibility = ["//visibility:public"],
+    exports = [
+        "@thrift_deps//:com_twitter_finagle_core_2_13",
+        "@thrift_deps//:com_twitter_finagle_thrift_2_13",
+        "@thrift_deps//:com_twitter_scrooge_core_2_13",
+        "@thrift_deps//:org_apache_thrift_libthrift",
+    ],
+)
+
+# Full finagle dependencies for Server/Client scala files
+scala_library(
+    name = "finagle_deps",
+    visibility = ["//visibility:public"],
+    exports = [
+        "@thrift_deps//:com_twitter_finagle_core_2_13",
+        "@thrift_deps//:com_twitter_finagle_http_2_13",
+        "@thrift_deps//:com_twitter_finagle_mux_2_13",
+        "@thrift_deps//:com_twitter_finagle_thrift_2_13",
+        "@thrift_deps//:com_twitter_finagle_thriftmux_2_13",
+    ],
+)
+
 scala_library(
     name = "thrift_jars",
     visibility = ["//visibility:public"],
@@ -94,6 +132,9 @@ scrooge_scala_library(
     deps = [
         ":thrift_library",
     ],
+    exports = [
+        ":scrooge_runtime_deps",
+    ],
 )
 
 scala_library(
@@ -110,6 +151,7 @@ scala_binary(
     srcs = glob(["**/*.scala"]),
     main_class = "Server",
     deps = [
+        ":finagle_deps",
         ":thriftmux_scrooge",
     ],
 )
@@ -119,6 +161,7 @@ scala_binary(
     srcs = glob(["**/*.scala"]),
     main_class = "Client",
     deps = [
+        ":finagle_deps",
         ":thriftmux_scrooge",
     ],
 )
@@ -144,6 +187,7 @@ scala_image(
     ],
     main_class = "Server",
     deps = [
+        ":finagle_deps",
         ":logging",
         ":thriftmux_scrooge",
     ],