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] Github Action trigger condition optimized for cost saving #411

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

nwangfw
Copy link
Collaborator

@nwangfw nwangfw commented Nov 20, 2024

Pull Request Description

Github Action trigger condition is optimized. Now each test will only be triggered when the corresponding code is modified.

Related Issues

Resolves: #253

Important: Before submitting, please complete the description above and review the checklist below.

@nwangfw nwangfw force-pushed the ning/optimize-action-trigger branch 3 times, most recently from 6c52467 to 0b3da7b Compare November 20, 2024 18:33
@nwangfw
Copy link
Collaborator Author

nwangfw commented Nov 20, 2024

Let me know if some trigger conditions are not considered or wrongly considered.

@nwangfw nwangfw requested a review from Jeffwan November 20, 2024 18:42
@nwangfw nwangfw marked this pull request as ready for review November 20, 2024 18:48
@Jeffwan
Copy link
Collaborator

Jeffwan commented Nov 20, 2024

can you check .hack/https://update-codegen.sh and make sure that works as expected?

@@ -1,6 +1,7 @@
name: Release Build and Push

on:
workflow_dispatch: # Allows manual trigger
Copy link
Collaborator

Choose a reason for hiding this comment

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

this makes sense

@@ -1,11 +1,15 @@
name: Python Tests

on:
workflow_dispatch: # Allows manual trigger
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we want manual trigger here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

deleted

- 'go.mod'
- 'go.sum'
tags:
- 'v*'
Copy link
Collaborator

Choose a reason for hiding this comment

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

we don't push to tags (they are protected). This is something unnecessary I think.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

deleted

@nwangfw nwangfw force-pushed the ning/optimize-action-trigger branch from 0b3da7b to 3c4d13f Compare November 20, 2024 21:15
@nwangfw nwangfw force-pushed the ning/optimize-action-trigger branch from 3c4d13f to 83b4e14 Compare November 20, 2024 21:17
@nwangfw
Copy link
Collaborator Author

nwangfw commented Nov 20, 2024

can you check .hack/https://update-codegen.sh and make sure that works as expected?

tested.

@nwangfw nwangfw requested a review from Jeffwan November 20, 2024 21:22
Copy link
Collaborator

@Jeffwan Jeffwan left a comment

Choose a reason for hiding this comment

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

/lgtm

We have an offline discussion. CI always build new manifest if there's a change merged in. We won't save cost on the post-submit, otherwise, it's hard to track back to the commit if we want to pin version for debug purpose.

@Jeffwan Jeffwan merged commit 91a7acc into main Nov 20, 2024
2 checks passed
@Jeffwan Jeffwan deleted the ning/optimize-action-trigger branch November 20, 2024 22:33
gangmuk pushed a commit that referenced this pull request Jan 25, 2025
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.

[CI] Optimize the cost of Github actions by triggering necessary tests
2 participants