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

.github: add timeout for individual tests + bump dependencies #890

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

evgeniiz321
Copy link
Contributor

No description provided.

Signed-off-by: Evgeniy Zayats <zayatsevgeniy@nspcc.io>
@evgeniiz321 evgeniiz321 force-pushed the ezayats/single-test-timeout branch 2 times, most recently from 514a6df to ed31363 Compare October 31, 2024 19:10
@evgeniiz321 evgeniiz321 marked this pull request as ready for review October 31, 2024 19:10
@evgeniiz321 evgeniiz321 changed the title .github: add timeout for individual tests .github: add timeout for individual tests + bump dependencies Oct 31, 2024
@evgeniiz321
Copy link
Contributor Author

@roman-khimov can you please check repo settings once again? Maybe we need to remove and add once again the rule for run_system_tests job.

@roman-khimov
Copy link
Member

Deleted/added back. Let's see how it goes.

@evgeniiz321
Copy link
Contributor Author

@roman-khimov I tried setting rules on my demo repo and I can select individual matrix jobs there:

image

can you do the same here?

@roman-khimov
Copy link
Member

Of course I can. But it allows to select parent entity as well which doesn't work. Weird.

.github/workflows/system-tests.yml Outdated Show resolved Hide resolved
.github/workflows/system-tests.yml Show resolved Hide resolved
.github/workflows/system-tests.yml Show resolved Hide resolved
Signed-off-by: Evgeniy Zayats <zayatsevgeniy@nspcc.io>
@evgeniiz321 evgeniiz321 merged commit b3d5d1a into master Nov 1, 2024
6 checks passed
@evgeniiz321 evgeniiz321 deleted the ezayats/single-test-timeout branch November 1, 2024 22:18
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.

2 participants