-
Notifications
You must be signed in to change notification settings - Fork 0
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
Make on-demand integration between EE and SDK #296
Comments
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 23, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 23, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 23, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
added a commit
to tarantool/actions
that referenced
this issue
Dec 26, 2022
Add an action for removing branches and other refs in GitHub repositories. Part of tarantool/tarantool-qa#296
NickVolynkin
changed the title
Make on-demand integration testing with Enterprise products
Make on-demand integration between EE and SDK
Jan 10, 2023
Resoved by https://github.com/tarantool/tarantool-ee/pull/311. There's a lot more to do, and it will be done as a separate epic #302 |
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Make a workflow for integration testing of Tarantool CE with Tarantool EE and SDK.
Pull requests
CI:
Tools:
Experiments and testing:
Choice matrix
sdk-test
sdk-test-intro
sdk-test-intro
sdk-test-intro
sdk-test
sdk-test-intro
sdk-test-intro
sdk-test
sdk-test-intro
sdk-test-cleanup
sdk-test-cleanup
sdk-test
sdk-test-cleanup
sdk-test
sdk-test
sdk-test
* when a PR is opened with a label, two events trigger:
pull_request.opened
,and immediately after it
pull_request.labeled
. Both will trigger a workflow run, butthe first one will be canceled due to concurrency rules. This logic is vulnerable when
there is more than one label on the PR. So the common rule is this: don't add other labels
at the same time as 'sdk-test'.
Examples
Introductory comment
Introductory comment in a fork-based PR
Integration testing in progress
Commit message
Reference
Subtasks:
sdk-test
is removed.alice/foo
andbob/foo
.event.pull_request.merge_commit_sha
)pull_request.head.sha
, skip or raise an error?Test-branch: make a link back to the PR. Is it useful? It creates extra noise in the PR.Don't.github.base_ref
value).Feedback: when PR is opened from a fork, make a notice thatsdk-test
doesn't work with forks.Feedback: add a notice about branch being not on top of stable branch (needs rebase). After merging such branch, integration tests may break even if they were okay on the branch.No need, as we're usingpull_request.merge_commit_sha
.&cachebust=${{ github.sha }}
.This will require triggering these workflows with
repository_dispatch
and then getting results back. So, there are subtasks:repository_dispatch
The text was updated successfully, but these errors were encountered: