Skip to content

Commit a83c670

Browse files
authored
[ci] fix windows test launch and building (#57684)
make that `test_in_docker` does not depend on the entire `ray_release` library, but only depends on python files that are required for the test db to work. this removes the dependency of `cryptography` library from `ray_ci`, so that windows wheels can be built and windows tests can run again. Signed-off-by: Lonnie Liu <[email protected]>
1 parent 5a5ad71 commit a83c670

File tree

2 files changed

+86
-2
lines changed

2 files changed

+86
-2
lines changed

ci/ray_ci/BUILD.bazel

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ py_library(
1717
deps = [
1818
"//release:bazel",
1919
"//release:global_config",
20-
"//release:ray_release",
20+
"//release:test",
21+
"//release:test_automation",
2122
ci_require("boto3"),
2223
ci_require("pyyaml"),
2324
ci_require("click"),

release/BUILD.bazel

Lines changed: 84 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -386,6 +386,79 @@ py_library(
386386
],
387387
)
388388

389+
py_library(
390+
name = "util",
391+
srcs = ["ray_release/util.py"],
392+
imports = ["."],
393+
visibility = ["//visibility:private"],
394+
deps = [
395+
":global_config",
396+
":logger",
397+
bk_require("anyscale"),
398+
],
399+
)
400+
401+
py_library(
402+
name = "anyscale_util",
403+
srcs = ["ray_release/anyscale_util.py"],
404+
imports = ["."],
405+
visibility = ["//visibility:private"],
406+
deps = [
407+
":exception",
408+
":logger",
409+
":util",
410+
bk_require("anyscale"),
411+
],
412+
)
413+
414+
py_library(
415+
name = "aws",
416+
srcs = ["ray_release/aws.py"],
417+
imports = ["."],
418+
visibility = ["//visibility:private"],
419+
deps = [
420+
":logger",
421+
":util",
422+
bk_require("boto3"),
423+
bk_require("botocore"),
424+
bk_require("aws-requests-auth"),
425+
],
426+
)
427+
428+
py_library(
429+
name = "test",
430+
srcs = ["ray_release/test.py"],
431+
imports = ["."],
432+
visibility = ["//ci/ray_ci:__pkg__"],
433+
deps = [
434+
":aws",
435+
":global_config",
436+
":logger",
437+
":result",
438+
":util",
439+
bk_require("aioboto3"),
440+
bk_require("boto3"),
441+
bk_require("botocore"),
442+
bk_require("pygithub"),
443+
],
444+
)
445+
446+
py_library(
447+
name = "test_automation",
448+
srcs = [
449+
"ray_release/test_automation/ci_state_machine.py",
450+
"ray_release/test_automation/release_state_machine.py",
451+
"ray_release/test_automation/state_machine.py",
452+
],
453+
imports = ["."],
454+
visibility = ["//ci/ray_ci:__pkg__"],
455+
deps = [
456+
":test",
457+
bk_require("pygithub"),
458+
bk_require("pybuildkite"),
459+
],
460+
)
461+
389462
py_library(
390463
name = "ray_release",
391464
srcs = glob(
@@ -394,14 +467,19 @@ py_library(
394467
"ray_release/tests/*.py",
395468
"ray_release/configs/*.py",
396469
"ray_release/scripts/*.py",
470+
"ray_release/test_automation/*.py",
397471
"ray_release/bazel.py",
398472
"ray_release/logger.py",
399473
"ray_release/result.py",
400474
"ray_release/exception.py",
401475
"ray_release/kuberay_util.py",
402476
"ray_release/wheels.py",
477+
"ray_release/util.py",
403478
"ray_release/retry.py",
404479
"ray_release/cloud_util.py",
480+
"ray_release/anyscale_util.py",
481+
"ray_release/aws.py",
482+
"ray_release/test.py",
405483
],
406484
),
407485
data = glob([
@@ -418,6 +496,8 @@ py_library(
418496
imports = ["."],
419497
visibility = ["//visibility:public"],
420498
deps = [
499+
":anyscale_util",
500+
":aws",
421501
":bazel",
422502
":cloud_util",
423503
":exception",
@@ -426,6 +506,9 @@ py_library(
426506
":logger",
427507
":result",
428508
":retry",
509+
":test",
510+
":test_automation",
511+
":util",
429512
":wheels",
430513
bk_require("anyscale"),
431514
bk_require("aws-requests-auth"),
@@ -803,7 +886,7 @@ py_test(
803886
],
804887
deps = [
805888
":bazel",
806-
":ray_release",
889+
":test",
807890
bk_require("pytest"),
808891
],
809892
)

0 commit comments

Comments
 (0)