-
Notifications
You must be signed in to change notification settings - Fork 166
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
WIP: Added onboarding token generation related functional tests for provider-client feature #9938
base: master
Are you sure you want to change the base?
WIP: Added onboarding token generation related functional tests for provider-client feature #9938
Conversation
cc42719
to
9020bb9
Compare
|
||
|
||
@tier1 | ||
@green_squad |
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.
This is not a green squad test. May be yellow squad ?
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.
Have updated in latest commit.
@pytest.fixture(autouse=True) | ||
def setup(self, request): | ||
""" | ||
Resetting the default value of KeyRotation |
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.
Any code missing in this method for resetting?
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.
Addressed in latest commit.
Test to verify the respin of ux pod. | ||
Steps: |
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.
Test to verify the respin of ux pod. | |
Steps: | |
Test to verify the respin of ux pod. | |
Steps: |
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.
Addressed in latest commit.
timeout=180, | ||
), "ux pod is not in running state as expected" | ||
|
||
def test_respin_of_ux_server_pod(self): |
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.
This is a tier4c test. Please add marker in the test case level for all test cases.
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.
Addressed in latest commit.
selector=constants.UX_BACKEND_SERVER_LABEL, | ||
resource_count=1, | ||
timeout=180, | ||
), "ux pod is not in running state as expected" |
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.
), "ux pod is not in running state as expected" | |
), "ux-backend-server pod is not in running state as expected" |
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.
Addressed in latest commit.
""" | ||
Test to verify the respin of ux pod. | ||
Steps: | ||
1. navigate to storage-->storage clients page |
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.
Please mention the step of deleting the ux-backend-server pod.
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.
Addressed in latest commit.
|
||
ux_pod_obj.delete() | ||
log.info("wait some time for another ux-backend-server pod to come up") | ||
time.sleep(30) |
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.
The new pod will come up in 3-5 seconds. Please remove sleep for short period and use wait conditions for checking the new pod.
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.
Addressed in latest commit.
timeout=180, | ||
), "ux pod is not in running state as expected" | ||
|
||
def test_respin_of_ux_server_pod(self): |
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.
I think a separate test case to check the re-spin of pod is not needed. This will be part of tier4c pod re-spin tests. Please combine the test with test_onboarding_token_generation_option_is_available_in_ui(and change the name of test) to verify the impact of ux pod respin on token generation.
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.
I have combined the test to check ux-backend-server pod is urunning and ux-backend-server pod respin testcases together.
Could you please include a test run? |
5d81873
to
65e6515
Compare
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
…client from the hosted cluster Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
…oken with limited storage quota Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
…timeout for local-cluster load Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
…own' locator in navigate_to_local_cluster method Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
…_rpm_package method Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
119ac5d
to
ea6ce94
Compare
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
d581402
to
205ec51
Compare
Signed-off-by: Amrita Mahapatra <49347640+amr1ta@users.noreply.github.com>
205ec51
to
a9ac0cb
Compare
Added onboarding token generation related functional tests for provider-client feature