You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@dolphinscheduler.apache.org by Jave-Chen <ke...@foxmail.com> on 2020/02/09 03:55:34 UTC

[CI][Discussion] discussion on Merge PR failed notification

Hi all,Sometimes merge PR to dev branch will cause GitHub actioin failed,
and now there isn't any notifications.


So is it needed to create a notification in this case? For example send
a mail to dev mail list.

re： [CI][Discussion] discussion on Merge PR failed notification

Posted by Jave-Chen <ke...@foxmail.com>.
Hi, all
Because sending mail in workflow requires hardcode mail account password
which is insecurity, so I filed a INFRA ticket[1] to ask whether INFRA can configure
GitHub notification for ds dev list.&nbsp;
Probably because I didn't make it clear, as a reslut, INFRA said "please have one of&nbsp;
your Mentors open a new Infra ticket with the specific changes DolphinScheduler
&nbsp;would like made".


It seems like we need more discussions. And as an alternative, you can wath
the ds repo at github, and when CI failed your mail box will receive a&nbsp;
notification.


[1]&nbsp;https://issues.apache.org/jira/browse/INFRA-19847


------------------&nbsp;原始邮件&nbsp;------------------
发件人:&nbsp;"lidong dai"<dailidong66@gmail.com&gt;;
发送时间:&nbsp;2020年2月9日(星期天) 下午2:52
收件人:&nbsp;"dev"<dev@dolphinscheduler.apache.org&gt;;

主题:&nbsp;Re: [CI][Discussion] discussion on Merge PR failed notification



hi ,

good idea,&nbsp; mail to dev list!



Best Regards
---------------
DolphinScheduler(Incubator) PPMC
Lidong Dai 代立冬
dailidong66@gmail.com
---------------


Jave-Chen <kejia.chen@foxmail.com&gt; 于2020年2月9日周日 上午11:55写道：

&gt; Hi all,Sometimes merge PR to dev branch will cause GitHub actioin failed,
&gt; and now there isn't any notifications.
&gt;
&gt;
&gt; So is it needed to create a notification in this case? For example send
&gt; a mail to dev mail list.

Re: [CI][Discussion] discussion on Merge PR failed notification

Posted by lidong dai <da...@gmail.com>.
hi ,

good idea,  mail to dev list!



Best Regards
---------------
DolphinScheduler(Incubator) PPMC
Lidong Dai 代立冬
dailidong66@gmail.com
---------------


Jave-Chen <ke...@foxmail.com> 于2020年2月9日周日 上午11:55写道：

> Hi all,Sometimes merge PR to dev branch will cause GitHub actioin failed,
> and now there isn't any notifications.
>
>
> So is it needed to create a notification in this case? For example send
> a mail to dev mail list.