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

typo: servicefabric/resource-manager/Microsoft.ServiceFabric #4529

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Nov 21, 2018

  • specificed -> specified
  • sucessful -> successful
  • resturns -> returns
  • commited -> committed
  • dataloss -> data loss
  • Double word "with"
  • applciation -> application
  • ABRIC -> FABRIC
  • resouce -> resource
  • mininum -> minimum
  • Double word "maximum"
  • revent -> prevent
  • empheral -> ephemeral
  • exmaple -> example
  • assgined -> assigned
  • carefull -> careful
  • primay -> primary
  • Dispaly -> Display
  • skus -> SKUs

If the values in x-ms-examples are breaking, I can rebase those out

@openapi-portal-comment
Copy link

If you're a MSFT employee, click this link
to view this PR's validation status on our new OpenAPI Hub spec management tool.

@AutorestCI
Copy link

AutorestCI commented Nov 21, 2018

Automation for azure-sdk-for-go

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-go#5063

@AutorestCI
Copy link

AutorestCI commented Nov 21, 2018

Automation for azure-sdk-for-js

Nothing to generate for azure-sdk-for-js

@AutorestCI
Copy link

AutorestCI commented Nov 21, 2018

Automation for azure-sdk-for-python

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-python#5997

@AutorestCI
Copy link

AutorestCI commented Nov 21, 2018

Automation for azure-sdk-for-ruby

Encountered a Subprocess error: (azure-sdk-for-ruby)

Command: ['/usr/local/bin/autorest', '/tmp/tmpfq07iqnd/rest/specification/servicefabric/resource-manager/readme.md', '--multiapi', '--ruby', '--ruby-sdks-folder=/tmp/tmpfq07iqnd/sdk', '[email protected]/autorest.ruby@preview', '--version=preview']
Finished with return code 1
and output:

AutoRest code generation utility [version: 2.0.4283; node: v10.15.3]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
   Loading AutoRest core      '/root/.autorest/@[email protected]/node_modules/@microsoft.azure/autorest-core/dist' (2.0.4384)
   Loading AutoRest extension '@microsoft.azure/autorest.ruby' (preview->3.1.40)
   Loading AutoRest extension '@microsoft.azure/autorest.modeler' (2.1.22->2.1.22)
Processing batch task - {"tag":"package-2019-03"} .
Processing batch task - {"tag":"package-2018-02"} .
FATAL: System.NullReferenceException: CodeModel doesn't have api-version set, specs in code model have mismatching api-versions.
   at AutoRest.Ruby.Azure.CodeGeneratorRba.<Generate>d__6.MoveNext() in /opt/vsts/work/1/s/src/azure/CodeGeneratorRba.cs:line 52
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at AutoRest.Ruby.Program.<ProcessInternal>d__4.MoveNext() in /opt/vsts/work/1/s/src/Program.cs:line 118
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at NewPlugin.<Process>d__20.MoveNext() in /opt/vsts/work/1/s/autorest.common/src/Plugins/NewPlugin.cs:line 163
FATAL: ruby/generate - FAILED
FATAL: Error: Plugin ruby reported failure.
Process() cancelled due to exception : Plugin ruby reported failure.
Failure during batch task - {"tag":"package-2018-02"} -- Error: Plugin ruby reported failure..
  Error: Plugin ruby reported failure.

@azuresdkci
Copy link
Contributor

Can one of the admins verify this patch?

@AutorestCI
Copy link

AutorestCI commented Nov 21, 2018

Automation for azure-sdk-for-node

Encountered a Subprocess error: (azure-sdk-for-node)

Command: ['/usr/local/bin/autorest', '/tmp/tmp9suipgfl/rest/specification/servicefabric/resource-manager/readme.md', '--license-header=MICROSOFT_MIT_NO_VERSION', '--node-sdks-folder=/tmp/tmp9suipgfl/azure-sdk-for-node', '--nodejs', '[email protected]/[email protected]']
Finished with return code 1
and output:

AutoRest code generation utility [version: 2.0.4283; node: v8.12.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
   Loading AutoRest core      '/root/.autorest/@[email protected]/node_modules/@microsoft.azure/autorest-core/dist' (2.0.4289)
   Loading AutoRest extension '@microsoft.azure/autorest.nodejs' (2.2.131->2.2.131)
   Loading AutoRest extension '@microsoft.azure/autorest.modeler' (2.3.44->2.3.44)

ERROR (Fatal/DuplicateModelCollsion): Duplicated model name with non-identical definitions
    - file:///tmp/tmp9suipgfl/rest/specification/servicefabric/resource-manager/Microsoft.ServiceFabric/preview/2017-07-01-preview/application.json:973:4 ($.definitions.ApplicationHealthPolicy)
    - file:///tmp/tmp9suipgfl/rest/specification/servicefabric/resource-manager/Microsoft.ServiceFabric/stable/2018-02-01/cluster.json:534:4 ($.definitions.ApplicationHealthPolicy)
Process() cancelled due to exception : Cancellation requested.
  Cancellation requested.

@AutorestCI
Copy link

AutorestCI commented Nov 21, 2018

Automation for azure-sdk-for-java

The initial PR has been merged into your service PR:
Azure/azure-sdk-for-java#3731

@nschonni nschonni changed the title servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric typo: servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric Nov 21, 2018
@nschonni nschonni closed this Nov 26, 2018
@nschonni nschonni reopened this Nov 26, 2018
@openapi-portal-comment
Copy link

If you're a MSFT employee, click this link
to view this PR's validation status on our new OpenAPI Hub spec management tool.

@nschonni nschonni force-pushed the typo-servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric branch from dd14d41 to d35e0f7 Compare November 29, 2018 02:14
@nschonni nschonni changed the title typo: servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric typo: Microsoft.ServiceFabric Nov 29, 2018
@nschonni nschonni force-pushed the typo-servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric branch from d35e0f7 to 19f0265 Compare November 29, 2018 02:16
@nschonni nschonni closed this Nov 30, 2018
@nschonni nschonni reopened this Nov 30, 2018
@openapi-portal-comment
Copy link

If you're a MSFT employee, click this link
to view this PR's validation status on our new OpenAPI Hub spec management tool.

@nschonni
Copy link
Contributor Author

@juhacket any idea why the build is failing?

@salameer
Copy link
Member

salameer commented Apr 4, 2019

Hi @anuchandy

What's the status here? are these changes impacting the Main Swagger or is it only related to documentation

@nschonni nschonni force-pushed the typo-servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric branch 3 times, most recently from 46678fd to 2eb3c25 Compare April 12, 2019 17:20
@salameer
Copy link
Member

@vipul-modi

@nschonni is an awesome external contributer that provided these fixes to the swagger comments maily for Service fabric, would you kindly mind reviewing or tagging someone who can review?

@salameer salameer requested review from amanbha and removed request for anuchandy April 25, 2019 19:39
@salameer
Copy link
Member

@amanbha can you kindly review here?

@nschonni nschonni force-pushed the typo-servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric branch from 2eb3c25 to 9122a02 Compare May 20, 2019 20:45
@nschonni
Copy link
Contributor Author

@salameer this got approved. I rebased just to get the latest checks, but the failures are pre-existing

- specificed -> specified
- sucessful -> successful
- resturns -> returns
- commited -> committed
- dataloss -> data loss
- Double word "with"
- applciation -> application
- ABRIC -> FABRIC
- resouce -> resource
- mininum -> minimum
- Double word "maximum"
- revent -> prevent
- empheral -> ephemeral
- exmaple -> example
- assgined -> assigned
- carefull -> careful
- primay -> primary
- Dispaly -> Display
- skus -> SKUs
@nschonni nschonni force-pushed the typo-servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric branch from 3196659 to 6680c1b Compare June 6, 2019 20:13
@nschonni nschonni requested a review from samedder as a code owner June 6, 2019 20:13
@openapi-sdkautomation
Copy link

openapi-sdkautomation bot commented Jun 6, 2019

SDK Automation [Logs] (Generated from 6680c1b)

Pending Python: Azure/azure-sdk-for-python
  • Package generation pending.
Pending Java: Azure/azure-sdk-for-java
  • Package generation pending.
Pending Go: Azure/azure-sdk-for-go
  • Package generation pending.
Pending Ruby: Azure/azure-sdk-for-ruby
  • Package generation pending.

@nschonni
Copy link
Contributor Author

nschonni commented Jun 6, 2019

@sergey-shandar is the "ReadOnly" rule in model validation expected to fail on older APIs?

@nschonni
Copy link
Contributor Author

@jhendrixMSFT can you help land this?

@jhendrixMSFT jhendrixMSFT merged commit f90d70f into Azure:master Jun 20, 2019
@nschonni nschonni deleted the typo-servicefabric/resource-manager/Microsoft.ServiceFabric/servicefabric branch June 20, 2019 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants