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

Fail to save the changed access level for one blob container #380

Closed
v-xuanzh opened this issue Jul 4, 2018 · 5 comments
Closed

Fail to save the changed access level for one blob container #380

v-xuanzh opened this issue Jul 4, 2018 · 5 comments
Assignees
Labels
⚙️ blobs Related to blob storage 🧪 testing Found through regular testing ✅ merged A fix for this issue has been merged
Milestone

Comments

@v-xuanzh
Copy link

v-xuanzh commented Jul 4, 2018

Storage Explorer Version: 1.3.0
Platform/OS Version: Windows 10/MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32/x64
Build Number: 20180704.1
Commit: 865aab62
Regression From: Works well on previous release 1.2.0(20180604.2)

Steps to Reproduce:

  1. Go to subscription -> Storage Accounts.
  2. Right click one blob container ->Change the default access level -> 'Save'.
  3. Reopen the 'Set Container Public Access Level' dialog then check whether the changed level is saved or not.

Expected Experience:

The changed access level is saved.

Actual Experience:

The changed access level isn't saved.

@craxal
Copy link
Contributor

craxal commented Jul 5, 2018

Did you wait for the activity to complete? Did you refresh the node? After doing these things, I can confirm that the access policy is indeed set.

@MRayermannMSFT, we should consider refreshing the node automatically when the access level is set. We're getting the access level from an attribute, which needs to be refreshed if the access level is changed.

@MRayermannMSFT
Copy link
Member

Yes. This is the same issue as the access tier. Let me try and find if there is a way to do it.

@MRayermannMSFT MRayermannMSFT added this to the 1.3.0 milestone Jul 5, 2018
@MRayermannMSFT
Copy link
Member

Ok @v-xuanzh , into the 1.3.0 branch I've merged a change that refreshes the public access level property after you change it (there may be a slight delay, just give it a sec). It'll be in the build I send out tonight.

@MRayermannMSFT MRayermannMSFT added the ✅ merged A fix for this issue has been merged label Jul 6, 2018
@v-xuanzh
Copy link
Author

v-xuanzh commented Jul 6, 2018

Thanks for your updating.
Now the access level property can auto refresh after changing it. :)

@v-xuanzh
Copy link
Author

Hi @MRayermannMSFT , the changed access level property doesn't auto refresh aftering waiting about 3 minutes, and need to manually refresh the blob container.
So we think it may mislead customers if they forget to refresh it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ blobs Related to blob 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