[Breaking Change] OAD support request by Tim Lovell-Smith: non-breaking change alerted on (3/14/2023)

[konrad-jamrozik]

As reported on email thread with subject New Response to Swagger Validation Feedback: 3/14/2023 3:33:46 PM

https://github.com/Azure/azure-rest-api-specs/pull/23049/checks?check_run_id=11963838160

Question detail:
This refactoring seems like a non-breaking change to swagger to me.
But it is flagged as a breaking change, because [I think] it is too sensitive to $ref redirection.
Azure/azure-rest-api-specs#23049

❌ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.Cache/preview/2023-03-01-preview/redisenterprise.json#L1302:9
Old: Microsoft.Cache/preview/2023-03-01-preview/redisenterprise.json#L1302:9

❌ 1017 - ReferenceRedirection The '$ref' property points to different models in the old and new versions.
New: Microsoft.Cache/preview/2023-03-01-preview/redisenterprise.json#L1927:9
Old: Microsoft.Cache/preview/2023-03-01-preview/redisenterprise.json#L1940:9

[konrad-jamrozik]

The relevant PR got merged but it is not yet clear to me if the OAD was mistaken here. I left a comment asking Tim about it, and also emailed him.

[konrad-jamrozik]

Based on my chat with Tim over email, it is not clear if this was "clearly not a breaking change" or "possibly a breaking change". Hence for now assuming it was possibly a breaking change, and the tool behaved correctly. Closing this issue.

[konrad-jamrozik]

Reopening. Further discussion over email led to @mikekistler saying this looks like a false positive - the tool should not be reporting it.