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

[ISSUE #6742]Support daily build tests #6744

Merged
merged 18 commits into from
Jul 3, 2023
Merged

Conversation

cryptoya
Copy link
Contributor

@cryptoya cryptoya commented May 12, 2023

Which Issue(s) This PR Fixes

Fixes #6742

Brief Description

  • Add a e2e test in workflow, when run test failure, the SNAPSHOT package will not deploy to repository
  • Changed maxUniqueSnapshots settings for Maven in order to go back over the last 2 months of build history
  • Changed the SNAPSHOT version name of the develop branch build to prevent package overwriting that might result from other branch builds

How Did You Test This Change?

image
image

The last step requires you to upload the package using the apache repository permissions

@codecov-commenter
Copy link

codecov-commenter commented May 12, 2023

Codecov Report

Merging #6744 (22f3aac) into develop (aaa4a4b) will increase coverage by 0.09%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #6744      +/-   ##
=============================================
+ Coverage      42.64%   42.74%   +0.09%     
- Complexity      9106     9160      +54     
=============================================
  Files           1127     1128       +1     
  Lines          79942    80129     +187     
  Branches       10392    10432      +40     
=============================================
+ Hits           34094    34248     +154     
- Misses         41572    41602      +30     
- Partials        4276     4279       +3     

see 55 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@tsunghanjacktsai tsunghanjacktsai left a comment

Choose a reason for hiding this comment

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

LGTM. Guess we could try if it could work.

Copy link
Member

@lizhimins lizhimins left a comment

Choose a reason for hiding this comment

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

File should add licensed to the ASF

@aaron-ai
Copy link
Member

How can GitHub Action's daily builds help us identify which commit introduced the issue?

caigy
caigy previously requested changes May 24, 2023
.github/workflows/snapshot-automation.yml Outdated Show resolved Hide resolved
Copy link
Member

@aaron-ai aaron-ai left a comment

Choose a reason for hiding this comment

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

Works for me, Thanks @cryptoya !

aaron-ai
aaron-ai previously approved these changes Jun 14, 2023
@lizhimins lizhimins dismissed caigy’s stale review July 3, 2023 06:54

already resolved

@lizhimins lizhimins merged commit 5c3bd98 into apache:develop Jul 3, 2023
lizhimins pushed a commit to lizhimins/rocketmq that referenced this pull request Jul 26, 2023
* Update to support "Snapshot Release" daily test
* Modify 'snapshot-automation.yml'
* Add Apache header
* Supports manual triggering through branch and commitId
* Remove the proxy in servers of github actions.
* Update docker repo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement] Support daily build tests
6 participants