-
Notifications
You must be signed in to change notification settings - Fork 335
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
[Bug report] Gravitino should trigger backend CI pipeleine when module integration-test-common
is changed.
#3207
Comments
qqqttt123
pushed a commit
that referenced
this issue
Apr 29, 2024
…ntegration-test-common` (#3201) ### What changes were proposed in this pull request? Trigger integration test when there is a change in module `integration-test-common`. ### Why are the changes needed? We should start the integration test if there are changes in module `integration-test-common`. however, CI https://github.com/datastrato/gravitino/actions/runs/8872110159/job/24356012525?pr=3197 of PR #3197 can't start the CI pipeline. Fixed: #3207 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? CI passed.
github-actions bot
pushed a commit
that referenced
this issue
Apr 29, 2024
…ntegration-test-common` (#3201) ### What changes were proposed in this pull request? Trigger integration test when there is a change in module `integration-test-common`. ### Why are the changes needed? We should start the integration test if there are changes in module `integration-test-common`. however, CI https://github.com/datastrato/gravitino/actions/runs/8872110159/job/24356012525?pr=3197 of PR #3197 can't start the CI pipeline. Fixed: #3207 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? CI passed.
qqqttt123
pushed a commit
that referenced
this issue
Apr 29, 2024
…ntegration-test-common` (#3208) ### What changes were proposed in this pull request? Trigger integration test when there is a change in module `integration-test-common`. ### Why are the changes needed? We should start the integration test if there are changes in module `integration-test-common`. however, CI https://github.com/datastrato/gravitino/actions/runs/8872110159/job/24356012525?pr=3197 of PR #3197 can't start the CI pipeline. Fixed: #3207 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? CI passed. Co-authored-by: Qi Yu <yuqi@datastrato.com>
diqiu50
pushed a commit
to diqiu50/gravitino
that referenced
this issue
Jun 13, 2024
…ule `integration-test-common` (apache#3201) ### What changes were proposed in this pull request? Trigger integration test when there is a change in module `integration-test-common`. ### Why are the changes needed? We should start the integration test if there are changes in module `integration-test-common`. however, CI https://github.com/datastrato/gravitino/actions/runs/8872110159/job/24356012525?pr=3197 of PR apache#3197 can't start the CI pipeline. Fixed: apache#3207 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? CI passed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version
main branch
Describe what's wrong
We should start the integration test if there are changes in module
integration-test-common
. however, CI https://github.com/datastrato/gravitino/actions/runs/8872110159/job/24356012525?pr=3197 of PR #3197 can't start the CI pipeline.Error message and/or stacktrace
Please see #3197
How to reproduce
Please see above.
Additional context
No response
The text was updated successfully, but these errors were encountered: