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

More debug info for failed integration selenium tests during CI. #11217

Merged
merged 2 commits into from
Jan 25, 2021
Merged

More debug info for failed integration selenium tests during CI. #11217

merged 2 commits into from
Jan 25, 2021

Conversation

jmchilton
Copy link
Member

No description provided.

@github-actions github-actions bot added this to the 21.05 milestone Jan 22, 2021
@jmchilton jmchilton changed the title More debugging integration selenium More debug info for failed integration selenium tests during CI. Jan 22, 2021
@jmchilton jmchilton mentioned this pull request Jan 22, 2021
Merged
nsoranzo
nsoranzo reviewed Jan 22, 2021
View reviewed changes
Copy link
Member

@nsoranzo nsoranzo left a comment

Choose a reason for hiding this comment

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

I've changed the artifact name to distinguish it from the one above, but not sure if it's the best name.

Do you want to rebase to remove the merged commits?

.github/workflows/integration_selenium.yaml Outdated Show resolved Hide resolved
@jmchilton
Copy link
Member Author

@OlegZharkov Are you able to see the debug information? The test you added is failing a lot of CI jobs - any clues how to improve it?

@OlegZharkov
Copy link
Contributor

OlegZharkov commented Jan 25, 2021 •
edited
Loading

@OlegZharkov Are you able to see the debug information? The test you added is failing a lot of CI jobs - any clues how to improve it?

Hi. Sorry, didn't realize. I will investigate.

Edit:
This is very weird, cannot make it fail locally. There should be something wrong about it

@nsoranzo nsoranzo force-pushed the more_debugging_integration_selenium branch from d72a626 to fe2b963 Compare January 25, 2021 19:09
@nsoranzo nsoranzo merged commit fca1d13 into galaxyproject:dev Jan 25, 2021
@nsoranzo nsoranzo deleted the more_debugging_integration_selenium branch January 25, 2021 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nsoranzo nsoranzo nsoranzo left review comments

Assignees
No one assigned
Labels
area/testing/integration area/testing/selenium kind/enhancement
Projects
None yet
Milestone
21.05
Development

Successfully merging this pull request may close these issues.
3 participants
@jmchilton @OlegZharkov @nsoranzo