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.

Sign up for GitHub

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

Jump to bottom

revert PR 22489 #23078

Merged
merged 2 commits into from
Feb 27, 2024
Merged

revert PR 22489 #23078

merged 2 commits into from
Feb 27, 2024

Conversation

wangleis
Copy link
Contributor

@wangleis wangleis commented Feb 26, 2024 •
edited by peterchen-intel
Loading

Details:

  • revert PR 22489

Tickets:

@wangleis wangleis requested review from a team as code owners February 26, 2024 08:19
@wangleis wangleis requested review from zKulesza and dmitry-gorokhov and removed request for a team February 26, 2024 08:19
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin category: CI OpenVINO public CI category: docs OpenVINO documentation github_actions Pull requests that update GitHub Actions code labels Feb 26, 2024
@dmitry-gorokhov dmitry-gorokhov added this to the 2024.1 milestone Feb 26, 2024
@wangleis wangleis enabled auto-merge February 26, 2024 13:31
@wangleis wangleis added this pull request to the merge queue Feb 27, 2024
Merged via the queue into openvinotoolkit:master with commit af5557f Feb 27, 2024
106 checks passed
@wangleis wangleis deleted the revert_22489 branch February 27, 2024 01:49
iefode
iefode reviewed Feb 29, 2024
View reviewed changes
src/tests/test_utils/functional_test_utils/layer_tests_summary/run_parallel.py
DEFAULT_PROCESS_TIMEOUT = 5400
DEFAULT_SUITE_TIMEOUT = 5400
DEFAULT_TEST_TIMEOUT = 5400
DEFAULT_PROCESS_TIMEOUT = 3600
Copy link
Contributor

Choose a reason for hiding this comment

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

@wangleis @dmitry-gorokhov
I will ask you not to do such changes in such utils without review from ie test infra maintainers. Such changes can affect GHA CI
CC @andrei-kochin

Copy link
Contributor Author

@wangleis wangleis Feb 29, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

@iefode PR 22489 is reverted in this PR and replaced by PR 23099 which do not change GHA CI setting. Thanks.

Copy link
Contributor

@iefode iefode Feb 29, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

@wangleis
FYI: this script is used to run func tests in GHA. It is the main reason why I asked you do not make such changes without approve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iefode iefode iefode left review comments

@dmitry-gorokhov dmitry-gorokhov dmitry-gorokhov approved these changes

@zKulesza zKulesza Awaiting requested review from zKulesza zKulesza is a code owner automatically assigned from openvinotoolkit/openvino-docs-maintainers

Assignees

@dmitry-gorokhov dmitry-gorokhov

Labels
category: CI OpenVINO public CI category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation category: IE Tests OpenVINO Test: plugins and common github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
2024.1
Development

Successfully merging this pull request may close these issues.
3 participants
@wangleis @dmitry-gorokhov @iefode