Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Disable laop_6 #15976

Merged
merged 1 commit into from
Aug 26, 2019
Merged

Disable laop_6 #15976

merged 1 commit into from
Aug 26, 2019

Conversation

marcoabreu
Copy link
Contributor

Crashes due to #15975

Copy link
Contributor

@ChaiBapchya ChaiBapchya left a comment

Choose a reason for hiding this comment

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

LGTM!

@larroy
Copy link
Contributor

larroy commented Aug 22, 2019

What's the root cause of the crash? this test was not crashing not long ago.
Disabling a test that crashes willy-nilly should have some good justification. What's next?

@marcoabreu
Copy link
Contributor Author

Next step is to investigate the crash, fix it and re-enable the test.

@gyshi
Copy link
Contributor

gyshi commented Aug 24, 2019

my ci has an error with laop_6 ,
#15976,
you will see detail messages in my ci tests

@marcoabreu marcoabreu merged commit 7d8c2ec into master Aug 26, 2019
@szha szha deleted the disable-laop-6 branch September 8, 2019 03:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants