Skip to content

Commit e1f53fb

Browse files
mziccardmziccard
committed
---
yaml --- r: 2483 b: refs/heads/update-datastore c: c3b6616 h: refs/heads/master i: 2481: 081b005 2479: 5c5f50f
1 parent 0f6b48a commit e1f53fb

3 files changed

Lines changed: 22 additions & 41 deletions

File tree

‎[refs]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ refs/heads/gh-pages: 4e0561bb4504bf647db669a14417b2b2c87ba45d
55
refs/heads/bigquery: 762fa5830e6c398c0396177e3e7fd243bd62cfc3
66
refs/heads/pubsub-alpha: 1a0e970f265af871e02274085b9662b3fe29058b
77
refs/heads/resource-manager: ebf4adc5ee835cd2086c4ac5b4e78d01a5a005a7
8-
refs/heads/update-datastore: 315a0eb44b3bbebdd963f69376f5a4145ded5d7c
8+
refs/heads/update-datastore: c3b6616c98982a201c1dcd6a5b1e7d3210d009d6
99
refs/tags/0.0.9: 22f1839238f66c39e67ed4dfdcd273b1ae2e8444
1010
refs/tags/v0.0.10: 207ebd2a3472fddee69fe1298eb90429e3306efd
1111
refs/tags/v0.0.11: ffbfba48a6426ff63c08ff2117e58681f251fbf2

‎branches/update-datastore/gcloud-java-storage/src/main/java/com/google/gcloud/storage/testing/RemoteGcsHelper.java‎

Lines changed: 19 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,6 @@ public class RemoteGcsHelper {
4545

4646
private static final Logger log = Logger.getLogger(RemoteGcsHelper.class.getName());
4747
private static final String BUCKET_NAME_PREFIX = "gcloud-test-bucket-temp-";
48-
private static final String PROJECT_ID_ENV_VAR = "GCLOUD_TESTS_PROJECT_ID";
49-
private static final String PRIVATE_KEY_ENV_VAR = "GCLOUD_TESTS_KEY";
5048
private final StorageOptions options;
5149

5250
private RemoteGcsHelper(StorageOptions options) {
@@ -107,13 +105,7 @@ public static RemoteGcsHelper create(String projectId, InputStream keyStream)
107105
StorageOptions storageOptions = StorageOptions.builder()
108106
.authCredentials(AuthCredentials.createForJson(keyStream))
109107
.projectId(projectId)
110-
.retryParams(RetryParams.builder()
111-
.retryMaxAttempts(10)
112-
.retryMinAttempts(6)
113-
.maxRetryDelayMillis(30000)
114-
.totalRetryPeriodMillis(120000)
115-
.initialRetryDelayMillis(250)
116-
.build())
108+
.retryParams(retryParams())
117109
.connectTimeout(60000)
118110
.readTimeout(60000)
119111
.build();
@@ -145,41 +137,30 @@ public static RemoteGcsHelper create(String projectId, String keyPath)
145137
log.log(Level.WARNING, ex.getMessage());
146138
}
147139
throw GcsHelperException.translate(ex);
148-
} catch (IOException ex) {
149-
if (log.isLoggable(Level.WARNING)) {
150-
log.log(Level.WARNING, ex.getMessage());
151-
}
152-
throw GcsHelperException.translate(ex);
153140
}
154141
}
155142

156143
/**
157-
* Creates a {@code RemoteGcsHelper} object. Project id and path to JSON key are read from two
158-
* environment variables: {@code GCLOUD_TESTS_PROJECT_ID} and {@code GCLOUD_TESTS_KEY}.
159-
*
160-
* @return A {@code RemoteGcsHelper} object for the provided options.
161-
* @throws com.google.gcloud.storage.testing.RemoteGcsHelper.GcsHelperException if environment
162-
* variables {@code GCLOUD_TESTS_PROJECT_ID} and {@code GCLOUD_TESTS_KEY} are not set or if
163-
* the file pointed by {@code GCLOUD_TESTS_KEY} does not exist
144+
* Creates a {@code RemoteGcsHelper} object using default project id and authentication
145+
* credentials.
164146
*/
165147
public static RemoteGcsHelper create() throws GcsHelperException {
166-
String projectId = System.getenv(PROJECT_ID_ENV_VAR);
167-
String keyPath = System.getenv(PRIVATE_KEY_ENV_VAR);
168-
if (projectId == null) {
169-
String message = "Environment variable " + PROJECT_ID_ENV_VAR + " not set";
170-
if (log.isLoggable(Level.WARNING)) {
171-
log.log(Level.WARNING, message);
172-
}
173-
throw new GcsHelperException(message);
174-
}
175-
if (keyPath == null) {
176-
String message = "Environment variable " + PRIVATE_KEY_ENV_VAR + " not set";
177-
if (log.isLoggable(Level.WARNING)) {
178-
log.log(Level.WARNING, message);
179-
}
180-
throw new GcsHelperException(message);
181-
}
182-
return create(projectId, keyPath);
148+
StorageOptions storageOptions = StorageOptions.builder()
149+
.retryParams(retryParams())
150+
.connectTimeout(60000)
151+
.readTimeout(60000)
152+
.build();
153+
return new RemoteGcsHelper(storageOptions);
154+
}
155+
156+
private static RetryParams retryParams() {
157+
return RetryParams.builder()
158+
.retryMaxAttempts(10)
159+
.retryMinAttempts(6)
160+
.maxRetryDelayMillis(30000)
161+
.totalRetryPeriodMillis(120000)
162+
.initialRetryDelayMillis(250)
163+
.build();
183164
}
184165

185166
private static class DeleteBucketTask implements Callable<Boolean> {

‎branches/update-datastore/utilities/integration_test_env.sh‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
# Export test env variables
2-
export GCLOUD_TESTS_PROJECT_ID="gcloud-devel"
3-
export GCLOUD_TESTS_KEY=$TRAVIS_BUILD_DIR/signing-tools/gcloud-devel-travis.json
2+
export GCLOUD_PROJECT="gcloud-devel"
3+
export GOOGLE_APPLICATION_CREDENTIALS=$TRAVIS_BUILD_DIR/signing-tools/gcloud-devel-travis.json

0 commit comments

Comments
 (0)