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

[CI] Upload Test Log for CI failure shall contain trino engine log #2094 #2097

Closed
wants to merge 1 commit into from

Conversation

zhaomin1423
Copy link
Member

Why are the changes needed?

Upload Test Log for CI failure shall contain trino engine log.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before make a pull request

@yaooqinn yaooqinn added the kind:infra license, community building, project builds, asf infra related, etc. label Mar 11, 2022
@yaooqinn yaooqinn added this to the v1.6.0 milestone Mar 11, 2022
@yaooqinn yaooqinn closed this in c5a7c66 Mar 11, 2022
@yaooqinn
Copy link
Member

thanks, merged to master

wForget pushed a commit to wForget/kyuubi that referenced this pull request Mar 11, 2022
…n trino engine log apache#2094

### _Why are the changes needed?_

Upload Test Log for CI failure shall contain trino engine log.

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes apache#2097 from zhaomin1423/update_ci.

Closes apache#2097

5796174 [Min Zhao] [CI] Upload Test Log for CI failure shall contain trino engine log apache#2094

Authored-by: Min Zhao <[email protected]>
Signed-off-by: Kent Yao <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:infra license, community building, project builds, asf infra related, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants