Skip to content

feat: Initial CLI for SSB integration - #2166

Merged
sydney-munro merged 13 commits into
mainfrom
ssbcli
Aug 23, 2023
Merged

feat: Initial CLI for SSB integration#2166
sydney-munro merged 13 commits into
mainfrom
ssbcli

Conversation

@sydney-munro

Copy link
Copy Markdown
Contributor

No description provided.

@sydney-munro
sydney-munro requested review from a team August 15, 2023 16:15
@product-auto-label product-auto-label Bot added size: l Pull request size is large. api: storage Issues related to the googleapis/java-storage API. labels Aug 15, 2023
@sydney-munro
sydney-munro marked this pull request as draft August 15, 2023 16:15
Comment thread ssb/src/main/java/com/google/cloud/StorageSharedBenchmarkingCli.java Outdated
Comment thread ssb/src/main/java/com/google/cloud/StorageSharedBenchmarkingCli.java Outdated
StorageOptions alwaysRetryStorageOptions =
StorageOptions.newBuilder().setProjectId(project).setRetrySettings(retrySettings).build();
Storage storageClient = alwaysRetryStorageOptions.getService();
Path tempDir = Paths.get(System.getProperty("java.io.tmpdir"));

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We may want this to be an argument, sometimes java.io.tmpdir can be in a not so great place. (Not something that needs to change in this PR)

Comment thread ssb/src/main/java/com/google/cloud/StorageSharedBenchmarkingUtils.java Outdated
Comment thread ssb/pom.xml
Comment thread ssb/src/main/java/com/google/cloud/CloudMonitoringResult.java Outdated
Comment thread ssb/pom.xml Outdated
Comment thread ssb/pom.xml Outdated
@sydney-munro
sydney-munro force-pushed the ssbcli branch 2 times, most recently from ea179b9 to 5231f36 Compare August 17, 2023 13:57
@sydney-munro
sydney-munro marked this pull request as ready for review August 17, 2023 14:33
@product-auto-label product-auto-label Bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Aug 17, 2023
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: xl Pull request size is extra large. labels Aug 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants