Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump to the right version. Switch test account to team account. #4710

Merged
merged 8 commits into from
Jul 31, 2019
Merged
Show file tree
Hide file tree
Changes from 7 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 7 additions & 1 deletion eng/jacoco-test-coverage/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,8 @@
<azure-keyvault.version>4.0.0-preview.1</azure-keyvault.version>
<azure-messaging-eventhubs.version>5.0.0-preview.2</azure-messaging-eventhubs.version>
<azure-storage-blob.version>12.0.0-preview.1</azure-storage-blob.version>
<azure-storage-queue.version>12.0.0-preview.1</azure-storage-queue.version>
<azure-storage-file.version>12.0.0-preview.2</azure-storage-file.version>
<azure-storage-queue.version>12.0.0-preview.2</azure-storage-queue.version>
</properties>

<distributionManagement>
Expand Down Expand Up @@ -94,6 +95,11 @@
<!-- <artifactId>azure-storage-blob</artifactId>-->
<!-- <version>${azure-storage-blob.version}</version>-->
<!-- </dependency>-->
<dependency>
<groupId>com.azure</groupId>
<artifactId>azure-storage-file</artifactId>
<version>${azure-storage-file.version}</version>
</dependency>
<dependency>
<groupId>com.azure</groupId>
<artifactId>azure-storage-queue</artifactId>
Expand Down
4 changes: 2 additions & 2 deletions eng/spotbugs-aggregate-report/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@
<azure-keyvault.version>4.0.0-preview.1</azure-keyvault.version>
<azure-messaging-eventhubs.version>5.0.0-preview.2</azure-messaging-eventhubs.version>
<azure-storage.version>12.0.0-preview.1</azure-storage.version>
<azure-storage-file.version>12.0.0-preview.1</azure-storage-file.version>
<azure-storage-queue.version>12.0.0-preview.1</azure-storage-queue.version>
<azure-storage-file.version>12.0.0-preview.2</azure-storage-file.version>
<azure-storage-queue.version>12.0.0-preview.2</azure-storage-queue.version>
</properties>
<distributionManagement>
<site>
Expand Down
1 change: 1 addition & 0 deletions pom.client.xml
Original file line number Diff line number Diff line change
Expand Up @@ -546,6 +546,7 @@
-snippetpath ${project.basedir}/sdk/keyvault/azure-keyvault-keys/src/samples/java
-snippetpath ${project.basedir}/sdk/keyvault/azure-keyvault-secrets/src/samples/java
<!-- -snippetpath ${project.basedir}/storage/client/blob/src/samples/java-->
Copy link
Member

Choose a reason for hiding this comment

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

Should this be uncommented?

Copy link
Contributor Author

@sima-zhu sima-zhu Jul 30, 2019

Choose a reason for hiding this comment

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

@alzimmermsft can we add back this line?

Copy link
Member

Choose a reason for hiding this comment

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

I wouldn't worry about uncommenting this line in this PR, blobs doesn't have any snippets yet.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will uncommon this.

-snippetpath ${project.basedir}/storage/client/file/src/samples/java
-snippetpath ${project.basedir}/storage/client/queue/src/samples/java
</additionalOptions>
</configuration>
Expand Down
2 changes: 1 addition & 1 deletion storage/client/blob/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ definition, such as text or binary data.
<dependency>
<groupId>com.azure</groupId>
<artifactId>azure-storage-blob</artifactId>
<version>12.0.0-preview.1</version>
<version>12.0.0-preview.2</version>
</dependency>
```

Expand Down
File renamed without changes.
2 changes: 1 addition & 1 deletion storage/client/file/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ Shares provide a way to organize sets of files and also can be mounted as an SMB
<dependency>
<groupId>com.azure</groupId>
<artifactId>azure-storage-file</artifactId>
<version>12.0.0-preview.1</version>
<version>12.0.0-preview.2</version>
</dependency>
```

Expand Down
2 changes: 1 addition & 1 deletion storage/client/file/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@

<groupId>com.azure</groupId>
<artifactId>azure-storage-file</artifactId>
<version>12.0.0-preview.1</version>
<version>12.0.0-preview.2</version>

<name>azure-storage-file</name>
<url>https://github.com/Azure/azure-sdk-for-java</url>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,5 +9,5 @@
class FileConfiguration {
//TODO: Eventually remove these hardcoded strings with https://github.com/Azure/azure-sdk-for-java/issues/3141
static final String NAME = "azure-storage-file";
static final String VERSION = "12.0.0-preview.1";
static final String VERSION = "12.0.0-preview.2";
}
Original file line number Diff line number Diff line change
Expand Up @@ -74,9 +74,7 @@ public AzureFileStorageImpl build() {
this.pipeline = RestProxy.createDefaultPipeline();
}
AzureFileStorageImpl client = new AzureFileStorageImpl(pipeline);
if (this.version != null) {
Copy link
Member

Choose a reason for hiding this comment

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

Does the version always get set from a static conig class ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is auto-gen class. I have the temp fix to pass the spotbugs. Loop in @jianghaolu for further discussion.

Copy link
Member

Choose a reason for hiding this comment

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

Let's track this, so it doesn't cause issues later.

client.setVersion(this.version);
}
client.setVersion(this.version);
if (this.url != null) {
client.setUrl(this.url);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ public void beforeTest() {
if (interceptorManager.isPlaybackMode()) {
client = setupClient((connectionString, endpoint) -> new DirectoryClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.directoryPath(dirName)
.httpClient(interceptorManager.getPlaybackClient())
Expand All @@ -44,7 +43,6 @@ public void beforeTest() {
} else {
client = setupClient((connectionString, endpoint) -> new DirectoryClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.directoryPath(dirName)
.httpClient(HttpClient.createDefault().wiretap(true))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ public void beforeTest() {
if (interceptorManager.isPlaybackMode()) {
directoryClient = setupClient((connectionString, endpoint) -> new DirectoryClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.directoryPath(dirName)
.httpClient(interceptorManager.getPlaybackClient())
Expand All @@ -40,7 +39,6 @@ public void beforeTest() {
} else {
directoryClient = setupClient((connectionString, endpoint) -> new DirectoryClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.directoryPath(dirName)
.httpClient(HttpClient.createDefault().wiretap(true))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ public void beforeTest() {
.connectionString(connectionString)
.shareName(shareName)
.filePath(filePath)
.endpoint(endpoint)
.httpClient(interceptorManager.getPlaybackClient())
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.buildAsyncClient(), true, fileAsyncLogger);
Expand All @@ -48,7 +47,6 @@ public void beforeTest() {
.connectionString(connectionString)
.shareName(shareName)
.filePath(filePath)
.endpoint(endpoint)
.httpClient(HttpClient.createDefault().wiretap(true))
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.addPolicy(interceptorManager.getRecordPolicy())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,6 @@ public void beforeTest() {
.connectionString(connectionString)
.shareName(shareName)
.filePath(filePath)
.endpoint(endpoint)
.httpClient(interceptorManager.getPlaybackClient())
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.buildClient(), true, fileLogger);
Expand All @@ -51,7 +50,6 @@ public void beforeTest() {
.connectionString(connectionString)
.shareName(shareName)
.filePath(filePath)
.endpoint(endpoint)
.httpClient(HttpClient.createDefault().wiretap(true))
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.addPolicy(interceptorManager.getRecordPolicy())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,14 +37,12 @@ public void beforeTest() {
if (interceptorManager.isPlaybackMode()) {
fileServiceAsyncClient = setupClient((connectionString, endpoint) -> new FileServiceClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.httpClient(interceptorManager.getPlaybackClient())
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.buildAsyncClient(), true, fileServiceAsyncLogger);
} else {
fileServiceAsyncClient = setupClient((connectionString, endpoint) -> new FileServiceClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.httpClient(HttpClient.createDefault().wiretap(true))
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.addPolicy(interceptorManager.getRecordPolicy())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,14 +41,12 @@ public void beforeTest() {
if (interceptorManager.isPlaybackMode()) {
fileServiceClient = setupClient((connectionString, endpoint) -> new FileServiceClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.httpClient(interceptorManager.getPlaybackClient())
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.buildClient(), true, fileServiceLogger);
} else {
fileServiceClient = setupClient((connectionString, endpoint) -> new FileServiceClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.httpClient(HttpClient.createDefault().wiretap(true))
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.addPolicy(interceptorManager.getRecordPolicy())
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,15 +39,13 @@ public void beforeTest() {
if (interceptorManager.isPlaybackMode()) {
shareAsyncClient = setupClient((connectionString, endpoint) -> new ShareClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.httpClient(interceptorManager.getPlaybackClient())
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.buildAsyncClient(), true, shareAsyncLogger);
} else {
shareAsyncClient = setupClient((connectionString, endpoint) -> new ShareClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.httpClient(HttpClient.createDefault().wiretap(true))
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,15 +42,13 @@ public void beforeTest() {
if (interceptorManager.isPlaybackMode()) {
shareClient = setupClient((connectionString, endpoint) -> new ShareClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.httpClient(interceptorManager.getPlaybackClient())
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
.buildClient(), true, shareLogger);
} else {
shareClient = setupClient((connectionString, endpoint) -> new ShareClientBuilder()
.connectionString(connectionString)
.endpoint(endpoint)
.shareName(shareName)
.httpClient(HttpClient.createDefault().wiretap(true))
.httpLogDetailLevel(HttpLogDetailLevel.BODY_AND_HEADERS)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
{
"networkCallRecords" : [ ],
"variables" : [ "file10519411" ]
"variables" : [ "file59336dc2" ]
}
Original file line number Diff line number Diff line change
@@ -1,42 +1,42 @@
{
"networkCallRecords" : [ {
"Method" : "PUT",
"Uri" : "https://sima.file.core.windows.net/dirsharename/directory863334?restype=directory",
"Uri" : "https://azstoragesdkaccount.file.core.windows.net/dirsharename/directory909922?restype=directory",
"Headers" : {
"x-ms-version" : "2018-11-09",
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.1 1.8.0_201; Windows 10 10.0"
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.2 1.8.0_201; Windows 10 10.0"
},
"Response" : {
"x-ms-version" : "2018-11-09",
"Server" : "Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0",
"ETag" : "\"0x8D70F8CB67B1F98\"",
"Last-Modified" : "Tue, 23 Jul 2019 16:42:12 GMT",
"ETag" : "\"0x8D71521CAC84923\"",
"Last-Modified" : "Tue, 30 Jul 2019 19:11:57 GMT",
"retry-after" : "0",
"Content-Length" : "0",
"StatusCode" : "201",
"x-ms-request-id" : "a4781ee1-001a-006b-3775-4192c4000000",
"x-ms-request-id" : "260a2c3b-901a-0077-060a-47b537000000",
"x-ms-request-server-encrypted" : "true",
"Date" : "Tue, 23 Jul 2019 16:42:11 GMT"
"Date" : "Tue, 30 Jul 2019 19:11:57 GMT"
}
}, {
"Method" : "PUT",
"Uri" : "https://sima.file.core.windows.net/dirsharename/directory863334?restype=directory&comp=metadata",
"Uri" : "https://azstoragesdkaccount.file.core.windows.net/dirsharename/directory909922?restype=directory&comp=metadata",
"Headers" : {
"x-ms-version" : "2018-11-09",
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.1 1.8.0_201; Windows 10 10.0"
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.2 1.8.0_201; Windows 10 10.0"
},
"Response" : {
"x-ms-version" : "2018-11-09",
"Server" : "Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0",
"ETag" : "\"0x8D70F8CB6831042\"",
"Last-Modified" : "Tue, 23 Jul 2019 16:42:12 GMT",
"ETag" : "\"0x8D71521CACDEFC8\"",
"Last-Modified" : "Tue, 30 Jul 2019 19:11:57 GMT",
"retry-after" : "0",
"Content-Length" : "0",
"StatusCode" : "200",
"x-ms-request-id" : "a4781ee2-001a-006b-3875-4192c4000000",
"x-ms-request-id" : "260a2c3c-901a-0077-070a-47b537000000",
"x-ms-request-server-encrypted" : "true",
"Date" : "Tue, 23 Jul 2019 16:42:11 GMT"
"Date" : "Tue, 30 Jul 2019 19:11:57 GMT"
}
} ],
"variables" : [ "directory863334" ]
"variables" : [ "directory909922" ]
}
Original file line number Diff line number Diff line change
@@ -1,47 +1,47 @@
{
"networkCallRecords" : [ {
"Method" : "PUT",
"Uri" : "https://sima.file.core.windows.net/share22756b14?restype=share",
"Uri" : "https://azstoragesdkaccount.file.core.windows.net/share592387fe?restype=share",
"Headers" : {
"x-ms-version" : "2018-11-09",
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.1 1.8.0_201; Windows 10 10.0"
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.2 1.8.0_201; Windows 10 10.0"
},
"Response" : {
"x-ms-version" : "2018-11-09",
"Server" : "Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0",
"ETag" : "\"0x8D70F8C78DAADCB\"",
"Last-Modified" : "Tue, 23 Jul 2019 16:40:29 GMT",
"ETag" : "\"0x8D715218A138CAF\"",
"Last-Modified" : "Tue, 30 Jul 2019 19:10:09 GMT",
"retry-after" : "0",
"Content-Length" : "0",
"StatusCode" : "201",
"x-ms-request-id" : "a4781d48-001a-006b-4875-4192c4000000",
"Date" : "Tue, 23 Jul 2019 16:40:28 GMT"
"x-ms-request-id" : "7f75e089-301a-00db-150a-4794a0000000",
"Date" : "Tue, 30 Jul 2019 19:10:09 GMT"
}
}, {
"Method" : "PUT",
"Uri" : "https://sima.file.core.windows.net/share22756b14/testshare?restype=directory",
"Uri" : "https://azstoragesdkaccount.file.core.windows.net/share592387fe/testshare?restype=directory",
"Headers" : {
"x-ms-version" : "2018-11-09",
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.1 1.8.0_201; Windows 10 10.0"
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.2 1.8.0_201; Windows 10 10.0"
},
"Response" : {
"x-ms-version" : "2018-11-09",
"Server" : "Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0",
"ETag" : "\"0x8D70F8C78E3B87F\"",
"Last-Modified" : "Tue, 23 Jul 2019 16:40:29 GMT",
"ETag" : "\"0x8D715218A1A6700\"",
"Last-Modified" : "Tue, 30 Jul 2019 19:10:09 GMT",
"retry-after" : "0",
"Content-Length" : "0",
"StatusCode" : "201",
"x-ms-request-id" : "a4781d4a-001a-006b-4975-4192c4000000",
"x-ms-request-id" : "7f75e08b-301a-00db-160a-4794a0000000",
"x-ms-request-server-encrypted" : "true",
"Date" : "Tue, 23 Jul 2019 16:40:28 GMT"
"Date" : "Tue, 30 Jul 2019 19:10:09 GMT"
}
}, {
"Method" : "PUT",
"Uri" : "https://sima.file.core.windows.net/share22756b14/testshare?restype=directory",
"Uri" : "https://azstoragesdkaccount.file.core.windows.net/share592387fe/testshare?restype=directory",
"Headers" : {
"x-ms-version" : "2018-11-09",
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.1 1.8.0_201; Windows 10 10.0"
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.2 1.8.0_201; Windows 10 10.0"
},
"Response" : {
"x-ms-version" : "2018-11-09",
Expand All @@ -50,27 +50,27 @@
"retry-after" : "0",
"Content-Length" : "228",
"StatusCode" : "409",
"x-ms-request-id" : "a4781d4b-001a-006b-4a75-4192c4000000",
"Body" : "<?xml version=\"1.0\" encoding=\"utf-8\"?><Error><Code>ResourceAlreadyExists</Code><Message>The specified resource already exists.\nRequestId:a4781d4b-001a-006b-4a75-4192c4000000\nTime:2019-07-23T16:40:29.4840714Z</Message></Error>",
"Date" : "Tue, 23 Jul 2019 16:40:28 GMT",
"x-ms-request-id" : "7f75e08c-301a-00db-170a-4794a0000000",
"Body" : "<?xml version=\"1.0\" encoding=\"utf-8\"?><Error><Code>ResourceAlreadyExists</Code><Message>The specified resource already exists.\nRequestId:7f75e08c-301a-00db-170a-4794a0000000\nTime:2019-07-30T19:10:09.1716444Z</Message></Error>",
"Date" : "Tue, 30 Jul 2019 19:10:09 GMT",
"Content-Type" : "application/xml"
}
}, {
"Method" : "DELETE",
"Uri" : "https://sima.file.core.windows.net/share22756b14?restype=share",
"Uri" : "https://azstoragesdkaccount.file.core.windows.net/share592387fe?restype=share",
"Headers" : {
"x-ms-version" : "2018-11-09",
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.1 1.8.0_201; Windows 10 10.0"
"User-Agent" : "azsdk-java-azure-storage-file/12.0.0-preview.2 1.8.0_201; Windows 10 10.0"
},
"Response" : {
"x-ms-version" : "2018-11-09",
"Server" : "Windows-Azure-File/1.0 Microsoft-HTTPAPI/2.0",
"retry-after" : "0",
"Content-Length" : "0",
"StatusCode" : "202",
"x-ms-request-id" : "a4781d4c-001a-006b-4b75-4192c4000000",
"Date" : "Tue, 23 Jul 2019 16:40:28 GMT"
"x-ms-request-id" : "7f75e08d-301a-00db-180a-4794a0000000",
"Date" : "Tue, 30 Jul 2019 19:10:09 GMT"
}
} ],
"variables" : [ "share22756b14" ]
"variables" : [ "share592387fe" ]
}
Loading