diff --git a/.kokoro/build.sh b/.kokoro/build.sh index 85bd795d0f..2096f0ece7 100755 --- a/.kokoro/build.sh +++ b/.kokoro/build.sh @@ -48,8 +48,8 @@ set +e case ${JOB_TYPE} in test) echo "SUREFIRE_JVM_OPT: ${SUREFIRE_JVM_OPT}" - mvn test -B -ntp -Dclirr.skip=true -Denforcer.skip=true ${SUREFIRE_JVM_OPT} - RETURN_CODE=$? +# mvn test -B -ntp -Dclirr.skip=true -Denforcer.skip=true ${SUREFIRE_JVM_OPT} + RETURN_CODE=1 ;; lint) mvn com.coveo:fmt-maven-plugin:check -B -ntp @@ -60,25 +60,25 @@ javadoc) RETURN_CODE=$? ;; integration) - mvn -B ${INTEGRATION_TEST_ARGS} \ - -ntp \ - -Penable-integration-tests \ - -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ - -fae \ - verify - RETURN_CODE=$? +# mvn -B ${INTEGRATION_TEST_ARGS} \ +# -ntp \ +# -Penable-integration-tests \ +# -DtrimStackTrace=false \ +# -Dclirr.skip=true \ +# -Denforcer.skip=true \ +# -fae \ +# verify + RETURN_CODE=1 ;; graalvm) # Run Unit and Integration Tests with Native Image - mvn -B ${INTEGRATION_TEST_ARGS} -ntp -Pnative test - RETURN_CODE=$? +# mvn -B ${INTEGRATION_TEST_ARGS} -ntp -Pnative test + RETURN_CODE=1 ;; graalvm17) # Run Unit and Integration Tests with Native Image - mvn -B ${INTEGRATION_TEST_ARGS} -ntp -Pnative test - RETURN_CODE=$? +# mvn -B ${INTEGRATION_TEST_ARGS} -ntp -Pnative test + RETURN_CODE=1 ;; samples) SAMPLES_DIR=samples diff --git a/google-cloud-storage/src/test/java/com/google/cloud/storage/it/BucketCleaner.java b/google-cloud-storage/src/test/java/com/google/cloud/storage/it/BucketCleaner.java index 3f70e9d443..75b81d46c4 100644 --- a/google-cloud-storage/src/test/java/com/google/cloud/storage/it/BucketCleaner.java +++ b/google-cloud-storage/src/test/java/com/google/cloud/storage/it/BucketCleaner.java @@ -16,6 +16,7 @@ package com.google.cloud.storage.it; +import com.google.api.gax.grpc.GrpcCallContext; import com.google.api.gax.paging.Page; import com.google.api.gax.rpc.ApiException; import com.google.api.gax.rpc.FailedPreconditionException; @@ -26,8 +27,12 @@ import com.google.cloud.storage.Storage.BlobSourceOption; import com.google.cloud.storage.Storage.BucketSourceOption; import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; import com.google.storage.control.v2.BucketName; import com.google.storage.control.v2.Folder; +import com.google.storage.control.v2.GetStorageLayoutRequest; +import com.google.storage.control.v2.ListFoldersRequest; +import com.google.storage.control.v2.ListManagedFoldersRequest; import com.google.storage.control.v2.StorageControlClient; import com.google.storage.control.v2.StorageLayout; import com.google.storage.control.v2.StorageLayoutName; @@ -97,15 +102,34 @@ public static void doCleanup(String bucketName, Storage s, StorageControlClient boolean anyFailedFolderDelete = false; boolean anyFailedManagedFolderDelete = false; + GrpcCallContext grpcCallContext = + GrpcCallContext.createDefault() + .withExtraHeaders( + ImmutableMap.of("x-goog-user-project", ImmutableList.of(projectId))); if (!anyFailedObjectDelete) { BucketName parent = BucketName.of("_", bucketName); StorageLayout storageLayout = - ctrl.getStorageLayout(StorageLayoutName.of(parent.getProject(), parent.getBucket())); + ctrl.getStorageLayoutCallable() + .call( + GetStorageLayoutRequest.newBuilder() + .setName( + StorageLayoutName.of(parent.getProject(), parent.getBucket()) + .toString()) + .build(), + grpcCallContext); + List folderDeletes; if (storageLayout.hasHierarchicalNamespace() && storageLayout.getHierarchicalNamespace().getEnabled()) { folderDeletes = - StreamSupport.stream(ctrl.listFolders(parent).iterateAll().spliterator(), false) + StreamSupport.stream( + ctrl.listFoldersPagedCallable() + .call( + ListFoldersRequest.newBuilder().setParent(parent.toString()).build(), + grpcCallContext) + .iterateAll() + .spliterator(), + false) .collect(Collectors.toList()) .stream() .sorted(Collections.reverseOrder(Comparator.comparing(Folder::getName))) @@ -129,7 +153,15 @@ public static void doCleanup(String bucketName, Storage s, StorageControlClient try { managedFolderDeletes = StreamSupport.stream( - ctrl.listManagedFolders(parent).iterateAll().spliterator(), false) + ctrl.listManagedFoldersPagedCallable() + .call( + ListManagedFoldersRequest.newBuilder() + .setParent(parent.toString()) + .build(), + grpcCallContext) + .iterateAll() + .spliterator(), + false) .map( managedFolder -> { LOGGER.warning( diff --git a/samples/pom.xml b/samples/pom.xml index f059468a4d..37be43768a 100644 --- a/samples/pom.xml +++ b/samples/pom.xml @@ -29,26 +29,9 @@ - snapshot - - - - org.slf4j - jul-to-slf4j - 2.0.16 - - - ch.qos.logback - logback-classic - 1.4.11 - runtime - - - - diff --git a/samples/snapshot/pom.xml b/samples/snapshot/pom.xml index 37f33a152b..0d60dae6dc 100644 --- a/samples/snapshot/pom.xml +++ b/samples/snapshot/pom.xml @@ -51,22 +51,17 @@ 4.13.2 test + - com.google.truth - truth - 1.4.4 - test - - - org.apache.httpcomponents - httpmime - 4.5.14 + org.slf4j + jul-to-slf4j + 2.0.16 test - com.google.cloud - google-cloud-pubsub - 1.132.2 + ch.qos.logback + logback-classic + 1.4.11 test @@ -74,6 +69,7 @@ + diff --git a/samples/snippets/src/test/java/com/example/storage/ITVerboseBucketCleanupTest.java b/samples/snapshot/src/test/java/com/example/storage/ITVerboseBucketCleanupTest.java similarity index 100% rename from samples/snippets/src/test/java/com/example/storage/ITVerboseBucketCleanupTest.java rename to samples/snapshot/src/test/java/com/example/storage/ITVerboseBucketCleanupTest.java