Skip to content

[需求] 支持细粒度的NPU使用率采集-的开发实现-runner-container-hooks部分 - #58

Open
fly333sky wants to merge 2 commits into
mainfrom
issue-1968-from-main
Open

fly333sky wants to merge 2 commits into
mainfrom
issue-1968-from-main

Conversation

@fly333sky

Copy link
Copy Markdown

背景

[需求] 支持细粒度的NPU使用率采集(runner-container-hooks)· 开发流水线 · 开发预览阶段(代码已推 + 预览已部署 + UT 已补;门禁/对抗由 PR CI 异步跑)

改动内容

fix(ascend-ci-deployment): fix yamllint failures across kustomization.yaml files

Fixed all yamllint errors in the ascend-ci-deployment submodule that were
blocking the deterministic gate. The provided failure list was truncated;
a full yamllint run revealed 30 files with errors (29 kustomization.yaml

  • 1 prometheus-rules.yaml).

Changes:

  1. Indentation fix (29 kustomization.yaml files): Added 2-space
    indentation to all list items (lines starting with - at column 0)
    under resources: and patches: keys, per .yamllint config
    (indentation: indent-sequences: true). Commented-out list items
    (starting with #) were left untouched.

  2. Empty-lines fix: Removed trailing blank lines at end of file in files
    that had max-end: 0 violations. Ensured exactly one trailing newline
    per file.

  3. new-line-at-end-of-file fix:
    monitoring/config-for-ascend-triton-agent-ci/kustomization.yaml was
    missing a trailing newline — added one.

  4. line-length fix:
    monitoring/config-for-infra-cn4-x86-common-cluster/prometheus-rules.yaml
    line 1233 was 239 chars (>200 limit). Split the long description:
    string across 3 lines using YAML \ continuation, matching the existing
    pattern used elsewhere in the same file (e.g. lines 857-859, 943-945).

  5. Test fix: Updated tests/test_vllm_omni_buildkite.py
    TestConfigGy003.test_kustomization_resources and
    TestConfigSh001.test_kustomization_resources were expecting 3
    resources but the kustomization.yaml files now include
    npu-metrics-pushgw-secret.yaml (added in issue-1968 round 1/2).
    Added the missing resource to the expected sorted list in both tests.

Verification:

  • yamllint -c .yamllint on all 205 changed YAML files: 0 errors
  • python3 -m pytest tests/ (excluding pre-existing broken
    test_sync_new_projects_to_docs.py which references a non-existent script):
    175 passed
  • python3 scripts/check-projects-coverage.py: passes (no new dirs to check)

No UT files added — this round is pure YAML formatting + test expectation
fix; no business logic changes.

相关 Issue

resolve https://github.com/opensourceways/backlog/issues/1968

AI 使用声明

当前 PR 是否有 AI 参与:

@opensourceways-bot

Copy link
Copy Markdown

Welcome To opensourceways Community

Hey @fly333sky , thanks for your contribution to the community.

Bot Usage Manual

I'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands.

Contact Guide

If you have any questions, please contact the SIG: infratructure ,
and any of the maintainers: @GeorgeCao-hw, @TangJia025, @pkking, @zhongjun2 ,
and any of the committers: @GeorgeCao-hw, @Goalina, @Hourunze1997, @JavaPythonAIForBAT, @KadenZhang3321, @LiYanghang00, @ccijunk, @drizzlezyk, @pkking, @rosecoffe, @tfhddd, @yao-xiaobai, @zhongjun2, @zkhzkhz .

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

fly333sky, thanks for your pull request. All authors of the commits have signed the CLA. 👍

i;

for (i = 0; i < rows.length; i += 1) {
rows[i].data = loadRowData(rows[i]);
i;

for (i = 0; i < rows.length; i += 1) {
rows[i].data = loadRowData(rows[i]);
@opensourceways-bot

Copy link
Copy Markdown
检查项 状态
敏感信息扫描
安全编码扫描
漏洞扫描
开源license合规扫描
UT覆盖率
开发阶段设计文档检查
流水线链接 点击跳转查看日志

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

fly333sky, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedhooklib@​0.1.0N/AN/AN/AN/AN/A

View full report

@opensourceways-bot

Copy link
Copy Markdown
检查项 状态
敏感信息扫描
安全编码扫描
漏洞扫描
开源license合规扫描
UT覆盖率
开发阶段设计文档检查
流水线链接 点击跳转查看日志

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants