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

The cloned table is opened under the SAS attached storage account when cloning one table under the subscription storage account #5628

Closed
v-kellyluo opened this issue Apr 14, 2022 · 2 comments
Assignees
Labels
🪲 regression Issue was working in a previous version ⚙️ tables Related to table storage 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone

Comments

@v-kellyluo
Copy link

Storage Explorer Version: 1.24.0-dev
Build Number: 20220414.17
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.2.1(MacBook Air M1, 2020)
Architecture: ia32\x64
How Found: Ad-hoc testing
Regression From: Previous release (1.22.1)

Steps to Reproduce

  1. Expand one storage account -> Tables -> Create a table.
  2. SAS attach the storage account.
  3. Go to the original storage account -> Tables.
  4. Select and right click one table -> Click 'Clone...'.
  5. Input a valid name -> Click 'Clone'.
  6. Check whether the table is opened under the subscription storage account.

Expected Experience

The table is opened under the subscription storage account.

Actual Experience

The table is opened under the SAS attached storage account.

Additional Context

  1. This issue doesn't reproduce for blob containers/file shares.
  2. This issue doesn't reproduce for Name & Key attached storage accounts.
  3. Here is the record:
    table
@v-kellyluo v-kellyluo added 🧪 testing Found through regular testing ⚙️ tables Related to table storage 🪲 regression Issue was working in a previous version labels Apr 14, 2022
@craxal craxal self-assigned this Apr 14, 2022
@craxal craxal added this to the 1.24.0 milestone Apr 14, 2022
@craxal
Copy link
Contributor

craxal commented Apr 14, 2022

The logic for getting the parent node of the table node we're cloning wasn't strict enough. This is easily solved by using the "get parent node" API we already have.

@craxal craxal added the ✅ merged A fix for this issue has been merged label Apr 15, 2022
@craxal craxal closed this as completed Apr 15, 2022
@v-xianya
Copy link
Member

Verified this issue on the build 20220418.1. Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🪲 regression Issue was working in a previous version ⚙️ tables Related to table storage 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

3 participants