Chore/1.34 pre release

.bazelversion

@@ -1 +1 @@
-7.4.0
+7.5.0

bazel/extensions/extensions_build_config.bzl

@@ -537,6 +537,9 @@ EXTENSIONS = {
     # "envoy.filters.generic.router":                             "//source/extensions/filters/network/generic_proxy/router:config",
     # "envoy.generic_proxy.codecs.dubbo":                         "//source/extensions/filters/network/generic_proxy/codecs/dubbo:config",
     # "envoy.generic_proxy.codecs.http1":                         "//source/extensions/filters/network/generic_proxy/codecs/http1:config",
+
+    # Dynamic mocules
+    "envoy.filters.http.dynamic_modules":                      "//source/extensions/filters/http/dynamic_modules:factory_registration",
 }
 
 # These can be changed to ["//visibility:public"], for  downstream builds which

bazel/repositories.bzl

@@ -67,7 +67,6 @@ def _repository_impl(name, **kwargs):
 
 def envoy_gloo_dependencies():
     _repository_impl("envoy", patches=[
-        "@envoy_gloo//bazel/foreign_cc:001-tlv-support.patch", #remove in  1.34
     ])
     _repository_impl("json", build_file = "@envoy_gloo//bazel/external:json.BUILD")
     _repository_impl("inja", build_file = "@envoy_gloo//bazel/external:inja.BUILD")

bazel/repository_locations.bzl

@@ -2,8 +2,8 @@ REPOSITORY_LOCATIONS = dict(
     # can't have more than one comment between envoy line and commit line in 
     # order to accommodate `check_extensions_build_config.sh`
     envoy = dict(
-        # envoy v1.33.0
-        commit = "b0f43d67aa25c1b03c97186a200cc187f4c22db3",
+        # envoy v1.34.0-dev
+        commit = "87b4ae8dde4a0cd35bc8c3584ec79d3625522c4d",
         remote = "https://github.com/envoyproxy/envoy",
     ),
     inja = dict(

changelog/v1.34.0-patch0/bumpenvoy.yaml

@@ -0,0 +1,9 @@
+changelog:
+  - type: DEPENDENCY_BUMP
+    issueLink: https://github.com/solo-io/envoy-gloo-ee/issues/887
+    dependencyOwner: envoyproxy
+    dependencyRepo: envoy
+    dependencyTag: 1.34.0
+    resolvesIssue: false
+    description: >-
+      Bumps to a prerelease of envoy 1.34.0 to allow for usage of dynamic modules 

source/common/matcher/solo_matcher.cc

@@ -120,6 +120,9 @@ class CompiledStdMatcher : public Regex::CompiledMatcher {
 public:
   CompiledStdMatcher(std::regex &&regex) : regex_(std::move(regex)) {}
 
+  // TODO(nfuden) refactor matchers or find a way to get back onto upstream matchers
+  const std::string& pattern() const override { return EMPTY_STRING; }
+
   // CompiledMatcher
   bool match(absl::string_view value) const override {
     try {

source/common/matcher/solo_matcher.h

@@ -37,7 +37,10 @@ class Matcher {
   static MatcherConstPtr
   create(const ::envoy::config::route::v3::RouteMatch &match,
          Server::Configuration::CommonFactoryContext& context);
+
 };
 
+
+
 } // namespace Matcher
 } // namespace Envoy

source/common/tcp/conn_pool_impl.h

@@ -283,7 +283,7 @@ template <typename T, typename D> class InstanceImpl : public Instance<T> {
         }
         LbContextImpl lb_context(hash_key);
         Upstream::HostConstSharedPtr host =
-            cluster->loadBalancer().chooseHost(&lb_context);
+        Upstream::LoadBalancer::onlyAllowSynchronousHostSelection(cluster->loadBalancer().chooseHost(&lb_context));
         if (!host) {
           // TODO(talnordan):
           // parent_.callbacks_->onFailure("no host");

source/extensions/filters/http/aws_lambda/aws_lambda_filter_config_factory.cc

@@ -24,7 +24,6 @@ AWSLambdaFilterConfigFactory::createFilterFactoryFromProtoTyped(
   // the upstream code from attempting to access the method. https://github.com/envoyproxy/envoy/issues/26653
   auto chain = std::make_unique<Extensions::Common::Aws::DefaultCredentialsProviderChain>(
           server_context.api(), absl::nullopt /* ServerFactoryContextOptRef context */,
-          server_context.singletonManager(),
           // We pass an empty string if we don't have a region
           proto_config.has_service_account_credentials() ? proto_config.service_account_credentials().region() : "",
           nullptr);

source/extensions/filters/http/transformation/transformation_filter_config.cc

@@ -64,7 +64,7 @@ class ResponseMatcherImpl : public ResponseMatcher {
 
 private:
   std::vector<Http::HeaderUtility::HeaderDataPtr> headers_;
-  absl::optional<Matchers::StringMatcherImpl<envoy::type::matcher::v3::StringMatcher>> response_code_details_match_;
+  absl::optional<Matchers::StringMatcherImpl> response_code_details_match_;
 };
 
 bool ResponseMatcherImpl::matches(

test/common/tcp/conn_pool_impl_test.cc

@@ -410,7 +410,9 @@ TEST_F(TcpConnPoolImplTest, NoHost) {
 
   T value;
   EXPECT_CALL(cm_.thread_local_cluster_.lb_, chooseHost(_))
-      .WillOnce(Return(nullptr));
+      .WillOnce(Invoke([] {
+        return Upstream::HostSelectionResponse{nullptr};
+      }));
   conn_pool_->makeRequest("foo", value);
 
   conn_pool_ = {};