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

HADOOP-17191. ABFS: Run the tests with various combinations of configurations and publish a consolidated results #2597

Merged
merged 11 commits into from
Mar 10, 2021

Conversation

sumangala-patki
Copy link
Contributor

  • Contributed by Bilahari T H

(cherry picked from commit 4c033ba)

…urations and publish a consolidated results

- Contributed by Bilahari T H

(cherry picked from commit 4c033ba)
@sumangala-patki
Copy link
Contributor Author

sumangala-patki commented Jan 6, 2021

TEST RESULTS

HNS Account Location: East US 2
NonHNS Account Location: East US 2, Central US

HNS OAuth

[INFO] Tests run: 87, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 450, Failures: 0, Errors: 0, Skipped: 73
[WARNING] Tests run: 207, Failures: 0, Errors: 0, Skipped: 24
HNS SharedKey

[INFO] Tests run: 87, Failures: 0, Errors: 0, Skipped: 0
[ERROR]   ITestAzureBlobFileSystemDelegationSAS.testCheckAccess:94 » AbfsRestOperation O...
[ERROR]   ITestAzureBlobFileSystemDelegationSAS.testRootPath:343 » AbfsRestOperation Ope...
[ERROR] Tests run: 450, Failures: 0, Errors: 2, Skipped: 33
[WARNING] Tests run: 207, Failures: 0, Errors: 0, Skipped: 16
Non-HNS SharedKey

[INFO] Tests run: 87, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 450, Failures: 0, Errors: 0, Skipped: 245
[WARNING] Tests run: 207, Failures: 0, Errors: 0, Skipped: 16

Errors tracked in JIRA Delegation SAS

@sumangala-patki
Copy link
Contributor Author

TEST RESULTS

HNS Account Location: East US 2
NonHNS Account Location: East US 2, Central US

HNS OAuth

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 68
[WARNING] Tests run: 207, Failures: 0, Errors: 0, Skipped: 24

HNS SharedKey

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 24
[WARNING] Tests run: 207, Failures: 0, Errors: 0, Skipped: 16

Non-HNS SharedKey

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 249
[WARNING] Tests run: 207, Failures: 0, Errors: 0, Skipped: 16

@sumangala-patki
Copy link
Contributor Author

TEST RESULTS

HNS Account Location: East US 2
NonHNS Account Location: East US 2, Central US

HNS OAuth

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 431, Failures: 0, Errors: 0, Skipped: 68
[WARNING] Tests run: 204, Failures: 0, Errors: 0, Skipped: 24

HNS SharedKey

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 24
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 16

Non-HNS SharedKey

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 456, Failures: 0, Errors: 0, Skipped: 247
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 16

@sumangala-patki sumangala-patki marked this pull request as ready for review January 13, 2021 09:44
@sumangala-patki
Copy link
Contributor Author

Yetus shellcheck warnings about unused variables in runtests.sh are not valid. Variables declared in the runtests script are used in testsupport.sh; this is evident from test results that contain each of the three combinations (HNS-OAuth, HNS-SharedKey, NonHNS-SharedKey) specified in the runtests script.

@sumangala-patki
Copy link
Contributor Author

Hi @steveloughran, request you to review this for backporting to branch-3.3
[The yetus warnings are possibly due to an older version of shellcheck used for this branch (0.3.7), while trunk had 0.4.6. Let me know if you have any concerns about it, or if we should suppress these warnings in the code.]

@apache apache deleted a comment from hadoop-yetus Mar 9, 2021
@apache apache deleted a comment from hadoop-yetus Mar 9, 2021
@apache apache deleted a comment from hadoop-yetus Mar 9, 2021
@apache apache deleted a comment from hadoop-yetus Mar 9, 2021
@apache apache deleted a comment from hadoop-yetus Mar 9, 2021
@apache apache deleted a comment from hadoop-yetus Mar 9, 2021
@steveloughran
Copy link
Contributor

I'm not worried about shellcheck if its some version mismatch there.

Is the change actually enabling a check which is disabled on the later branch? if so: interesting.

Do you think this is ready to merge? If so, do a rebase, run the tests, push up the rebased PR and I'll do the merge

@apache apache deleted a comment from hadoop-yetus Mar 10, 2021
@steveloughran steveloughran merged commit cdaa644 into apache:branch-3.3 Mar 10, 2021
jojochuang pushed a commit to jojochuang/hadoop that referenced this pull request May 23, 2023
…ns of configurations and publish a consolidated results (apache#2597)

Contributed by Bilahari T H and Sumangala Patki

Change-Id: Ib85e59dc17a6b54a274c6e4364bfdc5ccca7ea9b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants