Skip to content

add --- to fold the logs correctly#2730

Closed
tinaxfwu wants to merge 70 commits intoray-project:masterfrom
tinaxfwu:fix/buildkite-folding-format
Closed

add --- to fold the logs correctly#2730
tinaxfwu wants to merge 70 commits intoray-project:masterfrom
tinaxfwu:fix/buildkite-folding-format

Conversation

@tinaxfwu
Copy link
Contributor

@tinaxfwu tinaxfwu commented Jan 11, 2025

Why are these changes needed?

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

tinaxfwu and others added 10 commits January 11, 2025 01:44
@tinaxfwu
add --- to fold the logs correctly
6b233ee 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
fix indent
d287337 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
trim white space
2dfc561 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
print logs at the beginning of each test
b7c8ac5 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
fix white space and print test function names
b8c4388 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
add --- RUN and +++ PASS
ebcc768 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
fix end white space
7325243 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
add parallel and use ### for PASS
082cf9b 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
merge stderr and stdout
f32e81e 
Signed-off-by: Tina Wu <[email protected]>
@CheyuWu
[Refactor] move validateRayClusterStatus out of RayClusterReconciler (r…
b15f5af 
…ay-project#2738)
@kevin85421 kevin85421 marked this pull request as ready for review January 13, 2025 18:01
@kevin85421 kevin85421 marked this pull request as draft January 13, 2025 18:02
rueian and others added 18 commits January 13, 2025 22:35
@rueian
[RayCluster][Fix] DesiredReplicas, MinReplicas and MaxReplicas should…
42f299a 
… respect workerGroupSpec.Suspend (ray-project#2728)
@CheyuWu
[Feature] Validation of RayFTEnabled is false and GcsFaultToleranceOp…
92c2907 
…tion is not nil (ray-project#2726)
@kevin85421
[RayCluster] Validate RayClusterSpec for empty containers and GCS FT (r…
eba1459 
…ay-project#2749)
@kevin85421
[chore] Refactor GetHeadPort (ray-project#2750)
3eaf133
@MortalHappiness
[Fix][kubectl-plugin] Create separate namespaces for each kubectl plu…
f687794 
…gin e2e test (ray-project#2745)
@fscnick
[RayCluster][Feature] add redis password to head pod from GcsFaultTol…
991b9c7 
…eranceOptions (ray-project#2731)
@kevin85421
[RayCluster] Validate GCSFaultToleranceOptions and redis password (ra…
829aad5 
…y-project#2754)

Signed-off-by: kaihsun <[email protected]>
@fscnick
[RayCluster][Refactor] use RayClusterAllPodsAssociationOptions instead (
ae880c4 
ray-project#2756)

Signed-off-by: fscnick <[email protected]>
@andrewsykim
[kubectl-plugin] add --worker-gpu flag for cluster creation (ray-proj…
0cdce72 
…ect#2675)

Signed-off-by: Andrew Sy Kim <[email protected]>
@andrewsykim
[kubectl-plugin] add create workergroup command (ray-project#2673)
bee67f2 
Signed-off-by: Andrew Sy Kim <[email protected]>
@win5923
[Refactor] Move functions that don’t rely on the controller to non-co…
28ab5c9 
…ntroller member functions (ray-project#2747)

Signed-off-by: win5923 <[email protected]>
@andrewsykim
[RayJob] implement deletion policy API (ray-project#2643)
0544f8b 
Signed-off-by: Andrew Sy Kim <[email protected]>
@kevin85421
[GCS FT] Unify configuring Gcs FT into a single function (ray-project…
937297c 
…#2755)
@kevin85421
[RayJob] Follow up of RayJob deletion policy PR (ray-project#2763)
370fc44
@rueian
[RayCluster][Feature] skip suspending worker groups if the in-tree au…
d86ea62 
…toscaler is enabled (ray-project#2748)
@rueian
skip suspending worker groups if the RayJobDeletionPolicy feature fla…
405fe67 
…g is not enabled (ray-project#2770)
@owenowenisme
Move matching labels to association.go (ray-project#2734)
5fde3c6
@kevin85421
[GCS FT] Add e2e tests for configuring GCS FT with annotations (ray-p…
4bbaa06 
…roject#2766)
andrewsykim and others added 29 commits January 21, 2025 13:21
@andrewsykim
[kubectl-plugin] silence warnings when creating worker groups (ray-pr…
522718e 
…oject#2792)

Signed-off-by: Andrew Sy Kim <[email protected]>
@MortalHappiness
[CI] Make release.yaml only be triggered manually (ray-project#2798)
fa67724 
Signed-off-by: Chi-Sheng Liu <[email protected]>
@rueian
[RayCluster][Feature] reject redis username to head pod out side of G…
7bb82db 
…csFaultToleranceOptions (ray-project#2796)

Signed-off-by: Rueian <[email protected]>
@MortalHappiness
[CI] Skip kubectl plugin flaky e2e tests (ray-project#2800)
4bb1226 
Signed-off-by: Chi-Sheng Liu <[email protected]>
@ryanaoleary
Bump golang.org/x/net to v0.33.0 fix upstream vulnerability (ray-proj…
aafe2e0 
…ect#2799)
@Abirdcfly
[Chore] update comment for headGroupSpec and entrypoint (ray-project#…
20ed56f 
…2802)

Signed-off-by: Abirdcfly <[email protected]>
@CheyuWu
[Refactor] Move ValidateRayClusterSpec to validation.go and its `…
8c53bd5 
…unit test` to `validation_test.go` (ray-project#2790)
@dependabot
Bump golang.org/x/net from 0.26.0 to 0.33.0 in /proto (ray-project#2723)
7e72627 
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.26.0 to 0.33.0.
- [Commits](golang/net@v0.26.0...v0.33.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@fscnick
[Chore] remove redundant var declaration (ray-project#2811)
dd46cb4 
Signed-off-by: fscnick <[email protected]>
@CheyuWu
[Refactor] Move ValidateRayJobStatus to validation.go and create its …
298539d 
…unit test (ray-project#2813)
@ryanaoleary
[autoscaler] Bump Ray e2e test image (ray-project#2814)
5d032a6 
Signed-off-by: Ryan O'Leary <[email protected]>
@fscnick
[Chore] make ingressClassName as a local variable (ray-project#2815)
d2ae625 
Signed-off-by: fscnick <[email protected]>
@CheyuWu
[Refactor] Move function ValidateRayJobSpec to validation.go and its …
3d1c6c3 
…unit test (ray-project#2812)
@rueian
[RayService] Unify the cluster switch over logic together (ray-projec…
46355ed 
…t#2805)
@tinaxfwu
add --- to fold the logs correctly
7aebbcb 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
fix indent
448bb99 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
trim white space
130b3e4 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
print logs at the beginning of each test
237ac37 
Signed-off-by: Tina <[email protected]>
@tinaxfwu
fix white space and print test function names
58a0793 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
add --- RUN and +++ PASS
36cdc08 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
fix end white space
4c600ab 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
add parallel and use ### for PASS
6008d01 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
merge stderr and stdout
e6f2965 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
try awk and rv fmt
48309e5 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
try awk
99c4ca6 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
test --- RUN
b3f77b7 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
add awk and remove println
5312735 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
fix path
a15473a 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu
change grouping message
0c5019a 
Signed-off-by: Tina Wu <[email protected]>
@tinaxfwu tinaxfwu closed this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kevin85421 kevin85421 Awaiting requested review from kevin85421 kevin85421 will be requested when the pull request is marked ready for review kevin85421 is a code owner

@andrewsykim andrewsykim Awaiting requested review from andrewsykim andrewsykim will be requested when the pull request is marked ready for review andrewsykim is a code owner

@rueian rueian Awaiting requested review from rueian rueian will be requested when the pull request is marked ready for review rueian is a code owner

@MortalHappiness MortalHappiness Awaiting requested review from MortalHappiness MortalHappiness will be requested when the pull request is marked ready for review MortalHappiness is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.