Skip to content

Autmation: update pipeline & fix hive file copy #20

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
Merged
tuhaihe merged 5 commits into from
Dec 26, 2025
Merged

Autmation: update pipeline & fix hive file copy #20

tuhaihe merged 5 commits into from
Dec 26, 2025

Conversation

@ostinru
Copy link
Collaborator

@ostinru ostinru commented Dec 22, 2025 •
edited
Loading

This PR consist of two changes:

  1. Update CI pipeline to mark steps failed when we have failed tests.
  2. Add a timeout on test phase (2 hours) that is lower than whole pipeline timeout 6h. In case of hanging tests github actions will kill single step rather than whole pipeline. And as a result all artifacts (logs) will be uploaded)
  3. Save more logs to github artifacts.
  4. Bump testng version. TestNG before 6.9.5 marked all tests as SKIPPED when @BeforeClass methods raised an exception.
  5. Fix Hive tests, by fixing file copy check.

@ostinru
Automation: add timeout on test phase
cb43779 
Tests may stuck for some reason and github will kill CI pipeline leaving
us without any clue what happened.
This change limits time for each step to run. When timeout raised -
github will kill single step, and other steps will collect results and
will provide test reports with logs.
@ostinru
Copy link
Collaborator Author

ostinru commented Dec 22, 2025

@tuhaihe , should I sign CLA Agreement? Commit message template says that I should. However there is no description on how this process organized.

@tuhaihe
Copy link
Member

tuhaihe commented Dec 22, 2025

@tuhaihe , should I sign CLA Agreement? Commit message template says that I should. However there is no description on how this process organized.

Hey @ostinru there is no need to sign it under the ASF repos. Sorry for the confusion...

I will update the related files following the ASF policies.

@ostinru ostinru marked this pull request as ready for review December 22, 2025 10:41
MisterRaindrop
MisterRaindrop reviewed Dec 23, 2025
View reviewed changes
MisterRaindrop
MisterRaindrop approved these changes Dec 26, 2025
View reviewed changes
Copy link
Collaborator

@MisterRaindrop MisterRaindrop left a comment

Choose a reason for hiding this comment

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

LGTM

@tuhaihe tuhaihe merged commit 5394ead into apache:merge-with-upstream Dec 26, 2025
13 of 20 checks passed
@ostinru ostinru deleted the better-automation branch December 29, 2025 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@MisterRaindrop MisterRaindrop MisterRaindrop approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ostinru @tuhaihe @MisterRaindrop