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

[Tests]: Add new access service #1939

Open
wants to merge 59 commits into
base: main
Choose a base branch
from
Open

[Tests]: Add new access service #1939

wants to merge 59 commits into from
+1,193 −35

Conversation

jkuradobery
Copy link
Collaborator

No description provided.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 2, 2024

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit c7a0621.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5722 5722 0 0 0 0

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 3, 2024

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 832ca85.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5723 5719 0 0 2 2

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 9, 2024

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 96715a4.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5722 5719 0 0 1 2

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 3f3b35f.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5722 5719 0 0 1 2

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 2a5c18f.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5722 5722 0 0 0 0

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 1678c6d.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5722 5721 0 1 0 0

@jkuradobery jkuradobery marked this pull request as ready for review September 21, 2024 19:19
@jkuradobery jkuradobery force-pushed the users/jkuradobery/add-iam-test-to-nbs branch from f2bf591 to 643d069 Compare September 21, 2024 19:22
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 643d069.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5818 5817 0 1 0 0

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 9ff1ae7.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5818 5818 0 0 0 0

@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit c20131c.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5818 5818 0 0 0 0

@jkuradobery
Add the folder id to the test env variable
96a5d81
@jkuradobery
Fix ya.makes
efbfb34
@jkuradobery
Rename ACCESS_SERVICE_VERSION -> ACCESS_SERVICE_TYPE
c08ac2a
@jkuradobery
Fix ya.make peerdir
0f94f58
@jkuradobery
Fix review discussions
441a3a1
@jkuradobery
Fix review discussions
fc7ced6
@jkuradobery
Return subprocess.Result in NfsCliClient.create()
eef1706 
It did return just stdout, add flag return_stdout to fix this.
@jkuradobery
Introduce auth tests for the filestore
bdfb726
@jkuradobery
User recipe even without test token
56ae5e8
@jkuradobery
Add config, use secure port for auth tests
712e19a
@jkuradobery
Use correct recipes for new access service
960d21e
@jkuradobery
Use client config, Fix TLS
1bdde02
@jkuradobery
Add logs in new access service mock
b76a922
@jkuradobery
Fix review discussions
329b31b
@jkuradobery
Fix review discussions
554f984
@jkuradobery
Fix review discussions
fb364a1
@jkuradobery
Print error json in filestoreclient.create
2c8e4d4
@jkuradobery
Fix permissions in filestore auth test
bedd487
@jkuradobery
Use json filestore client output in test
68b8d4c
@jkuradobery
Fix filestore auth test hanging
5612445
@jkuradobery
Fix style, add comments
55e9a51
@jkuradobery
Replace test_auth_wrong_token() with test_auth_empty_token()
9b2f732 
Filestore uses only authorize call, so it is either authorized or unauthorized, and the mock just authorizes all requests if "authorize" put mock handler was called.
@jkuradobery
Remove set RetryTimeout to 1 in tests
5325004
@jkuradobery
Fix review discussions
f3ce706
@jkuradobery
Remove pid check
78c5f7e
@jkuradobery
Remove test_auth_empty_token()
a306c57 
Filestore client can't use empty token and tries default token path anyway.
@jkuradobery
Fix flake8
ee0ada5
@jkuradobery jkuradobery force-pushed the users/jkuradobery/add-iam-test-to-nbs branch from fcf4ce7 to ee0ada5 Compare September 26, 2024 18:24
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit ee0ada5.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6185 6185 0 0 0 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SvartMetal SvartMetal SvartMetal left review comments

@qkrorlqr qkrorlqr qkrorlqr approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
large-tests Launch large tests for PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jkuradobery @qkrorlqr @SvartMetal