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

Smoke test for storage-blob-changefeed is failing with authentication error #11402

Closed
praveenkuttappan opened this issue Sep 22, 2020 · 3 comments
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files) test-enhancement

Comments

@praveenkuttappan
Copy link
Member

Smoke test is failing for storage-blob-changefeed with below error. Currently smoke test is disabled for this package. We should reenable this test once issue in this sample testing is resolved.

##[error]Test Failed - Package: storage-blob-changefeed - Sample File:basic.js
Exception: RestError: NoAuthenticationInformationServer failed to authenticate the request. Please refer to the information in the www-authenticate header.
RequestId:7691e14f-a01e-003a-63b4-90b961000000
Time:2020-09-22T07:44:25.9554149Z

@praveenkuttappan praveenkuttappan added the Storage Storage Service (Queues, Blobs, Files) label Sep 22, 2020
@ljian3377
Copy link
Member

ljian3377 commented Sep 23, 2020

@praveenkuttappan

How is the process env set in the smoke-test environment? Does the account has the change feed feature enabled?
Which config is smoke-test using to create the accounts for testing?
Change feed samples use the credentials in the same way as other storage packages. I wonder why it is failing with authentication error.
There is some "The container ($blobchangefeed) does not exist" errors in the log. If that's the real cause, the it's probably due to the change feed feature isn't turned on.
https://xportal.trafficmanager.net/autoanalysis/report/8d80b293-7cd6-4df3-bea1-24bb966852b7

We haven't figured out how to enable live tests for change feed as the enabling may take some time to take effect. So skipping the change feed samples is the right thing to do for now till we figure it out.

import * as dotenv from "dotenv";
dotenv.config();

export async function main() {
  // Enter your storage account name and shared key
  const account = process.env.ACCOUNT_NAME || "";
  const accountKey = process.env.ACCOUNT_KEY || "";
...
}

@ljian3377 ljian3377 self-assigned this Sep 23, 2020
@mikeharder
Copy link
Member

The test account is deployed with this ARM template:

https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/storage/test-resources.json

@ramya-rao-a ramya-rao-a added the Client This issue points to a problem in the data-plane of the library. label Sep 23, 2020
@ljian3377 ljian3377 added this to the Backlog milestone Oct 21, 2020
@ramya-rao-a ramya-rao-a assigned EmmaZhu and unassigned ljian3377 Sep 22, 2021
@xirzec
Copy link
Member

xirzec commented Jan 11, 2023

Smoke tests have been disabled globally for now, future work is tracked here: #24440

@xirzec xirzec closed this as completed Jan 11, 2023
@xirzec xirzec removed this from the Backlog milestone Mar 31, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jun 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Storage Storage Service (Queues, Blobs, Files) test-enhancement
Projects
None yet
Development

No branches or pull requests

6 participants