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

Remove gallery preview version 2020-09-30 from stable release #12222

Merged
merged 3 commits into from
Jan 5, 2021
Merged
Changes from all 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
6 changes: 2 additions & 4 deletions specification/compute/resource-manager/readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -219,8 +219,7 @@ input-file:
- Microsoft.Compute/stable/2020-06-01/runCommands.json
- Microsoft.Compute/stable/2019-04-01/skus.json
- Microsoft.Compute/stable/2020-06-30/disk.json
- Microsoft.Compute/preview/2020-09-30/gallery.json
- Microsoft.Compute/preview/2020-09-30/sharedGallery.json
- Microsoft.Compute/stable/2019-12-01/gallery.json
Copy link
Contributor

Choose a reason for hiding this comment

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

@ArcturusZhang @weidongxu-microsoft @msyyc @qiaozha to comments the sdk impact for this gallery api change

Copy link
Member

Choose a reason for hiding this comment

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

Compute for python is multiapi, so it is not permitted to delete tag because there may be customers to use the tag now. It's ok to add a new tag.

Copy link
Member Author

Choose a reason for hiding this comment

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

We have an ongoing email thread to discuss this impact. This is to remove breaking changes that were introduced in the gallery version.

Copy link
Member

Choose a reason for hiding this comment

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

per email discussion, Go, Java isn't released, but Python released.

Copy link
Member Author

Choose a reason for hiding this comment

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

Per email thread with Gallery team, no end users are whitelisted to use the 2020-09-30 version of Gallery and SharedGallery. No end behaviour should change.
@JeffreyRichter approved these changes, so I think we can proceed.

- Microsoft.ContainerService/stable/2017-01-31/containerService.json
- Microsoft.Compute/preview/2020-10-01-preview/cloudService.json
Copy link
Member Author

@Sandido Sandido Dec 28, 2020

Choose a reason for hiding this comment

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

Should this preview CloudService also be removed? @akning-ms , @msyyc

Copy link
Contributor

Choose a reason for hiding this comment

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

Is there any reason to remove it too? I think here is only discuss impact to change shareGallery.json ? maybe @msyyc or @yungezz to have more context to answer this

Copy link
Member

Choose a reason for hiding this comment

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

@akning-ms I think it's removed because it's a preview version

Copy link
Contributor

Choose a reason for hiding this comment

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

If so, I think we don't need remove it if it is already in previous SDK version

```
Expand All @@ -235,7 +234,6 @@ input-file:
```

### Tag: package-2020-09-30
Copy link
Contributor

Choose a reason for hiding this comment

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

We dont have to delete this tag as it will create avocado validation failure and we lost history for generation SDK base on this tag. so pls keep it or let me know why remove it?

Copy link
Member Author

Choose a reason for hiding this comment

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

I put the tag back, but kept it removed from the SDK generation for the latest version.

Copy link
Member Author

Choose a reason for hiding this comment

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

See our ongoing email thread "Candidate PR to fix breaking changes in compute SDK" for details.


These settings apply only when `--tag=package-2020-09-30` is specified on the command line.

``` yaml $(tag) == 'package-2020-09-30'
Expand Down Expand Up @@ -810,4 +808,4 @@ input-file:

## AzureResourceSchema

See configuration in [readme.azureresourceschema.md](./readme.azureresourceschema.md)
See configuration in [readme.azureresourceschema.md](./readme.azureresourceschema.md)