Azure Site Recovery support for managed Managed disk and Service Api Version bumpup changes #6905

viverm · 2018-08-11T12:46:13Z

Description

Checklist

I have read the Submitting Changes section of CONTRIBUTING.md
The title of the PR is clear and informative
The appropriate change log has been updated
The PR does not introduce breaking changes
If applicable, the changes made in the PR have proper test coverage
For public API changes to cmdlets:
- the changes have gone through a cmdlet design review
- the cmdlet markdown files were generated using the platyPS module

viverm · 2018-08-16T17:17:15Z

Cmdlet Review link :: https://github.com/Azure/azure-powershell-cmdlet-review-pr/issues/30

MiYanni · 2018-08-21T15:26:11Z

...overyServices.SiteRecovery/Commands.RecoveryServices.SiteRecovery.Test/AzureRm.Resources.ps1

@@ -0,0 +1,142 @@
+function Get-AzureRmResourceGroup


Why do you need to create your own AzureRM.Resources.ps1? Also, since the filename is 'AzureRm.Resources.ps1' and not 'AzureRM.Resources.ps1', this will cause issues in NetStandard as Linux files are case-sensitive.

AzureRM.Resources.ps1 is for doing operation which is not part ASR , but we need to do for infrastructure creation for Azure to Azure replication.
If I am not creating it what is the other way out ?
'AzureRm.Resources.ps1' and not 'AzureRM.Resources.ps1' fixing this.

@viverm A similar file to this is already being included with your project. This file comes from our azure-powershell-common repo. It is included here:

azure-powershell/tools/Common.Dependencies.Test.targets

Lines 78 to 80 in c2765af

<None Include="$(ScenarioTestToolsPath)AzureRM.Resources.ps1">

<CopyToOutputDirectory>Always</CopyToOutputDirectory>

</None>

That should cover your needs as long as you are using our common ResourceManager project. We don't suggest using Microsoft.Azure.Management.Resources anymore. You can see the entire file here: https://github.com/Azure/azure-powershell-common/blob/develop/src/ScenarioTest.ResourceManager/AzureRM.Resources.ps1

MiYanni · 2018-08-21T15:28:44Z

...mmands.RecoveryServices.SiteRecovery.Test/Commands.RecoveryServices.SiteRecovery.Test.csproj

@@ -34,12 +34,31 @@
    <WarningLevel>4</WarningLevel>
  </PropertyGroup>
  <ItemGroup>
+    <Reference Include="Hyak.Common, Version=1.0.0.0, Culture=neutral, PublicKeyToken=31bf3856ad364e35, processorArchitecture=MSIL" />


Do not add duplicate references. Most of these added references exist in our Common.Dependences.targets or Common.Dependencies.Test.targets file. In Visual Studio in the Solution Explorer, you'll see several references duplicated, one resolved properly and one with a yellow exclamation mark. Please remove the duplicate references.

MiYanni · 2018-08-21T15:34:33Z

...iteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/A2ATestsHelper.ps1

+            Write-Host $("Job Status:") -ForegroundColor Green
+            $Job
+
+            if($Job.State -eq "InProgress" -or $Job.State -eq "NotStarted")


If statements require boolean conditions. So, assigning a boolean based on a boolean is redundant. It can simply be:

$isJobLeftForProcessing = ($Job.State -eq 'InProgress' -or $Job.State -eq 'NotStarted')

Also, double quotes are used for interpolated strings (strings that can substitute variables in-place) where single quotes are used for string literals.

MiYanni · 2018-08-21T15:35:58Z

...iteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/A2ATestsHelper.ps1

+        do
+        {
+            $IRjobs = Get-AzureRmRecoveryServicesAsrJob -TargetObjectId $affectedObjectId | Sort-Object StartTime -Descending | select -First 2 | Where-Object{$_.JobType -eq "SecondaryIrCompletion"}
+            if($IRjobs -eq $null -or $IRjobs.Count -ne 1)


Same as above.

MiYanni · 2018-08-21T15:39:28Z

...iteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/AsrA2ATestsBase.cs

+                this.helper.SetupEnvironment(AzureModule.AzureResourceManager);
+
+                var rmProfileModule = this.helper.RMProfileModule;
+                var modules = new List<string>();


Why not simply declare an array with the values already initialized? No need to dynamically add values and then convert it to an array.

MiYanni · 2018-08-21T15:48:19Z

...overyServices.SiteRecovery/Commands.RecoveryServices.SiteRecovery/Common/PSComputeClient .cs

@@ -0,0 +1,76 @@
+// ----------------------------------------------------------------------------------


There is a space in this filename. Please name the file PSComputeClient.cs.

MiYanni · 2018-08-21T15:52:02Z

...very/Commands.RecoveryServices.SiteRecovery/Policy/UpdateAzureRmRecoveryServicesAsrPolicy.cs


            var hyperVReplicaAzurePolicyInput = new HyperVReplicaAzurePolicyInput
            {
                ApplicationConsistentSnapshotFrequencyInHours =
                    this.applicationConsistentSnapshotFrequencyInHours,
-                Encryption = this.encryption,
+// encryption is removed from latest API . Service is setting the default vault internally.


Was Encryption exposed to the PowerShell user? Is this a breaking change?

This field should have been already deprecated in older PowerShell and older API versions. (Service stopped honouring this field for several months already). This PR contains changes that use new api-version for APIs.

@viverm is this field no longer available in new api? Is this why this is removed?

In case this is a concern, one option I'm thinking continue to have PS parameter, but default to whatever value service is already having in implementation.

I think probably we just need to remove the comment as well. Encryption has been not honoured in service for an year now.

we cann't remove it from cmdlet but as service is not taking that input anymore , we are not populating it.

It is not the breaking change as service was not honoring it earlier . @MiYanni

MiYanni · 2018-08-21T15:55:38Z

...Manager/RecoveryServices.SiteRecovery/Commands.RecoveryServices.SiteRecovery/packages.config

-  <package id="Microsoft.Azure.Management.RecoveryServices.SiteRecovery" version="1.0.3-preview" targetFramework="net452" />
-</packages>
+  <package id="Microsoft.Azure.Management.RecoveryServices.SiteRecovery" version="1.2.0-preview" targetFramework="net452" />
+  <package id="Microsoft.Rest.ClientRuntime" version="2.3.11" targetFramework="net452" />


The ClientRuntime packages are handed by the Common.Dependencies.targets and Profile's packages.config respectively. These will not be used.

MiYanni · 2018-08-21T15:56:47Z

src/ResourceManager/RecoveryServices.SiteRecovery/RecoveryServices.SiteRecovery.sln

@@ -1,12 +1,15 @@
 Microsoft Visual Studio Solution File, Format Version 12.00
 # Visual Studio 15
-VisualStudioVersion = 15.0.27703.2042
+VisualStudioVersion = 15.0.26430.13
 MinimumVisualStudioVersion = 10.0.40219.1
 Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Tests", "Tests", "{95C16AED-FD57-42A0-86C3-2CF4300A4817}"
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Commands.RecoveryServices.SiteRecovery", "Commands.RecoveryServices.SiteRecovery\Commands.RecoveryServices.SiteRecovery.csproj", "{7C879645-31EE-4A78-AD81-5512300FA105}"
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Commands.RecoveryServices.SiteRecovery.Test", "Commands.RecoveryServices.SiteRecovery.Test\Commands.RecoveryServices.SiteRecovery.Test.csproj", "{6C7D3D81-37AB-445E-8081-78A1FEC0A774}"


There needs to be a project reference to Profile in this solution for it to load the Common.Dependencies.targets appropriately.

MiYanni · 2018-08-21T15:57:11Z

src/ResourceManager/RecoveryServices/Commands.RecoveryServices/Commands.RecoveryServices.csproj

@@ -88,12 +88,6 @@
    </None>
    <None Include="StartupScripts\*.ps1" />
  </ItemGroup>
-  <ItemGroup>
-    <ProjectReference Include="..\..\Profile\Commands.Profile\Commands.Profile.csproj">


Why was Profile removed?

markcowl · 2018-08-21T18:04:23Z

tools/StaticAnalysis/Exceptions/AzureRM.RecoveryServices.SiteRecovery/BreakingChangeIssues.csv

@@ -16,3 +16,8 @@
 "Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.dll","Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.GetAzureRmRecoveryServicesAsrStorageClassification","Get-AzureRmRecoveryServicesAsrStorageClassification","0","1020","The cmdlet 'Get-AzureRmRecoveryServicesAsrStorageClassification' no longer has output type 'System.Collections.Generic.IEnumerable`1[Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRStorageClassification]'.","Make cmdlet 'Get-AzureRmRecoveryServicesAsrStorageClassification' return type 'System.Collections.Generic.IEnumerable`1[Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRStorageClassification]'."
 "Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.dll","Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.GetAzureRmRecoveryServicesAsrStorageClassificationMapping","Get-AzureRmRecoveryServicesAsrStorageClassificationMapping","0","1020","The cmdlet 'Get-AzureRmRecoveryServicesAsrStorageClassificationMapping' no longer has output type 'System.Collections.Generic.IEnumerable`1[Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRStorageClassificationMapping]'.","Make cmdlet 'Get-AzureRmRecoveryServicesAsrStorageClassificationMapping' return type 'System.Collections.Generic.IEnumerable`1[Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRStorageClassificationMapping]'."
 "Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.dll","Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.GetAzureRmRecoveryServicesAsrvCenter","Get-AzureRmRecoveryServicesAsrvCenter","0","1020","The cmdlet 'Get-AzureRmRecoveryServicesAsrvCenter' no longer has output type 'System.Collections.Generic.IEnumerable`1[Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRvCenter]'.","Make cmdlet 'Get-AzureRmRecoveryServicesAsrvCenter' return type 'System.Collections.Generic.IEnumerable`1[Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRvCenter]'."
+"Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.dll","Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.StartAzureRmRecoveryServicesAsrSwitchProcessServerJob","Start-AzureRmRecoveryServicesAsrSwitchProcessServerJob","0","3010","The property 'ChildErrors' of type 'Microsoft.Azure.Management.RecoveryServices.SiteRecovery.Models.HealthError' has been removed.","Add the property 'ChildErrors' back to type 'Microsoft.Azure.Management.RecoveryServices.SiteRecovery.Models.HealthError'."


These all seem like actual breaking changes - you cannot ship breaking changes in this release.

Properties added in breaking change is coming from API version bump up.
These properties are not used as input / output of powershell cmdlet

maddieclayton · 2018-09-04T23:13:22Z

@viverm Looks like you will need to fix up this PR to pull in the the Netstandard changes. Also, looks like one of your tests is failing: https://azuresdkci.westus2.cloudapp.azure.com/job/powershell/7133/artifact/src/Publish/TestResults/FailingTests/Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.Test.dll.html

…nto asr_managedDisk

viverm · 2018-09-06T05:17:47Z

@maddieclayton resolved merge conflict and code changes done for test and netStandard.

maddieclayton · 2018-09-06T18:41:50Z

Assigning to @cormacpayne as he did the cmdlet design review.

MiYanni · 2018-09-06T19:48:51Z

...overyServices.SiteRecovery/Commands.RecoveryServices.SiteRecovery.Test/AzureRm.Resources.ps1

@@ -0,0 +1,142 @@
+function Get-AzureRmResourceGroup


@viverm A similar file to this is already being included with your project. This file comes from our azure-powershell-common repo. It is included here:

azure-powershell/tools/Common.Dependencies.Test.targets

Lines 78 to 80 in c2765af

<None Include="$(ScenarioTestToolsPath)AzureRM.Resources.ps1">

<CopyToOutputDirectory>Always</CopyToOutputDirectory>

</None>

That should cover your needs as long as you are using our common ResourceManager project. We don't suggest using Microsoft.Azure.Management.Resources anymore. You can see the entire file here: https://github.com/Azure/azure-powershell-common/blob/develop/src/ScenarioTest.ResourceManager/AzureRM.Resources.ps1

MiYanni · 2018-09-06T19:53:13Z

...s.SiteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/AsrA2ATests.ps1

+        $recoveryFabricName = getRecoveryFabric
+
+        New-AzureRmResourceGroup -name $vaultRg -location $vaultRgLocation -force
+        [Microsoft.Azure.Test.TestUtilities]::Wait(20 * 1000)


Should be using [Microsoft.Rest.ClientRuntime.Azure.TestFramework.TestUtilities]::Wait

MiYanni · 2018-09-06T19:53:16Z

...s.SiteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/AsrA2ATests.ps1

+        [Microsoft.Azure.Test.TestUtilities]::Wait(20 * 1000)
+    # vault Creation
+        New-azureRmRecoveryServicesVault -ResourceGroupName $vaultRg -Name $vaultName -Location $vaultLocation
+        [Microsoft.Azure.Test.TestUtilities]::Wait(20 * 1000)


Should be using [Microsoft.Rest.ClientRuntime.Azure.TestFramework.TestUtilities]::Wait

MiYanni · 2018-09-06T19:53:35Z

...iteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/A2ATestsHelper.ps1

+            {
+                Write-Host $("IR in Progress...") -ForegroundColor Yellow
+                Write-Host $("Waiting for: " + $JobQueryWaitTimeInSeconds.ToString + " Seconds") -ForegroundColor Yellow
+                [Microsoft.Azure.Test.TestUtilities]::Wait($JobQueryWaitTimeInSeconds * 1000)


Should be using [Microsoft.Rest.ClientRuntime.Azure.TestFramework.TestUtilities]::Wait

MiYanni · 2018-09-06T19:53:42Z

...iteRecovery/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/A2ATestsHelper.ps1

+                    Write-Host $($($Job.JobType) + " in Progress...") -ForegroundColor Yellow
+                }
+                Write-Host $("Waiting for: " + $JobQueryWaitTimeInSeconds.ToString + " Seconds") -ForegroundColor Yellow
+                [Microsoft.Azure.Test.TestUtilities]::Wait($JobQueryWaitTimeInSeconds * 1000)


Should be using [Microsoft.Rest.ClientRuntime.Azure.TestFramework.TestUtilities]::Wait

MiYanni · 2018-09-06T21:19:14Z

...eryServices/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/AsrA2ATestsBase.cs

+        public RecoveryServicesClient RecoveryServicesMgmtClient { get; private set; }
+        public SiteRecoveryManagementClient SiteRecoveryMgmtClient { get; private set; }
+
+        public SubscriptionClient SubscriptionClient { get; private set; }


Same comment at the other test controller above. I don't believe this should be needed.

MiYanni · 2018-09-06T21:20:31Z

...eryServices/Commands.RecoveryServices.SiteRecovery.Test/ScenarioTests/A2A/AsrA2ATestsBase.cs

+        public SubscriptionClient SubscriptionClient { get; private set; }
+
+        public ComputeManagementClient ComputeManagementClient { get; private set; }
+        public Microsoft.Azure.Management.ResourceManager.ResourceManagementClient ResourceManagementRestClient { get; private set; }


Same comment at the other test controller above. This should use Microsoft.Azure.Management.Internal.Resources.

MiYanni · 2018-09-06T21:28:10Z

...nds.RecoveryServices.SiteRecovery/RecoveryPlan/EditAzureRmRecoveryServicesAsrRecoveryPlan.cs

-    [Cmdlet("Edit", ResourceManager.Common.AzureRMConstants.AzureRMPrefix + "RecoveryServicesAsrRecoveryPlan",DefaultParameterSetName = ASRParameterSets.AppendGroup,SupportsShouldProcess = true)]
+    [Cmdlet(
+        VerbsData.Edit,
+        "AzureRmRecoveryServicesAsrRecoveryPlan",


Needs to be ResourceManager.Common.AzureRMConstants.AzureRMPrefix + "RecoveryServicesAsrRecoveryPlan"

MiYanni · 2018-09-06T21:30:00Z

...es/Commands.RecoveryServices.SiteRecovery/Vault/SetAzureRmRecoveryServicesAsrVaultContext.cs

@@ -76,7 +76,7 @@ public override void ExecuteSiteRecoveryCmdlet()
                catch (CloudException ex)
                {
                    Logger.Instance.WriteWarning(ex.Message);
-                    throw new Exception(Resources.TryDownloadingVaultFile);
+                    // throw new Exception(Resources.TryDownloadingVaultFile);


Commented code. Please remove if intended to no longer throw. Also, why was this exception commented out?

// code interanally handled the cloud exception thrown earlier.But still there are changes of other exception.
// suggesting alternate way to user to unblock if this command is failing.

MiYanni · 2018-09-06T21:31:27Z

...s/Commands.RecoveryServices.SiteRecovery/help/Edit-AzureRmRecoveryServicesAsrRecoveryPlan.md

@@ -56,7 +56,7 @@ Appends a group to existing Azure Site Recovery recovery plan and returns the in
 List of ASR replication protected items to be added to the recovery plan group in the recovery plan object.

 ```yaml
-Type: Microsoft.Azure.Commands.RecoveryServices.SiteRecovery.ASRReplicationProtectedItem[]
+Type: ASRReplicationProtectedItem[]


You'll have to regenerate all your help files. You forgot to add the -UseFullTypeName switch. See the documentation here: https://github.com/Azure/azure-powershell/blob/preview/documentation/development-docs/help-generation.md#updating-a-single-markdown-file

This is something new from what we are doing in the way to generate help file.
will do the changes follow this from now onwards

…nto asr_managedDisk

viverm · 2018-09-07T09:49:53Z

@MiYanni Addressed the comment .

one thing if I want to test the AZ recoveryServices where can I get installer to test?

MiYanni · 2018-09-07T21:40:23Z

demand: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/powershell-demand/1085/
ps-sign: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/ps-sign/1206/
psnetsign: https://azuresdkci.westus2.cloudapp.azure.com/view/PowerShell/job/psnetsign/174/

MiYanni · 2018-09-07T22:48:25Z

@viverm There is no installer for Az. You install the built modules using Register-PSRepository.

Breaking changes have been removed from the PR.

viverm added 4 commits August 11, 2018 14:52

code change

0efe5db

Test fixes

814e10f

Update Test

d98b839

Merge branch 'preview' into asr_managedDisk

bf62c55

azuresdkci added the needs-review label Aug 11, 2018

viverm changed the title ~~Asr managed disk + Service Api Version~~ Azure Site Recovery support for managed Managed disk and Service Api Version bumpup changes Aug 13, 2018

viverm added 3 commits August 13, 2018 18:56

Test changes

7501000

temperory skipping test

2442b16

Merge branch 'preview' into asr_managedDisk

bfd3ae7

viverm requested review from maddieclayton, cormacpayne and MiYanni August 13, 2018 20:28

markcowl added Do Not Merge 🚫 and removed needs-review labels Aug 13, 2018

viverm added 3 commits August 16, 2018 18:54

Adding A2A Test

4f3b024

Restoring tools folder

a18313a

Merge branch 'preview' into asr_managedDisk

b823d3a

maddieclayton assigned viverm Aug 16, 2018

maddieclayton added the needs-revision label Aug 16, 2018

Merge conflicts

7065b47

viverm force-pushed the asr_managedDisk branch 2 times, most recently from 2d8e601 to facf08f Compare August 17, 2018 13:25

Tools folder restore and Test fix after dependency nuget changes

6269e1b

viverm force-pushed the asr_managedDisk branch from facf08f to 6269e1b Compare August 17, 2018 13:41

praveennet added this to the 2018-08-24 milestone Aug 17, 2018

markcowl removed this from the 2018-08-24 milestone Aug 20, 2018

MiYanni suggested changes Aug 21, 2018

View reviewed changes

markcowl previously requested changes Aug 21, 2018

View reviewed changes

PR comments

96668f9

viverm added 4 commits September 4, 2018 16:50

push changes

97993ff

ReplicationProvider fix

b0fc138

push

ea7667e

breaking change class name change

7a7eb6a

maddieclayton added Merge Conflicts and removed Do Not Merge 🚫 labels Sep 4, 2018

viverm added 3 commits September 5, 2018 23:44

merge Module

32ee4e7

recovery services module bump for netcore

f0ca0be

Merge branch 'preview' of https://github.com/Azure/azure-powershell i…

614d875

…nto asr_managedDisk

maddieclayton added needs-review and removed Merge Conflicts needs-revision labels Sep 6, 2018

maddieclayton assigned cormacpayne and unassigned viverm Sep 6, 2018

MiYanni suggested changes Sep 6, 2018

View reviewed changes

MiYanni assigned viverm and unassigned cormacpayne Sep 6, 2018

maddieclayton added needs-revision and removed needs-review labels Sep 7, 2018

viverm added 2 commits September 7, 2018 09:59

Merge branch 'preview' of https://github.com/Azure/azure-powershell i…

ad8117e

…nto asr_managedDisk

PR comments

6e5d4ab

MiYanni approved these changes Sep 7, 2018

View reviewed changes

MiYanni merged commit b6a1b29 into Azure:preview Sep 7, 2018

	<None Include="$(ScenarioTestToolsPath)AzureRM.Resources.ps1">
	<CopyToOutputDirectory>Always</CopyToOutputDirectory>
	</None>

		@@ -0,0 +1,76 @@
		// ----------------------------------------------------------------------------------

Azure Site Recovery support for managed Managed disk and Service Api Version bumpup changes #6905

Azure Site Recovery support for managed Managed disk and Service Api Version bumpup changes #6905

Uh oh!

Conversation

viverm commented Aug 11, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Checklist

Uh oh!

viverm commented Aug 16, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

praveennet Aug 22, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

maddieclayton commented Sep 4, 2018

Uh oh!

viverm commented Sep 6, 2018

Uh oh!

maddieclayton commented Sep 6, 2018

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

viverm commented Sep 7, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MiYanni commented Sep 7, 2018

Uh oh!

viverm commented Aug 11, 2018 •

edited

Loading

praveennet Aug 22, 2018 •

edited

Loading

viverm commented Sep 7, 2018 •

edited

Loading