Skip to content

Conversation

@lhotari
Copy link
Member

@lhotari lhotari commented Apr 1, 2021

Motivation

Currently it's not possible to find out specific reasons why integration tests fail since the log files aren't available.

Modifications

Add changes to upload container logs and surefire reports when integration tests fail or timeout (get cancelled).

Copy link
Member

@michaeljmarshall michaeljmarshall left a comment

Choose a reason for hiding this comment

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

LGTM

@lhotari
Copy link
Member Author

lhotari commented Apr 1, 2021

/pulsarbot run-failure-checks

@sijie sijie added this to the 2.8.0 milestone Apr 1, 2021
@lhotari
Copy link
Member Author

lhotari commented Apr 1, 2021

/pulsarbot run-failure-checks

2 similar comments
@lhotari
Copy link
Member Author

lhotari commented Apr 2, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 2, 2021

/pulsarbot run-failure-checks

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

LGTM

@lhotari
Copy link
Member Author

lhotari commented Apr 2, 2021

/pulsarbot run-failure-checks

3 similar comments
@lhotari
Copy link
Member Author

lhotari commented Apr 2, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 2, 2021

/pulsarbot run-failure-checks

@lhotari
Copy link
Member Author

lhotari commented Apr 2, 2021

/pulsarbot run-failure-checks

@codelipenghui codelipenghui merged commit e72df2c into apache:master Apr 4, 2021
wangjialing218 pushed a commit to wangjialing218/pulsar that referenced this pull request Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants