Skip to content

Conversation

@WenLiuyi
Copy link
Contributor

@WenLiuyi WenLiuyi commented Jan 6, 2026

PR Category

CI/CD, Testing

PR Types

Refactoring

PR Description

Successfully passed the verl-FL (verl 0.7.0) compatibility tests, including:

  1. Optimize CI/CD Pipeline: Updated NVIDIA functional tests to replace the verl backend with verl-FL, ensuring compatibility and performance validation.
  2. Synchronized Test Artifacts: Updated golden test results to align with the current code behavior.

@WenLiuyi WenLiuyi requested a review from aoyulong as a code owner January 6, 2026 06:25
@CLAassistant
Copy link

CLAassistant commented Jan 6, 2026

CLA assistant check
All committers have signed the CLA.

commit b35b75c
Merge: 71de5a0 e820fc3
Author: WenLiuyi <1658441344@qq.com>
Date:   Tue Jan 6 14:57:35 2026 +0800

    Merge branch 'main' of https://github.com/WenLiuyi/FlagScale

commit 71de5a0
Author: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>
Date:   Mon Jan 5 19:14:17 2026 +0800

    fix config

    Signed-off-by: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>

    fix results_gold'

    Signed-off-by: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>

    fix installation steps

    Signed-off-by: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>

    [CICD] Temporarily toggle off the rl test (flagos-ai#1034)

    <!-- One of [ Train | Inference | Compress | Serve | RL | Core |
    Hardware | CICD | Tools | Others ] -->
    CICD

    <!-- One of [ User Experience | New Features | Bug Fixes | Improvements
    | Performance | Breaking Change| Deprecations | Test Case | Docs |
    Others ] -->
    Test Case

    <!-- Describe what you’ve done -->
    Temporarily close the RL test and reopen it once a better method for
    resolving its dependencies is established.

    Add README for llama.cpp example (flagos-ai#1030)

    Serve

    Docs

    Add README for llama.cpp.

    Update project README (flagos-ai#1033)

    Others

    Docs

    This PR updates the project README.

    ---------

    Co-authored-by: Yulong Ao <aoyulong@outlook.com>

commit e820fc3
Merge: 952461f ba89472
Author: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>
Date:   Tue Jan 6 14:41:45 2026 +0800

    Merge branch 'main' of https://github.com/WenLiuyi/FlagScale

commit 952461f
Author: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>
Date:   Tue Jan 6 14:41:04 2026 +0800

    fix installation steps

    Signed-off-by: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>

commit ba89472
Merge: 2696639 2731dc5
Author: LYWen <141755971+WenLiuyi@users.noreply.github.com>
Date:   Tue Jan 6 14:24:25 2026 +0800

    Merge branch 'flagos-ai:main' into main

commit 2731dc5
Author: Qiming Teng <tengqm@outlook.com>
Date:   Tue Jan 6 14:22:06 2026 +0800

    Update project README (flagos-ai#1033)

    Others

    Docs

    This PR updates the project README.

    ---------

    Co-authored-by: Yulong Ao <aoyulong@outlook.com>

commit 2696639
Author: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>
Date:   Tue Jan 6 14:21:43 2026 +0800

    fix results_gold'

    Signed-off-by: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>

commit e592c6f
Author: Chenrui.He <hechenrui123@gmail.com>
Date:   Tue Jan 6 11:46:45 2026 +0800

    Add README for llama.cpp example (flagos-ai#1030)
    Serve
    Docs
    Add README for llama.cpp.

commit 54aacb1
Author: Yulong Ao <aoyulong@outlook.com>
Date:   Tue Jan 6 09:52:03 2026 +0800

    [CICD] Temporarily toggle off the rl test (flagos-ai#1034)
    <!-- One of [ Train | Inference | Compress | Serve | RL | Core |
    Hardware | CICD | Tools | Others ] -->
    CICD
    <!-- One of [ User Experience | New Features | Bug Fixes | Improvements
    | Performance | Breaking Change| Deprecations | Test Case | Docs |
    Others ] -->
    Test Case
    <!-- Describe what you’ve done -->
    Temporarily close the RL test and reopen it once a better method for
    resolving its dependencies is established.

commit b23c36f
Author: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>
Date:   Mon Jan 5 19:14:17 2026 +0800

    fix config

    Signed-off-by: Corle-hyz <heyongzhe18@mails.ucas.edu.cn>
WenLiuyi and others added 3 commits January 6, 2026 15:17
Signed-off-by: WenLiuyi <1658441344@qq.com>
Signed-off-by: WenLiuyi <1658441344@qq.com>
Copy link
Collaborator

@heavyrain-lzy heavyrain-lzy left a comment

Choose a reason for hiding this comment

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

LGTM

@aoyulong aoyulong changed the title replace third_party/verl with verl-FL Replace third_party/verl with verl-FL Jan 8, 2026
Copy link
Contributor

@aoyulong aoyulong left a comment

Choose a reason for hiding this comment

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

LGTM

@aoyulong aoyulong merged commit f9bb0e9 into flagos-ai:main Jan 8, 2026
11 of 19 checks passed
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.

4 participants