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

Update Azure Cloud Shell API to the newer version #17115

Merged
merged 1 commit into from
Apr 24, 2024

Conversation

PankajBhojwani
Copy link
Contributor

Updates the api-version to 2023-02-01-preview when requesting for CloudShell settings and shell

Validation Steps Performed

Can still use Azure Cloud Shell through Windows Terminal

@PankajBhojwani
Copy link
Contributor Author

PankajBhojwani commented Apr 23, 2024

A few things to note:

  • The new api-version introduces some new user settings, the main one being allowing Ephemeral sessions, which differs from Mounted sessions in that ephemeral sessions have no storage associated with them. As long as the user updates their user settings in the Azure portal to use ephemeral sessions, they will be able to use ephemeral sessions through Windows Terminal as well (with this change)
  • We continue to not provide any user flow to change the azure cloud shell settings (so the user cannot change between ephemeral or mounted sessions within Windows Terminal, the same way they cannot change the preferred shell type)
  • Users that haven't updated their azure cloud shell settings in the portal recently will be unaffected by this change (the new api-version is backward compatible with their old settings)
  • Users who have updated their azure cloud shell settings in the portal but don't update Terminal with this commit will be unable to use Azure Cloud Shell via Terminal - we will attempt to access their new settings with the old api-version and that will fail

@DHowett DHowett enabled auto-merge April 23, 2024 23:03
@DHowett DHowett added this pull request to the merge queue Apr 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Apr 24, 2024
@DHowett DHowett added this pull request to the merge queue Apr 24, 2024
@DHowett DHowett removed this pull request from the merge queue due to a manual request Apr 24, 2024
@DHowett DHowett merged commit ce4e0df into main Apr 24, 2024
18 of 20 checks passed
@DHowett DHowett deleted the dev/pabhoj/cloud_shell_update branch April 24, 2024 00:17
DHowett pushed a commit that referenced this pull request Apr 26, 2024
Updates the `api-version` to `2023-02-01-preview` when requesting for
CloudShell settings and shell

## Validation Steps Performed
Can still use Azure Cloud Shell through Windows Terminal

(cherry picked from commit ce4e0df)
Service-Card-Id: 92411551
Service-Version: 1.19
DHowett pushed a commit that referenced this pull request Apr 26, 2024
Updates the `api-version` to `2023-02-01-preview` when requesting for
CloudShell settings and shell

## Validation Steps Performed
Can still use Azure Cloud Shell through Windows Terminal

(cherry picked from commit ce4e0df)
Service-Card-Id: 92411552
Service-Version: 1.20
Copy link

@Zionsammy Zionsammy left a comment

Choose a reason for hiding this comment

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

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Cherry Picked
Development

Successfully merging this pull request may close these issues.

4 participants