Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI] Adding STM to CI #5220

Closed
Fatman13 opened this issue Aug 26, 2022 · 5 comments
Closed

[CI] Adding STM to CI #5220

Fatman13 opened this issue Aug 26, 2022 · 5 comments
Assignees
@zl03jsj
Labels
C-dev-productivity Category: Developer productivity

Comments

@Fatman13
Copy link
Contributor

Fatman13 commented Aug 26, 2022 •
edited by zl03jsj
Loading

源：需求

Is there a way you could add an extra step to the CI?
The STM dashboard uses the following information for every test (unit and integration):

{
  "name": "TestMulticoreSDR",
  "result": "success",
  "url": "https://app.circleci.com/pipelines/github/filecoin-project/lotus/21569/workflows/0c270977-e0a9-47b1-a33b-0c10843f1483/jobs/490307/tests"
  }
}

To summarize:

  • Test name (required)
  • Test status (success/failure) (required)
  • URL to the CI that ran the test (optional, not required)

For lotus, the STM retrieves this information from the CircleCI's API, but since venus doesn't use CircleCI, we need this information to be uploaded in an artifact somewhere accessible.
Please note that this is not critical for the initial version of the STM dashboard, but the dashboard will not be able to link the behaviors with the test status in the CI.
@Fatman13 Fatman13 added C-dev-productivity Category: Developer productivity Bloxico labels Aug 26, 2022
@Fatman13 Fatman13 added the C-triage Category: Need triage label Aug 29, 2022
@zl03jsj zl03jsj assigned zl03jsj and unassigned hunjixin Sep 8, 2022
@zl03jsj zl03jsj moved this to In Progress in Venus Project Sep 8, 2022
@zl03jsj
Copy link
Contributor

zl03jsj commented Sep 8, 2022

https://filecoinproject.slack.com/archives/C0365S9E0JW/p1662612081589399

发起了一个相关的讨论.等待回复中.

@zl03jsj
Copy link
Contributor

zl03jsj commented Sep 14, 2022 •
edited
Loading

结果:
这个功能可选功能, 并且需要能上传下载附件的网络服务,或者重新使用circleci, 所以暂时不添加这个脚本.
如果有其它想法欢迎留言讨论.

@zl03jsj zl03jsj mentioned this issue Sep 14, 2022
Closed
31 tasks
@Fatman13 Fatman13 mentioned this issue Sep 16, 2022
Closed
54 tasks
@zl03jsj
Copy link
Contributor

zl03jsj commented Sep 16, 2022

备用方案: 参考lotus, 同时使用circleci和git action, circleci只在 需要 测试的和上传测试结果 的工作流.

@Fatman13 Fatman13 mentioned this issue Sep 30, 2022
Closed
45 tasks
@zl03jsj
Copy link
Contributor

zl03jsj commented Oct 6, 2022 •
edited by LinZexiao
Loading

确认使用circleci作为STM测试详细信息链接的方案.需要以下子任务:

@Fatman13 Fatman13 mentioned this issue Oct 21, 2022
Closed
38 tasks
@Fatman13 Fatman13 mentioned this issue Nov 4, 2022
Closed
45 tasks
@zl03jsj
Copy link
Contributor

zl03jsj commented Nov 11, 2022 •
edited
Loading

在这里汇总一下已完成添加注解的组件:

阻塞中的, 以下组件需等待STM团队将组件的注解分类系统加入到STM项目中之后才能继续

This was referenced Nov 17, 2022
Merged
Merged
Merged
Merged
Repository owner moved this from In Progress to Done in Venus Project Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zl03jsj zl03jsj

Labels
C-dev-productivity Category: Developer productivity
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Fatman13 @zl03jsj @hunjixin @LinZexiao