- Tc - mailweb.openeuler.org

[Patch] Add-author-signer-reviewer-in-Merge-description
by caihaomin 14 Jul '20

14 Jul '20

Hi Infra members~ As discussed in issue https://gitee.com/openeuler/community/issues/I1NQ50 I am trying to complete this feature. The problem is that I only can write some unit test to confirm the output right as showed below [cid:image001.png@01D65A20.448098A0] I can not set up the ci-bot locally so I can't fully test the feature. Can anyone give me a hand~ and review the patch ^_^ From 238cf5466ce8ecefbe4a2ca369b3dce9ac885f3c Mon Sep 17 00:00:00 2001 From: c00416947 <caihaomin(a)huawei.com> Date: Tue, 14 Jul 2020 20:12:50 +0800 Subject: [PATCH] Add author/signer/reviewer in Merge description Signed-off-by: Haomin <caihaomin(a)huawei.com> --- pkg/cibot/oauth.go | 2 +- pkg/cibot/pullrequest.go | 77 +++++++++++++++++++++++++++++++++++ pkg/cibot/pullrequest_test.go | 35 ++++++++++++++++ 3 files changed, 113 insertions(+), 1 deletion(-) create mode 100644 pkg/cibot/pullrequest_test.go diff --git a/pkg/cibot/oauth.go b/pkg/cibot/oauth.go index 79fac4b..d6e9983 100644 --- a/pkg/cibot/oauth.go +++ b/pkg/cibot/oauth.go @@ -30,7 +30,7 @@ func GetToken(code string) (*oauth2.Token, error) { ctx := context.Background() config := Setup(client, redirect, secret) - glog.Infof("Token request config : %v, code:", config, code) + glog.Infof("Token request config : %v, code: %s", config, code) return config.Exchange(ctx, code) diff --git a/pkg/cibot/pullrequest.go b/pkg/cibot/pullrequest.go index bf3c738..0cea8e2 100644 --- a/pkg/cibot/pullrequest.go +++ b/pkg/cibot/pullrequest.go @@ -408,6 +408,15 @@ func (s *Server) MergePullRequest(event *gitee.NoteEvent) error { // merge pr body := gitee.PullRequestMergePutParam{} body.AccessToken = s.Config.GiteeToken + + // generate merge body + description, err := s.generateMergeDescription(event) + if err != nil { + glog.Errorf("unable to get merge description. err: %v", err) + return err + } + + body.Description = description _, err = s.GiteeClient.PullRequestsApi.PutV5ReposOwnerRepoPullsNumberMerge(s.Context, owner, repo, prNumber, body) if err != nil { glog.Errorf("unable to merge pull request. err: %v", err) @@ -418,3 +427,71 @@ func (s *Server) MergePullRequest(event *gitee.NoteEvent) error { return nil } + +func (s *Server) generateMergeDescription(event *gitee.NoteEvent) (string, error) { + // get basic params + owner := event.Repository.Namespace + repo := event.Repository.Name + prNumber := event.PullRequest.Number + commentCount := event.PullRequest.Comments + var perPage int32 = 20 + pageCount := commentCount / perPage + if commentCount%perPage > 0 { + pageCount++ + } + + result := "" + localVarOptionals := &gitee.GetV5ReposOwnerRepoPullsNumberCommentsOpts{} + localVarOptionals.AccessToken = optional.NewString(s.Config.GiteeToken) + localVarOptionals.PerPage = optional.NewInt32(perPage) + + var signers = make([]string,0) + var reviewers = make([]string,0) + // range page and get comments + for page := pageCount; page > 0; page-- { + localVarOptionals.Page = optional.NewInt32(page) + comments,_, err := s.GiteeClient.PullRequestsApi.GetV5ReposOwnerRepoPullsNumberComments(s.Context,owner,repo,prNumber,localVarOptionals) + if err != nil { + glog.Errorf("unable to get pull request comments. err: %v", err) + return result, err + } + + signers, reviewers, err = getSignersAndReviewers(comments) + if err != nil { + glog.Errorf("failed to get signers or reviewers. err: %v", err) + } + } + + result = formatDescription(event.PullRequest.User.Login, reviewers, signers) + return result, nil +} + +func formatDescription(user string, reviewers,signers []string) string { + return fmt.Sprintf("From: @%s\nReviewed-by: %s\nSigned-off-by: %s\n", + user, strings.Join(reviewers,","), strings.Join(signers,",")) +} + +func getSignersAndReviewers(comments []gitee.PullRequestComments)([]string, []string ,error) { + var signers = make([]string,0) + var reviewers = make([]string,0) + + if len(comments)== 0 { + return signers,reviewers, fmt.Errorf("comment list is empty") + } + + for _, comment := range comments { + m := RegAddLgtm.FindStringSubmatch(comment.Body) + if m != nil && comment.UpdatedAt == comment.CreatedAt { + reviewer := fmt.Sprintf("@%s", comment.User.Login) + reviewers = append(reviewers,reviewer) + } + + m = RegAddApprove.FindStringSubmatch(comment.Body) + if m != nil && comment.UpdatedAt == comment.CreatedAt { + signer := fmt.Sprintf("@%s", comment.User.Login) + signers = append(signers,signer) + } + } + + return signers,reviewers,nil +} diff --git a/pkg/cibot/pullrequest_test.go b/pkg/cibot/pullrequest_test.go new file mode 100644 index 0000000..aa4ea28 --- /dev/null +++ b/pkg/cibot/pullrequest_test.go @@ -0,0 +1,35 @@ +package cibot + +import ( + "testing" +) + +func Test_formatDescription(t *testing.T) { + type args struct { + user string + reviewers []string + signers []string + } + tests := []struct { + name string + args args + want string + }{ + { + name: "test description format", + args: args{ + user: "fakeuser", + reviewers: []string{"@aaa", "@bbb"}, + signers: []string{"@ccc", "@ddd"}, + }, + want: "From: @fakeuser\nReviewed-by: @aaa,@bbb\nSigned-off-by: @ccc,@ddd\n", + }, + } + for _, tt := range tests { + t.Run(tt.name, func(t *testing.T) { + if got := formatDescription(tt.args.user, tt.args.reviewers, tt.args.signers); got != tt.want { + t.Errorf("formatDescription() = %v, want %v", got, tt.want) + } + }) + } +} -- 2.19.1

1 0

[新SIG提案]创建 Wine SIG，为 openEuler 带来 Windows 应用。
by 翁志艺 14 Jul '20

14 Jul '20

技术委员会：大家好！我希望创建一个新的 SIG 关于为社区通过带来 Wine ，从而带来 Windows 上海量的应用，丰富日常使用的场景。 [新SIG提案]创建 Wine SIG，为 openEuler 带来 Windows 应用。 PR链接：https://gitee.com/openeuler/community/pulls/668 谢谢。翁志艺

1 0

[新SIG提案]创建 Wine SIG，为 openEuler 带来 Windows 应用。
by 翁志艺 14 Jul '20

14 Jul '20

技术委员会：大家好！我希望创建一个新的 SIG 关于为社区通过带来 Wine ，从而带来 Windows 上海量的应用，丰富日常使用的场景。 [新SIG提案]创建 Wine SIG，为 openEuler 带来 Windows 应用。 PR链接：https://gitee.com/openeuler/community/pulls/668 谢谢。翁志艺

1 0

对引入新特性的代码归档格式询问
by Hubble_Zhu 14 Jul '20

14 Jul '20

各位openEuler的成员，大家好，     我有一个项目G想申请在openEuler上开源，这个项目G高度依赖第三方开源包W，对W有一些侵入式修改。现在我是以源码的方式将开源包W的代码归档在项目G仓中     请问下如果开源到openEuler上的话，是否和W相关的代码，我都需要以源码包+patch的方式归档，或者我可以沿用全源码归档的方式？     采用全源码归档的方式对开发者很友好。     我计划是当项目G稳定并realease发行版后，在src-openEuler里新建仓库并添加spec来打成rpm包。     Any help will be greatly appreciated. thanks.

4 5

【Meeting Notice】openEuler TC meeting for 2020-07-08 Time: 2020-07-08 10:00-12:30
by Meeting Book 13 Jul '20

13 Jul '20

4 5

增删repository提案
by 安传旭 13 Jul '20

13 Jul '20

Hi TC: 【增删repository提案】 ROS-SIG申请新建12个新仓库，用来存放移植完成打包成功的package。提案PR链接：https://gitee.com/openeuler/c…

1 0

关于UKUI-3.0 在openEuler上的编译适配
by 侯依然爸爸 12 Jul '20

12 Jul '20

社区的各位同仁好: 麒麟软件在openEuler社区成立之初，并已经成立了UKUI SIG组，也一直致力于社区建设，我们已经在进行UKUI-3.0移植到openEuler-20.03的工作，并已作出了一些成果，我们已经在X86平台上，完成了UKUI-3.0的适配和测试，已经能够稳定运行和使用，并且已经将UKUI的相关代码已经贡献到了openEuler社区，大家可以使用下面的教程来安装UKUI-3.0。目前我们在进行的工作如下： 1）将UKUI-3.0 相关编译和安装包移植到 OBS进行编译和构造（aarch64 和 X86），以方便社区的更好的集成和使用。 2) 推进UKUI发展，修复UKUI的相关问题。但在接下来的工作中，我们也希望得到大家的一些帮助，一起促进openEuler UKUI的发展： 1）UKUI-3.0涉及到的安装包和编译依赖包较多，移植到OBS工作量很大，也希望能得到TC和各位同仁的帮助。 2）目前在openEuler上适配的UKUI-3.0，是基础版本，也希望TC和社区各位同仁能提出一些需求、意见和建议，任何对openEuler UKUI话题都可以在https://gitee.com/openkylin/ukui-issues 项目中提出issue。 UKUI是由麒麟团队开发的基于Linux发行版的轻量级桌面环境，默认搭载在优麒麟开源操作系统和银河麒麟/中标麒麟商业发行版中。UKUI 3.0基于Qt全新开发，秉承着“友好易用，简单轻松” 的设计理念，为普通用户架起迈向Linux的桥梁。详细描述可参见：https://www.ukui.org/ UKUI在X86上安装使用的教程如下，欢迎大家使用： openEuler UKUI安装方法：在安装ukui前请先创建一个管理员用户，推荐在安装openEuler系统时创建。 1. 离线 1) 安装openEuler并配置yum源（推荐采用：x86_64: https://repo.openeuler.org/openEuler-20.03-LTS/everything/x86_64/），并安装creat… install createrepo，如果安装不成功请检查配置的yum源） 2) 下载安装必须文件（链接：https://pan.baidu.com/s/1bljArjqrzONfQilW9zUzHg 提取码：pg4q）。将下载的install.sh放入/root目录下并赋予执行权限，RPMS.zip、reinstall.zip文件解压缩。将解压出的RPMS文件夹放入/RPMS文件夹下，目录结构如下所示 reinstall文件夹放入/root目录下。 3) 搭建本地yum源：修改/etc/yum.repos.d/openEuler_x86_64.repo文件，在末尾添加如下内容： [local] name=local baseurl=file:///RPMS/ enabled=1 gpgcheck=0 priority=1 4）切换到/root目录下，执行./install.sh 2. 在线 1）配置openEuler官方源，（推荐采用：x86_64: https://repo.openeuler.org/openEuler-20.03-LTS/everything/x86_64/），可参考如下: [openeuler] name=openeuler baseurl=https://repo.openeuler.org/openEuler-20.03-LTS/everything/x86_64/ enabled=1 gpgcheck=0 2）配置ukui的在线源：修改/etc/yum.repos.d/openEuler_x86_64.repo文件，在末尾添加如下内容： [ukui] name=ukui baseurl= http://openkylin.com/repo/ukui/RPMS/ enabled=1 gpgcheck=0 priority=1 2) 在openKylin（http://openkylin.com/repo/ukui/）下载install_online.sh放入/root目录下, 下载reinstall.zip解压后的reinstall文件夹及内容放入/root目录下。切换到root目录，运行install_online.sh。

1 0

转发：【Meeting Notice】high-performance-network sig例会 Time: 2020-07-16 19:00-20:30
by newlife_lala＠163.com 10 Jul '20

10 Jul '20

1 0

【Meeting Notice】virt sig meeting Time: 2020-07-10 14:15-16:15
by Chenzhendong (alex) 08 Jul '20

08 Jul '20

virt会议议程： 1．Virt sig事务跟踪 a) qemu/libvirt 代码仓库从src-openeuler切换到openeuelr 2．openEuler 20.09特性进展 a) KVM支持custom模式---zhanghailiang b) kvmtop---nocjj c) SmartPolling---miaoyubo d) 支持SDEI设备---待定 e) 双层调度---fanhenglong f) 锁优化---YeZengruan g) 安全启动---待定 h) 可信启动---JackJF i) 热补丁---FangYing -------- 原始邮件 -------- 发件人： Meeting Book <uMeeting(a)huawei.com<mailto:uMeeting@huawei.com>> 日期： 2020年7月8日周三 11:11 收件人： zdchenhn(a)163.com<mailto:zdchenhn@163.com> 主题：【Meeting Notice】virt sig meeting Time: 2020-07-10 14:15-16:15 Topic virt sig meeting Time 2020-07-10 14:15-16:15((UTC+08:00)Beijing) Join Conference Join (External) >><https://welink-meeting.zoom.us/j/883868749> Meeting ID 883 868 749 Convener 陈振东 Tips: Dial the access number to join conference<http://app.huawei.com/eshare/voiceMeetings> 注：仅支持部分国家和地区。( Only some countries and regions are supported.) 如果加入会议提示“会议已结束或会议号不存在”，请在官网下载最新版本入会。立即下载<https://zoom.us/download> If a message is displayed, indicating that the meeting has ended or the meeting ID does not exist, download the latest meeting client and try again.Download<https://zoom.us/download>

1 0

【tc议题申报】将开源包attest-tools和digest-list-tools从GitHub转移到openEuler开发
by nettingsisyphus 07 Jul '20

07 Jul '20

申请将开源包attest-tools和digest-list-tools转移到openEuler开发： attest-tools：https://github.com/initlove/attest-tools digest-list-tools: https://github.com/initlove/digest-list-tools attest-tools提供了远程证明所需的一系列库，能够大大简化IMA远程证明的实现，目前尚未引入openEuler。 digest-list-tools提供了摘要列表生成相关的工具，目前已引入src-openeuler。 Pull Request链接：https://gitee.com/openeuler/community/pulls/649 | | 溯洄 | | 邮箱：benjamin93(a)163.com | 签名由网易邮箱大师定制

2 2