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.

Sign up for GitHub

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

Jump to bottom

Show a failed activity log after deleting one blob snapshot successfully #2402

Closed
v-limu opened this issue Dec 9, 2019 · 2 comments
Closed

Show a failed activity log after deleting one blob snapshot successfully #2402

v-limu opened this issue Dec 9, 2019 · 2 comments
Assignees
@JasonYeMSFT
Labels
🪲 regression Issue was working in a previous version ⚙️ blobs Related to blob storage ❔ external Root cause of this issue is in another component, product, or service 📦 electron update required Requires updating to a newer version of electron 🧪 testing Found through regular testing
Projects
Storage Explorer
Milestone
1.14.0

Comments

@v-limu
Copy link

v-limu commented Dec 9, 2019

Storage Explorer Version: 1.11.1
Build: 20191206.5
Branch: dev/marayerm/electron7/packaging
Platform/OS: Windows 10/Linux Ubuntu 18.04
Architecture: ia32
Regression From: Previous build (20191206.2)

Steps to reproduce:

  1. Expand one storage account -> Blob Containers.
  2. Create a new blob container -> Upload a blob to it -> Create snapshot for the blob.
  3. Manage Snapshots -> Try to delete the snapshot.
  4. Check the result.

Expect Experience:
Show a successful activity log.

Actual Experience:

  1. Succeed to delete the blob snapshot.
  2. The activity log shows 'Failed to delete...'.
    image

More Info:
This issue doesn't reproduce on MacOS.
@v-limu v-limu added 🧪 testing Found through regular testing ⚙️ blobs Related to blob storage 🪲 regression Issue was working in a previous version 💻 linux Related to Linux platforms 💻 windows Related to Windows platforms labels Dec 9, 2019
@MRayermannMSFT MRayermannMSFT added this to the 1.12.0 milestone Dec 9, 2019
@MRayermannMSFT MRayermannMSFT added this to Committed in Storage Explorer via automation Dec 9, 2019
@JasonYeMSFT JasonYeMSFT moved this from Committed to Investigating in Storage Explorer Dec 12, 2019
@JasonYeMSFT JasonYeMSFT moved this from Investigating to Under Review in Storage Explorer Dec 16, 2019
@JasonYeMSFT
Copy link
Contributor

The issue is caused by an Electron bug electron/electron#21515.

A workaround for this bug will be implemented.

@JasonYeMSFT JasonYeMSFT moved this from Under Review to Verifying in Storage Explorer Dec 19, 2019
@JasonYeMSFT JasonYeMSFT moved this from Verifying to Investigating in Storage Explorer Dec 19, 2019
@JasonYeMSFT JasonYeMSFT moved this from Investigating to Blocked in Storage Explorer Dec 19, 2019
@JasonYeMSFT JasonYeMSFT added ❔ external Root cause of this issue is in another component, product, or service and removed 💻 linux Related to Linux platforms 💻 windows Related to Windows platforms labels Dec 19, 2019
@JasonYeMSFT JasonYeMSFT modified the milestones: 1.12.0, 1.13.0 Jan 27, 2020
@MRayermannMSFT MRayermannMSFT added the 📦 electron update required Requires updating to a newer version of electron label Mar 2, 2020
@MRayermannMSFT MRayermannMSFT modified the milestones: 1.13.0, 1.14.0 Mar 2, 2020
@v-xianya
Copy link
Member

v-xianya commented May 7, 2020

This issue is fixed on build electron/8-20200504.7. So we close it.

@v-xianya v-xianya closed this as completed May 7, 2020
Storage Explorer automation moved this from Blocked to Done May 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JasonYeMSFT JasonYeMSFT

Labels
🪲 regression Issue was working in a previous version ⚙️ blobs Related to blob storage ❔ external Root cause of this issue is in another component, product, or service 📦 electron update required Requires updating to a newer version of electron 🧪 testing Found through regular testing
Projects
Storage Explorer
  
Done
Milestone
1.14.0
Development

No branches or pull requests
4 participants
@MRayermannMSFT @JasonYeMSFT @v-xianya @v-limu