Skip to content

Commit a1d5d5c

Browse files
committed
ci: rename local idf_ci folder, avoid name collision
1 parent fbc38a9 commit a1d5d5c

File tree

12 files changed

+213
-231
lines changed

12 files changed

+213
-231
lines changed

conftest.py

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -36,20 +36,23 @@
3636
from _pytest.fixtures import FixtureRequest
3737
from artifacts_handler import ArtifactType
3838
from dynamic_pipelines.constants import TEST_RELATED_APPS_DOWNLOAD_URLS_FILENAME
39-
from idf_ci.app import import_apps_from_txt
40-
from idf_ci.uploader import AppDownloader, AppUploader
41-
from idf_ci_utils import IDF_PATH, idf_relpath
42-
from idf_pytest.constants import (
43-
DEFAULT_SDKCONFIG,
44-
ENV_MARKERS,
45-
SPECIAL_MARKERS,
46-
TARGET_MARKERS,
47-
PytestCase,
48-
DEFAULT_LOGDIR,
49-
)
50-
from idf_pytest.plugin import IDF_PYTEST_EMBEDDED_KEY, ITEM_PYTEST_CASE_KEY, IdfPytestEmbedded
39+
from idf_ci_local.app import import_apps_from_txt
40+
from idf_ci_local.uploader import AppDownloader
41+
from idf_ci_local.uploader import AppUploader
42+
from idf_ci_utils import IDF_PATH
43+
from idf_ci_utils import idf_relpath
44+
from idf_pytest.constants import DEFAULT_LOGDIR
45+
from idf_pytest.constants import DEFAULT_SDKCONFIG
46+
from idf_pytest.constants import ENV_MARKERS
47+
from idf_pytest.constants import SPECIAL_MARKERS
48+
from idf_pytest.constants import TARGET_MARKERS
49+
from idf_pytest.constants import PytestCase
50+
from idf_pytest.plugin import IDF_PYTEST_EMBEDDED_KEY
51+
from idf_pytest.plugin import ITEM_PYTEST_CASE_KEY
52+
from idf_pytest.plugin import IdfPytestEmbedded
5153
from idf_pytest.utils import format_case_id
52-
from pytest_embedded.plugin import multi_dut_argument, multi_dut_fixture
54+
from pytest_embedded.plugin import multi_dut_argument
55+
from pytest_embedded.plugin import multi_dut_fixture
5356
from pytest_embedded_idf.dut import IdfDut
5457
from pytest_embedded_idf.unity_tester import CaseTester
5558

@@ -426,8 +429,10 @@ def pytest_addoption(parser: pytest.Parser) -> None:
426429

427430

428431
def pytest_configure(config: Config) -> None:
429-
from pytest_embedded_idf.utils import supported_targets, preview_targets
430-
from idf_pytest.constants import SUPPORTED_TARGETS, PREVIEW_TARGETS
432+
from idf_pytest.constants import PREVIEW_TARGETS
433+
from idf_pytest.constants import SUPPORTED_TARGETS
434+
from pytest_embedded_idf.utils import preview_targets
435+
from pytest_embedded_idf.utils import supported_targets
431436

432437
supported_targets.set(SUPPORTED_TARGETS)
433438
preview_targets.set(PREVIEW_TARGETS)

tools/ci/dynamic_pipelines/report.py

Lines changed: 76 additions & 75 deletions
Large diffs are not rendered by default.

tools/ci/dynamic_pipelines/scripts/child_pipeline_build_apps.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,14 @@
44
import sys
55

66
import __init__ # noqa: F401 # inject the system path
7-
from dynamic_pipelines.constants import TEST_RELATED_APPS_FILENAME
87
from idf_build_apps import build_apps
98
from idf_build_apps import setup_logging
109
from idf_build_apps.utils import semicolon_separated_str_to_list
11-
from idf_ci.app import import_apps_from_txt
10+
from idf_ci_local.app import import_apps_from_txt
1211
from idf_pytest.constants import DEFAULT_IGNORE_WARNING_FILEPATH
1312

13+
from dynamic_pipelines.constants import TEST_RELATED_APPS_FILENAME
14+
1415
if __name__ == '__main__':
1516
parser = argparse.ArgumentParser(description='Build Apps for Dynamic Pipeline')
1617
parser.add_argument('app_list_file', default=TEST_RELATED_APPS_FILENAME, help='List of apps to build')

tools/ci/dynamic_pipelines/scripts/generate_build_child_pipeline.py

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,22 @@
11
# SPDX-FileCopyrightText: 2024 Espressif Systems (Shanghai) CO LTD
22
# SPDX-License-Identifier: Apache-2.0
33
"""This file is used for generating the child pipeline for build jobs."""
4+
45
import argparse
56
import os
67
import typing as t
78

89
import __init__ # noqa: F401 # inject the system path
910
import yaml
11+
from idf_build_apps.utils import semicolon_separated_str_to_list
12+
from idf_ci_local.app import dump_apps_to_txt
13+
from idf_ci_utils import IDF_PATH
14+
from idf_pytest.constants import DEFAULT_CONFIG_RULES_STR
15+
from idf_pytest.constants import DEFAULT_FULL_BUILD_TEST_COMPONENTS
16+
from idf_pytest.constants import DEFAULT_FULL_BUILD_TEST_FILEPATTERNS
17+
from idf_pytest.constants import CollectMode
18+
from idf_pytest.script import get_all_apps
19+
1020
from dynamic_pipelines.constants import DEFAULT_APPS_BUILD_PER_JOB
1121
from dynamic_pipelines.constants import DEFAULT_BUILD_CHILD_PIPELINE_FILEPATH
1222
from dynamic_pipelines.constants import DEFAULT_BUILD_CHILD_PIPELINE_NAME
@@ -18,14 +28,6 @@
1828
from dynamic_pipelines.models import BuildJob
1929
from dynamic_pipelines.models import EmptyJob
2030
from dynamic_pipelines.utils import dump_jobs_to_yaml
21-
from idf_build_apps.utils import semicolon_separated_str_to_list
22-
from idf_ci.app import dump_apps_to_txt
23-
from idf_ci_utils import IDF_PATH
24-
from idf_pytest.constants import CollectMode
25-
from idf_pytest.constants import DEFAULT_CONFIG_RULES_STR
26-
from idf_pytest.constants import DEFAULT_FULL_BUILD_TEST_COMPONENTS
27-
from idf_pytest.constants import DEFAULT_FULL_BUILD_TEST_FILEPATTERNS
28-
from idf_pytest.script import get_all_apps
2931

3032

3133
def _separate_str_to_list(s: str) -> t.List[str]:
@@ -101,7 +103,8 @@ def main(arguments: argparse.Namespace) -> None:
101103
print(f'Generate test related apps file {TEST_RELATED_APPS_FILENAME} with {len(test_related_apps)} apps')
102104
dump_apps_to_txt(sorted(non_test_related_apps), NON_TEST_RELATED_APPS_FILENAME)
103105
print(
104-
f'Generate non-test related apps file {NON_TEST_RELATED_APPS_FILENAME} with {len(non_test_related_apps)} apps'
106+
f'Generate non-test related apps file {NON_TEST_RELATED_APPS_FILENAME} '
107+
f'with {len(non_test_related_apps)} apps'
105108
)
106109

107110
if test_related_apps:
@@ -171,7 +174,7 @@ def main(arguments: argparse.Namespace) -> None:
171174
parser.add_argument(
172175
'--compare-manifest-sha-filepath',
173176
default=os.path.join(IDF_PATH, '.manifest_sha'),
174-
help='Path to the recorded manifest sha file generated by `idf-build-apps dump-manifest-sha`'
177+
help='Path to the recorded manifest sha file generated by `idf-build-apps dump-manifest-sha`',
175178
)
176179
parser.add_argument(
177180
'--modified-components',

tools/ci/dynamic_pipelines/scripts/generate_report.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,15 @@
66
import typing as t
77

88
import __init__ # noqa: F401 # inject the system path
9+
from idf_ci_local.app import enrich_apps_with_metrics_info
10+
from idf_ci_local.app import import_apps_from_txt
11+
912
from dynamic_pipelines.report import BuildReportGenerator
1013
from dynamic_pipelines.report import JobReportGenerator
1114
from dynamic_pipelines.report import TargetTestReportGenerator
1215
from dynamic_pipelines.utils import fetch_app_metrics
1316
from dynamic_pipelines.utils import fetch_failed_jobs
1417
from dynamic_pipelines.utils import parse_testcases_from_filepattern
15-
from idf_ci.app import enrich_apps_with_metrics_info
16-
from idf_ci.app import import_apps_from_txt
1718

1819

1920
def main() -> None:

tools/ci/dynamic_pipelines/scripts/generate_target_test_child_pipeline.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
2. Post the Build Report if it's running in an MR pipeline.
77
3. Generate the child pipeline for target test jobs.
88
"""
9+
910
import argparse
1011
import glob
1112
import os
@@ -15,24 +16,23 @@
1516

1617
import __init__ # noqa: F401 # inject the system path
1718
import yaml
19+
from idf_build_apps import App
20+
from idf_ci_local.app import import_apps_from_txt
21+
from idf_pytest.constants import TIMEOUT_4H_MARKERS
22+
from idf_pytest.script import get_pytest_cases
23+
1824
from dynamic_pipelines.constants import BUILD_ONLY_LABEL
1925
from dynamic_pipelines.constants import DEFAULT_CASES_TEST_PER_JOB
2026
from dynamic_pipelines.constants import DEFAULT_TARGET_TEST_CHILD_PIPELINE_FILEPATH
2127
from dynamic_pipelines.constants import DEFAULT_TARGET_TEST_CHILD_PIPELINE_NAME
2228
from dynamic_pipelines.constants import DEFAULT_TARGET_TEST_JOB_TEMPLATE_NAME
2329
from dynamic_pipelines.constants import DEFAULT_TEST_PATHS
24-
from dynamic_pipelines.constants import (
25-
KNOWN_GENERATE_TEST_CHILD_PIPELINE_WARNINGS_FILEPATH,
26-
)
30+
from dynamic_pipelines.constants import KNOWN_GENERATE_TEST_CHILD_PIPELINE_WARNINGS_FILEPATH
2731
from dynamic_pipelines.constants import TIMEOUT_4H_TEMPLATE_NAME
2832
from dynamic_pipelines.models import EmptyJob
2933
from dynamic_pipelines.models import Job
3034
from dynamic_pipelines.models import TargetTestJob
3135
from dynamic_pipelines.utils import dump_jobs_to_yaml
32-
from idf_build_apps import App
33-
from idf_ci.app import import_apps_from_txt
34-
from idf_pytest.constants import TIMEOUT_4H_MARKERS
35-
from idf_pytest.script import get_pytest_cases
3636

3737

3838
def get_tags_with_amount(s: str) -> t.List[str]:

0 commit comments

Comments
 (0)