-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
[TestCase] Fix TestAccDashboardGrafana_withSku #24128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@neil-yechenwei this sku was added just 2 weeks ago and is also available in the portal. This could be region specific or there's an issue with the API, can you please find out which is the case?
@stephybun , it's weird. The "essential" value was removed from this commit #24014. I add it back and change the test location. Now test case passed. Thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🔄
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Recently TestAccDashboardGrafana_withSku is failed with below error so that I submitted this PR to fix it.
Error Message returned by API:
Below is the test result after the fix is applied.