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

[WIP] 1.3 new UI #125

Merged
merged 8 commits into from
Jan 12, 2022
Merged

[WIP] 1.3 new UI #125

merged 8 commits into from
Jan 12, 2022

Conversation

lugi0
Copy link
Contributor

@lugi0 lugi0 commented Nov 22, 2021

Signed-off-by: Luca Giorgi lgiorgi@redhat.com

@lugi0
Copy link
Contributor Author

lugi0 commented Nov 22, 2021

Currently blocked by a bug, will test via jenkins job once it's updated

libs/Helpers.py Outdated Show resolved Hide resolved
@lugi0
Copy link
Contributor Author

lugi0 commented Nov 25, 2021

I've had to move from openshiftcli to running the oc command directly to fix a test failure due to a timing issue

jgarciao
jgarciao previously approved these changes Nov 26, 2021
libs/Helpers.py Outdated Show resolved Hide resolved
vasukulkarni
vasukulkarni previously approved these changes Nov 30, 2021
@vasukulkarni
Copy link
Contributor

Please feel free to merge after testing with current build.

Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@jgarciao jgarciao self-requested a review January 11, 2022 13:35
jgarciao
jgarciao previously approved these changes Jan 11, 2022
Copy link
Contributor

@jgarciao jgarciao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

bdattoma
bdattoma previously approved these changes Jan 11, 2022
@bdattoma
Copy link
Contributor

LGTM. could you run a jenkins job before merging?

@lugi0
Copy link
Contributor Author

lugi0 commented Jan 11, 2022

I think we will run smoke with this PR before merging

@jgarciao
Copy link
Contributor

I believe the keyword User Is Allowed is failing in 1.5.0 because it's using xpath:/html/body/div[1]/form/div/div/div[2], which I think doesn't exist any more.

Maybe you could add a new keyword "Wait Until Spawner Is Loaded" or similar and use it in User Is Allowed and Select Notebook Image

@jgarciao jgarciao self-requested a review January 11, 2022 17:56
@jgarciao jgarciao dismissed their stale review January 11, 2022 17:56

New changes requested in comments

Signed-off-by: Luca Giorgi <lgiorgi@redhat.com>
@lugi0
Copy link
Contributor Author

lugi0 commented Jan 12, 2022

@jgarciao quick and dirty fix for the keyword for now, we can think of a way to improve it later

Copy link
Contributor

@tarukumar tarukumar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@aloganat aloganat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

None yet

6 participants