Re: [NOTICE] CiBot improvements

2021-10-29 Thread JING ZHANG
Thanks a lot @Chesnay, I would try again.

Chesnay Schepler  于2021年10月29日周五 下午4:21写道:

> When you 'run azure', check the build after a minute or so and you
> should see that the failed job is running again.
>
>
>
> On 29/10/2021 10:19, Chesnay Schepler wrote:
> > It's working just fine. You need to rebase your branch because on your
> > branch flink-python is simply broken.
> >
> > On 29/10/2021 08:57, JING ZHANG wrote:
> >> Hi Chesnay,
> >> I meet with something weird.
> >> I submit a pull request https://github.com/apache/flink/pull/17571,
> >> it trigger a ci
> >> (
> https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=25470&view=results
> >> <
> https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=25470&view=results>)
>
> >> which fails at first time because of python module failure.
> >> I try to run 'run azure' for two times, nothing happened which is
> >> different from the behavior in the past.
> >> I ask some other guys, they said they also meet with this problem.
> >> Please help me check whether it is expected, thanks a lot.
> >> image.png
> >>
> >> Best,
> >> JING ZHANG
> >>
> >> Yuepeng Pan  于2021年10月12日周二 下午8:08写道:
> >>
> >> Thank you for your effort!
> >> Best,
> >> Roc
> >>
> >> At 2021-10-12 14:07:18, "Arvid Heise"  wrote:
> >> >Awesome!
> >> >
> >> >On Tue, Oct 12, 2021 at 3:11 AM Guowei Ma 
> >> wrote:
> >> >
> >> >> Thanks for your effort!
> >> >>
> >> >> Best,
> >> >> Guowei
> >> >>
> >> >>
> >> >> On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen 
> >> wrote:
> >> >>
> >> >> > Great initiative, thanks for doing this!
> >> >> >
> >> >> > On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann
> >> 
> >> >> > wrote:
> >> >> >
> >> >> > > Thanks a lot for this effort Chesnay! The improvements
> >> sound really
> >> >> good.
> >> >> > >
> >> >> > > Cheers,
> >> >> > > Till
> >> >> > >
> >> >> > > On Mon, Oct 11, 2021 at 8:46 AM David Morávek
> >>  wrote:
> >> >> > >
> >> >> > > > Nice! Thanks for the effort Chesnay, this is really a
> >> huge step
> >> >> > forward!
> >> >> > > >
> >> >> > > > Best,
> >> >> > > > D.
> >> >> > > >
> >> >> > > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song
> >> 
> >> >> > > > wrote:
> >> >> > > >
> >> >> > > > > Thanks for the effort, @Chesnay. This is super helpful.
> >> >> > > > >
> >> >> > > > > @Jing,
> >> >> > > > > Every push to the PR branch should automatically
> >> trigger an entire
> >> >> > new
> >> >> > > > > build. `@flinkbot run azure` should only be used when
> >> you want to
> >> >> > > re-run
> >> >> > > > > the failed stages without changing the PR. E.g., when
> >> running into
> >> >> > > known
> >> >> > > > > unstable cases that are unrelated to the PR.
> >> >> > > > >
> >> >> > > > > Thank you~
> >> >> > > > >
> >> >> > > > > Xintong Song
> >> >> > > > >
> >> >> > > > >
> >> >> > > > >
> >> >> > > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG
> >> 
> >> >> > > > wrote:
> >> >> > > > >
> >> >> > > > > > Hi Chesnay,
> >> >> > > > > > Thanks for the effort. It is a very useful improvement.
> >> >> > > > > > I have a minor question. Please forgive me if the
> >> question is too
> >> >> > > > naive.
> >> >> > > > > > Since '@flinkbot run azure' now behaves like "Rerun
> >> failed jobs",
> >> >> > is
> >> >> > > > > there
> >> >> > > > > > any way to trigger an entirely new build? Because
> >> some times I'm
> >> >> > not
> >> >> > > > sure
> >> >> > > > > > the passed cases in the last build could still
> >> success in the new
> >> >> > > build
> >> >> > > > > > because of introduced updates in new commit.
> >> >> > > > > >
> >> >> > > > > > Best,
> >> >> > > > > > JING ZHANG
> >> >> > > > > >
> >> >> > > > > >
> >> >> > > > > > Yangze Guo  于2021年10月11日周一
> >> 上午10:31写道:
> >> >> > > > > >
> >> >> > > > > > > Thanks for that great job, Chesnay! "Rerun failed
> >> jobs" will
> >> >> > help a
> >> >> > > > > lot.
> >> >> > > > > > >
> >> >> > > > > > > Best,
> >> >> > > > > > > Yangze Guo
> >> >> > > > > > >
> >> >> > > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
> >> >> > > ches...@apache.org
> >> >> > > > >
> >> >> > > > > > > wrote:
> >> >> > > > > > > >
> >> >> > > > > > > > I made a number of changes to the CiBot over the
> >> weekend.
> >> >> > > > > > > >
> >> >> > > > > > > > - '@flinkbot run azure' previously triggered an
> >> entirely new
> >> >> > > build
> >> >> > > > > > based
> >> >> > > > > > > > on the last completed one. It now instead retries
> >> the last
> >> >> > > > completed
> >> >> > > > > > > > build, only running the

Re: [NOTICE] CiBot improvements

2021-10-29 Thread Chesnay Schepler
When you 'run azure', check the build after a minute or so and you 
should see that the failed job is running again.




On 29/10/2021 10:19, Chesnay Schepler wrote:
It's working just fine. You need to rebase your branch because on your 
branch flink-python is simply broken.


On 29/10/2021 08:57, JING ZHANG wrote:

Hi Chesnay,
I meet with something weird.
I submit a pull request https://github.com/apache/flink/pull/17571, 
it trigger a ci 
(https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=25470&view=results 
) 
which fails at first time because of python module failure.
I try to run 'run azure' for two times, nothing happened which is 
different from the behavior in the past.

I ask some other guys, they said they also meet with this problem.
Please help me check whether it is expected, thanks a lot.
image.png

Best,
JING ZHANG

Yuepeng Pan  于2021年10月12日周二 下午8:08写道:

    Thank you for your effort!
    Best,
    Roc

    At 2021-10-12 14:07:18, "Arvid Heise"  wrote:
    >Awesome!
    >
    >On Tue, Oct 12, 2021 at 3:11 AM Guowei Ma 
    wrote:
    >
    >> Thanks for your effort!
    >>
    >> Best,
    >> Guowei
    >>
    >>
    >> On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen 
    wrote:
    >>
    >> > Great initiative, thanks for doing this!
    >> >
    >> > On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann
    
    >> > wrote:
    >> >
    >> > > Thanks a lot for this effort Chesnay! The improvements
    sound really
    >> good.
    >> > >
    >> > > Cheers,
    >> > > Till
    >> > >
    >> > > On Mon, Oct 11, 2021 at 8:46 AM David Morávek
     wrote:
    >> > >
    >> > > > Nice! Thanks for the effort Chesnay, this is really a
    huge step
    >> > forward!
    >> > > >
    >> > > > Best,
    >> > > > D.
    >> > > >
    >> > > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song
    
    >> > > > wrote:
    >> > > >
    >> > > > > Thanks for the effort, @Chesnay. This is super helpful.
    >> > > > >
    >> > > > > @Jing,
    >> > > > > Every push to the PR branch should automatically
    trigger an entire
    >> > new
    >> > > > > build. `@flinkbot run azure` should only be used when
    you want to
    >> > > re-run
    >> > > > > the failed stages without changing the PR. E.g., when
    running into
    >> > > known
    >> > > > > unstable cases that are unrelated to the PR.
    >> > > > >
    >> > > > > Thank you~
    >> > > > >
    >> > > > > Xintong Song
    >> > > > >
    >> > > > >
    >> > > > >
    >> > > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG
    
    >> > > > wrote:
    >> > > > >
    >> > > > > > Hi Chesnay,
    >> > > > > > Thanks for the effort. It is a very useful improvement.
    >> > > > > > I have a minor question. Please forgive me if the
    question is too
    >> > > > naive.
    >> > > > > > Since '@flinkbot run azure' now behaves like "Rerun
    failed jobs",
    >> > is
    >> > > > > there
    >> > > > > > any way to trigger an entirely new build? Because
    some times I'm
    >> > not
    >> > > > sure
    >> > > > > > the passed cases in the last build could still
    success in the new
    >> > > build
    >> > > > > > because of introduced updates in new commit.
    >> > > > > >
    >> > > > > > Best,
    >> > > > > > JING ZHANG
    >> > > > > >
    >> > > > > >
    >> > > > > > Yangze Guo  于2021年10月11日周一
    上午10:31写道:
    >> > > > > >
    >> > > > > > > Thanks for that great job, Chesnay! "Rerun failed
    jobs" will
    >> > help a
    >> > > > > lot.
    >> > > > > > >
    >> > > > > > > Best,
    >> > > > > > > Yangze Guo
    >> > > > > > >
    >> > > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
    >> > > ches...@apache.org
    >> > > > >
    >> > > > > > > wrote:
    >> > > > > > > >
    >> > > > > > > > I made a number of changes to the CiBot over the
    weekend.
    >> > > > > > > >
    >> > > > > > > > - '@flinkbot run azure' previously triggered an
    entirely new
    >> > > build
    >> > > > > > based
    >> > > > > > > > on the last completed one. It now instead retries
    the last
    >> > > > completed
    >> > > > > > > > build, only running the jobs that actually 
failed. It

    >> basically
    >> > > > > behaves
    >> > > > > > > > like the "Rerun failed jobs" button in the Azure 
UI.

    >> > > > > > > > - various optimizations to increase
    responsiveness (primarily
    >> > by
    >> > > > > doing
    >> > > > > > > > significantly less unnecessary work / requests 
to GH)

    >> > > > > > > > - removed TravisCI support (since we no longer
    support a
    >> > release
    >> > > > that
    >> > > > > > > > used Travis)
    >> > > > > > > >
    >> > > > > > > > Please ping me if you spot anything weird.
    >> > > > > > >
    >> > > > > >
    >> > > > >
    >> > > >
    >> > >
    >> >
    >>







Re: [NOTICE] CiBot improvements

2021-10-29 Thread Chesnay Schepler
It's working just fine. You need to rebase your branch because on your 
branch flink-python is simply broken.


On 29/10/2021 08:57, JING ZHANG wrote:

Hi Chesnay,
I meet with something weird.
I submit a pull request https://github.com/apache/flink/pull/17571, it 
trigger a ci 
(https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=25470&view=results 
) 
which fails at first time because of python module failure.
I try to run 'run azure' for two times, nothing happened which is 
different from the behavior in the past.

I ask some other guys, they said they also meet with this problem.
Please help me check whether it is expected, thanks a lot.
image.png

Best,
JING ZHANG

Yuepeng Pan  于2021年10月12日周二 下午8:08写道:

Thank you for your effort!
Best,
Roc

At 2021-10-12 14:07:18, "Arvid Heise"  wrote:
>Awesome!
>
>On Tue, Oct 12, 2021 at 3:11 AM Guowei Ma 
wrote:
>
>> Thanks for your effort!
>>
>> Best,
>> Guowei
>>
>>
>> On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen 
wrote:
>>
>> > Great initiative, thanks for doing this!
>> >
>> > On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann

>> > wrote:
>> >
>> > > Thanks a lot for this effort Chesnay! The improvements
sound really
>> good.
>> > >
>> > > Cheers,
>> > > Till
>> > >
>> > > On Mon, Oct 11, 2021 at 8:46 AM David Morávek
 wrote:
>> > >
>> > > > Nice! Thanks for the effort Chesnay, this is really a
huge step
>> > forward!
>> > > >
>> > > > Best,
>> > > > D.
>> > > >
>> > > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song

>> > > > wrote:
>> > > >
>> > > > > Thanks for the effort, @Chesnay. This is super helpful.
>> > > > >
>> > > > > @Jing,
>> > > > > Every push to the PR branch should automatically
trigger an entire
>> > new
>> > > > > build. `@flinkbot run azure` should only be used when
you want to
>> > > re-run
>> > > > > the failed stages without changing the PR. E.g., when
running into
>> > > known
>> > > > > unstable cases that are unrelated to the PR.
>> > > > >
>> > > > > Thank you~
>> > > > >
>> > > > > Xintong Song
>> > > > >
>> > > > >
>> > > > >
>> > > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG

>> > > > wrote:
>> > > > >
>> > > > > > Hi Chesnay,
>> > > > > > Thanks for the effort. It is a very useful improvement.
>> > > > > > I have a minor question. Please forgive me if the
question is too
>> > > > naive.
>> > > > > > Since '@flinkbot run azure' now behaves like "Rerun
failed jobs",
>> > is
>> > > > > there
>> > > > > > any way to trigger an entirely new build? Because
some times I'm
>> > not
>> > > > sure
>> > > > > > the passed cases in the last build could still
success in the new
>> > > build
>> > > > > > because of introduced updates in new commit.
>> > > > > >
>> > > > > > Best,
>> > > > > > JING ZHANG
>> > > > > >
>> > > > > >
>> > > > > > Yangze Guo  于2021年10月11日周一
上午10:31写道:
>> > > > > >
>> > > > > > > Thanks for that great job, Chesnay! "Rerun failed
jobs" will
>> > help a
>> > > > > lot.
>> > > > > > >
>> > > > > > > Best,
>> > > > > > > Yangze Guo
>> > > > > > >
>> > > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
>> > > ches...@apache.org
>> > > > >
>> > > > > > > wrote:
>> > > > > > > >
>> > > > > > > > I made a number of changes to the CiBot over the
weekend.
>> > > > > > > >
>> > > > > > > > - '@flinkbot run azure' previously triggered an
entirely new
>> > > build
>> > > > > > based
>> > > > > > > > on the last completed one. It now instead retries
the last
>> > > > completed
>> > > > > > > > build, only running the jobs that actually failed. It
>> basically
>> > > > > behaves
>> > > > > > > > like the "Rerun failed jobs" button in the Azure UI.
>> > > > > > > > - various optimizations to increase
responsiveness (primarily
>> > by
>> > > > > doing
>> > > > > > > > significantly less unnecessary work / requests to GH)
>> > > > > > > > - removed TravisCI support (since we no longer
support a
>> > release
>> > > > that
>> > > > > > > > used Travis)
>> > > > > > > >
>> > > > > > > > Please ping me if you spot anything weird.
>> > > > > > >
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>>



Re: Re: [NOTICE] CiBot improvements

2021-10-28 Thread JING ZHANG
Hi Chesnay,
I meet with something weird.
I submit a pull request https://github.com/apache/flink/pull/17571, it
trigger a ci (
https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=25470&view=results)
which fails at first time because of python module failure.
I try to run 'run azure' for two times, nothing happened which is different
from the behavior in the past.
I ask some other guys, they said they also meet with this problem.
Please help me check whether it is expected, thanks a lot.
[image: image.png]

Best,
JING ZHANG

Yuepeng Pan  于2021年10月12日周二 下午8:08写道:

> Thank you for your effort!
> Best,
> Roc
>
> At 2021-10-12 14:07:18, "Arvid Heise"  wrote:
> >Awesome!
> >
> >On Tue, Oct 12, 2021 at 3:11 AM Guowei Ma  wrote:
> >
> >> Thanks for your effort!
> >>
> >> Best,
> >> Guowei
> >>
> >>
> >> On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen  wrote:
> >>
> >> > Great initiative, thanks for doing this!
> >> >
> >> > On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann 
> >> > wrote:
> >> >
> >> > > Thanks a lot for this effort Chesnay! The improvements sound really
> >> good.
> >> > >
> >> > > Cheers,
> >> > > Till
> >> > >
> >> > > On Mon, Oct 11, 2021 at 8:46 AM David Morávek 
> wrote:
> >> > >
> >> > > > Nice! Thanks for the effort Chesnay, this is really a huge step
> >> > forward!
> >> > > >
> >> > > > Best,
> >> > > > D.
> >> > > >
> >> > > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song <
> tonysong...@gmail.com>
> >> > > > wrote:
> >> > > >
> >> > > > > Thanks for the effort, @Chesnay. This is super helpful.
> >> > > > >
> >> > > > > @Jing,
> >> > > > > Every push to the PR branch should automatically trigger an
> entire
> >> > new
> >> > > > > build. `@flinkbot run azure` should only be used when you want
> to
> >> > > re-run
> >> > > > > the failed stages without changing the PR. E.g., when running
> into
> >> > > known
> >> > > > > unstable cases that are unrelated to the PR.
> >> > > > >
> >> > > > > Thank you~
> >> > > > >
> >> > > > > Xintong Song
> >> > > > >
> >> > > > >
> >> > > > >
> >> > > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG <
> beyond1...@gmail.com>
> >> > > > wrote:
> >> > > > >
> >> > > > > > Hi Chesnay,
> >> > > > > > Thanks for the effort. It is a very useful improvement.
> >> > > > > > I have a minor question. Please forgive me if the question is
> too
> >> > > > naive.
> >> > > > > > Since '@flinkbot run azure' now behaves like "Rerun failed
> jobs",
> >> > is
> >> > > > > there
> >> > > > > > any way to trigger an entirely new build? Because some times
> I'm
> >> > not
> >> > > > sure
> >> > > > > > the passed cases in the last build could still success in the
> new
> >> > > build
> >> > > > > > because of introduced updates in new commit.
> >> > > > > >
> >> > > > > > Best,
> >> > > > > > JING ZHANG
> >> > > > > >
> >> > > > > >
> >> > > > > > Yangze Guo  于2021年10月11日周一 上午10:31写道:
> >> > > > > >
> >> > > > > > > Thanks for that great job, Chesnay! "Rerun failed jobs" will
> >> > help a
> >> > > > > lot.
> >> > > > > > >
> >> > > > > > > Best,
> >> > > > > > > Yangze Guo
> >> > > > > > >
> >> > > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
> >> > > ches...@apache.org
> >> > > > >
> >> > > > > > > wrote:
> >> > > > > > > >
> >> > > > > > > > I made a number of changes to the CiBot over the weekend.
> >> > > > > > > >
> >> > > > > > > > - '@flinkbot run azure' previously triggered an entirely
> new
> >> > > build
> >> > > > > > based
> >> > > > > > > > on the last completed one. It now instead retries the last
> >> > > > completed
> >> > > > > > > > build, only running the jobs that actually failed. It
> >> basically
> >> > > > > behaves
> >> > > > > > > > like the "Rerun failed jobs" button in the Azure UI.
> >> > > > > > > > - various optimizations to increase responsiveness
> (primarily
> >> > by
> >> > > > > doing
> >> > > > > > > > significantly less unnecessary work / requests to GH)
> >> > > > > > > > - removed TravisCI support (since we no longer support a
> >> > release
> >> > > > that
> >> > > > > > > > used Travis)
> >> > > > > > > >
> >> > > > > > > > Please ping me if you spot anything weird.
> >> > > > > > >
> >> > > > > >
> >> > > > >
> >> > > >
> >> > >
> >> >
> >>
>


Re:Re: [NOTICE] CiBot improvements

2021-10-12 Thread Yuepeng Pan
Thank you for your effort!
Best,
Roc

At 2021-10-12 14:07:18, "Arvid Heise"  wrote:
>Awesome!
>
>On Tue, Oct 12, 2021 at 3:11 AM Guowei Ma  wrote:
>
>> Thanks for your effort!
>>
>> Best,
>> Guowei
>>
>>
>> On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen  wrote:
>>
>> > Great initiative, thanks for doing this!
>> >
>> > On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann 
>> > wrote:
>> >
>> > > Thanks a lot for this effort Chesnay! The improvements sound really
>> good.
>> > >
>> > > Cheers,
>> > > Till
>> > >
>> > > On Mon, Oct 11, 2021 at 8:46 AM David Morávek  wrote:
>> > >
>> > > > Nice! Thanks for the effort Chesnay, this is really a huge step
>> > forward!
>> > > >
>> > > > Best,
>> > > > D.
>> > > >
>> > > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song 
>> > > > wrote:
>> > > >
>> > > > > Thanks for the effort, @Chesnay. This is super helpful.
>> > > > >
>> > > > > @Jing,
>> > > > > Every push to the PR branch should automatically trigger an entire
>> > new
>> > > > > build. `@flinkbot run azure` should only be used when you want to
>> > > re-run
>> > > > > the failed stages without changing the PR. E.g., when running into
>> > > known
>> > > > > unstable cases that are unrelated to the PR.
>> > > > >
>> > > > > Thank you~
>> > > > >
>> > > > > Xintong Song
>> > > > >
>> > > > >
>> > > > >
>> > > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG 
>> > > > wrote:
>> > > > >
>> > > > > > Hi Chesnay,
>> > > > > > Thanks for the effort. It is a very useful improvement.
>> > > > > > I have a minor question. Please forgive me if the question is too
>> > > > naive.
>> > > > > > Since '@flinkbot run azure' now behaves like "Rerun failed jobs",
>> > is
>> > > > > there
>> > > > > > any way to trigger an entirely new build? Because some times I'm
>> > not
>> > > > sure
>> > > > > > the passed cases in the last build could still success in the new
>> > > build
>> > > > > > because of introduced updates in new commit.
>> > > > > >
>> > > > > > Best,
>> > > > > > JING ZHANG
>> > > > > >
>> > > > > >
>> > > > > > Yangze Guo  于2021年10月11日周一 上午10:31写道:
>> > > > > >
>> > > > > > > Thanks for that great job, Chesnay! "Rerun failed jobs" will
>> > help a
>> > > > > lot.
>> > > > > > >
>> > > > > > > Best,
>> > > > > > > Yangze Guo
>> > > > > > >
>> > > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
>> > > ches...@apache.org
>> > > > >
>> > > > > > > wrote:
>> > > > > > > >
>> > > > > > > > I made a number of changes to the CiBot over the weekend.
>> > > > > > > >
>> > > > > > > > - '@flinkbot run azure' previously triggered an entirely new
>> > > build
>> > > > > > based
>> > > > > > > > on the last completed one. It now instead retries the last
>> > > > completed
>> > > > > > > > build, only running the jobs that actually failed. It
>> basically
>> > > > > behaves
>> > > > > > > > like the "Rerun failed jobs" button in the Azure UI.
>> > > > > > > > - various optimizations to increase responsiveness (primarily
>> > by
>> > > > > doing
>> > > > > > > > significantly less unnecessary work / requests to GH)
>> > > > > > > > - removed TravisCI support (since we no longer support a
>> > release
>> > > > that
>> > > > > > > > used Travis)
>> > > > > > > >
>> > > > > > > > Please ping me if you spot anything weird.
>> > > > > > >
>> > > > > >
>> > > > >
>> > > >
>> > >
>> >
>>


Re: [NOTICE] CiBot improvements

2021-10-11 Thread Arvid Heise
Awesome!

On Tue, Oct 12, 2021 at 3:11 AM Guowei Ma  wrote:

> Thanks for your effort!
>
> Best,
> Guowei
>
>
> On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen  wrote:
>
> > Great initiative, thanks for doing this!
> >
> > On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann 
> > wrote:
> >
> > > Thanks a lot for this effort Chesnay! The improvements sound really
> good.
> > >
> > > Cheers,
> > > Till
> > >
> > > On Mon, Oct 11, 2021 at 8:46 AM David Morávek  wrote:
> > >
> > > > Nice! Thanks for the effort Chesnay, this is really a huge step
> > forward!
> > > >
> > > > Best,
> > > > D.
> > > >
> > > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song 
> > > > wrote:
> > > >
> > > > > Thanks for the effort, @Chesnay. This is super helpful.
> > > > >
> > > > > @Jing,
> > > > > Every push to the PR branch should automatically trigger an entire
> > new
> > > > > build. `@flinkbot run azure` should only be used when you want to
> > > re-run
> > > > > the failed stages without changing the PR. E.g., when running into
> > > known
> > > > > unstable cases that are unrelated to the PR.
> > > > >
> > > > > Thank you~
> > > > >
> > > > > Xintong Song
> > > > >
> > > > >
> > > > >
> > > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG 
> > > > wrote:
> > > > >
> > > > > > Hi Chesnay,
> > > > > > Thanks for the effort. It is a very useful improvement.
> > > > > > I have a minor question. Please forgive me if the question is too
> > > > naive.
> > > > > > Since '@flinkbot run azure' now behaves like "Rerun failed jobs",
> > is
> > > > > there
> > > > > > any way to trigger an entirely new build? Because some times I'm
> > not
> > > > sure
> > > > > > the passed cases in the last build could still success in the new
> > > build
> > > > > > because of introduced updates in new commit.
> > > > > >
> > > > > > Best,
> > > > > > JING ZHANG
> > > > > >
> > > > > >
> > > > > > Yangze Guo  于2021年10月11日周一 上午10:31写道:
> > > > > >
> > > > > > > Thanks for that great job, Chesnay! "Rerun failed jobs" will
> > help a
> > > > > lot.
> > > > > > >
> > > > > > > Best,
> > > > > > > Yangze Guo
> > > > > > >
> > > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
> > > ches...@apache.org
> > > > >
> > > > > > > wrote:
> > > > > > > >
> > > > > > > > I made a number of changes to the CiBot over the weekend.
> > > > > > > >
> > > > > > > > - '@flinkbot run azure' previously triggered an entirely new
> > > build
> > > > > > based
> > > > > > > > on the last completed one. It now instead retries the last
> > > > completed
> > > > > > > > build, only running the jobs that actually failed. It
> basically
> > > > > behaves
> > > > > > > > like the "Rerun failed jobs" button in the Azure UI.
> > > > > > > > - various optimizations to increase responsiveness (primarily
> > by
> > > > > doing
> > > > > > > > significantly less unnecessary work / requests to GH)
> > > > > > > > - removed TravisCI support (since we no longer support a
> > release
> > > > that
> > > > > > > > used Travis)
> > > > > > > >
> > > > > > > > Please ping me if you spot anything weird.
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> >
>


Re: [NOTICE] CiBot improvements

2021-10-11 Thread Guowei Ma
Thanks for your effort!

Best,
Guowei


On Mon, Oct 11, 2021 at 9:26 PM Stephan Ewen  wrote:

> Great initiative, thanks for doing this!
>
> On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann 
> wrote:
>
> > Thanks a lot for this effort Chesnay! The improvements sound really good.
> >
> > Cheers,
> > Till
> >
> > On Mon, Oct 11, 2021 at 8:46 AM David Morávek  wrote:
> >
> > > Nice! Thanks for the effort Chesnay, this is really a huge step
> forward!
> > >
> > > Best,
> > > D.
> > >
> > > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song 
> > > wrote:
> > >
> > > > Thanks for the effort, @Chesnay. This is super helpful.
> > > >
> > > > @Jing,
> > > > Every push to the PR branch should automatically trigger an entire
> new
> > > > build. `@flinkbot run azure` should only be used when you want to
> > re-run
> > > > the failed stages without changing the PR. E.g., when running into
> > known
> > > > unstable cases that are unrelated to the PR.
> > > >
> > > > Thank you~
> > > >
> > > > Xintong Song
> > > >
> > > >
> > > >
> > > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG 
> > > wrote:
> > > >
> > > > > Hi Chesnay,
> > > > > Thanks for the effort. It is a very useful improvement.
> > > > > I have a minor question. Please forgive me if the question is too
> > > naive.
> > > > > Since '@flinkbot run azure' now behaves like "Rerun failed jobs",
> is
> > > > there
> > > > > any way to trigger an entirely new build? Because some times I'm
> not
> > > sure
> > > > > the passed cases in the last build could still success in the new
> > build
> > > > > because of introduced updates in new commit.
> > > > >
> > > > > Best,
> > > > > JING ZHANG
> > > > >
> > > > >
> > > > > Yangze Guo  于2021年10月11日周一 上午10:31写道:
> > > > >
> > > > > > Thanks for that great job, Chesnay! "Rerun failed jobs" will
> help a
> > > > lot.
> > > > > >
> > > > > > Best,
> > > > > > Yangze Guo
> > > > > >
> > > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
> > ches...@apache.org
> > > >
> > > > > > wrote:
> > > > > > >
> > > > > > > I made a number of changes to the CiBot over the weekend.
> > > > > > >
> > > > > > > - '@flinkbot run azure' previously triggered an entirely new
> > build
> > > > > based
> > > > > > > on the last completed one. It now instead retries the last
> > > completed
> > > > > > > build, only running the jobs that actually failed. It basically
> > > > behaves
> > > > > > > like the "Rerun failed jobs" button in the Azure UI.
> > > > > > > - various optimizations to increase responsiveness (primarily
> by
> > > > doing
> > > > > > > significantly less unnecessary work / requests to GH)
> > > > > > > - removed TravisCI support (since we no longer support a
> release
> > > that
> > > > > > > used Travis)
> > > > > > >
> > > > > > > Please ping me if you spot anything weird.
> > > > > >
> > > > >
> > > >
> > >
> >
>


Re: [NOTICE] CiBot improvements

2021-10-11 Thread Stephan Ewen
Great initiative, thanks for doing this!

On Mon, Oct 11, 2021 at 10:52 AM Till Rohrmann  wrote:

> Thanks a lot for this effort Chesnay! The improvements sound really good.
>
> Cheers,
> Till
>
> On Mon, Oct 11, 2021 at 8:46 AM David Morávek  wrote:
>
> > Nice! Thanks for the effort Chesnay, this is really a huge step forward!
> >
> > Best,
> > D.
> >
> > On Mon, Oct 11, 2021 at 6:02 AM Xintong Song 
> > wrote:
> >
> > > Thanks for the effort, @Chesnay. This is super helpful.
> > >
> > > @Jing,
> > > Every push to the PR branch should automatically trigger an entire new
> > > build. `@flinkbot run azure` should only be used when you want to
> re-run
> > > the failed stages without changing the PR. E.g., when running into
> known
> > > unstable cases that are unrelated to the PR.
> > >
> > > Thank you~
> > >
> > > Xintong Song
> > >
> > >
> > >
> > > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG 
> > wrote:
> > >
> > > > Hi Chesnay,
> > > > Thanks for the effort. It is a very useful improvement.
> > > > I have a minor question. Please forgive me if the question is too
> > naive.
> > > > Since '@flinkbot run azure' now behaves like "Rerun failed jobs", is
> > > there
> > > > any way to trigger an entirely new build? Because some times I'm not
> > sure
> > > > the passed cases in the last build could still success in the new
> build
> > > > because of introduced updates in new commit.
> > > >
> > > > Best,
> > > > JING ZHANG
> > > >
> > > >
> > > > Yangze Guo  于2021年10月11日周一 上午10:31写道:
> > > >
> > > > > Thanks for that great job, Chesnay! "Rerun failed jobs" will help a
> > > lot.
> > > > >
> > > > > Best,
> > > > > Yangze Guo
> > > > >
> > > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler <
> ches...@apache.org
> > >
> > > > > wrote:
> > > > > >
> > > > > > I made a number of changes to the CiBot over the weekend.
> > > > > >
> > > > > > - '@flinkbot run azure' previously triggered an entirely new
> build
> > > > based
> > > > > > on the last completed one. It now instead retries the last
> > completed
> > > > > > build, only running the jobs that actually failed. It basically
> > > behaves
> > > > > > like the "Rerun failed jobs" button in the Azure UI.
> > > > > > - various optimizations to increase responsiveness (primarily by
> > > doing
> > > > > > significantly less unnecessary work / requests to GH)
> > > > > > - removed TravisCI support (since we no longer support a release
> > that
> > > > > > used Travis)
> > > > > >
> > > > > > Please ping me if you spot anything weird.
> > > > >
> > > >
> > >
> >
>


Re: [NOTICE] CiBot improvements

2021-10-11 Thread Till Rohrmann
Thanks a lot for this effort Chesnay! The improvements sound really good.

Cheers,
Till

On Mon, Oct 11, 2021 at 8:46 AM David Morávek  wrote:

> Nice! Thanks for the effort Chesnay, this is really a huge step forward!
>
> Best,
> D.
>
> On Mon, Oct 11, 2021 at 6:02 AM Xintong Song 
> wrote:
>
> > Thanks for the effort, @Chesnay. This is super helpful.
> >
> > @Jing,
> > Every push to the PR branch should automatically trigger an entire new
> > build. `@flinkbot run azure` should only be used when you want to re-run
> > the failed stages without changing the PR. E.g., when running into known
> > unstable cases that are unrelated to the PR.
> >
> > Thank you~
> >
> > Xintong Song
> >
> >
> >
> > On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG 
> wrote:
> >
> > > Hi Chesnay,
> > > Thanks for the effort. It is a very useful improvement.
> > > I have a minor question. Please forgive me if the question is too
> naive.
> > > Since '@flinkbot run azure' now behaves like "Rerun failed jobs", is
> > there
> > > any way to trigger an entirely new build? Because some times I'm not
> sure
> > > the passed cases in the last build could still success in the new build
> > > because of introduced updates in new commit.
> > >
> > > Best,
> > > JING ZHANG
> > >
> > >
> > > Yangze Guo  于2021年10月11日周一 上午10:31写道:
> > >
> > > > Thanks for that great job, Chesnay! "Rerun failed jobs" will help a
> > lot.
> > > >
> > > > Best,
> > > > Yangze Guo
> > > >
> > > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler  >
> > > > wrote:
> > > > >
> > > > > I made a number of changes to the CiBot over the weekend.
> > > > >
> > > > > - '@flinkbot run azure' previously triggered an entirely new build
> > > based
> > > > > on the last completed one. It now instead retries the last
> completed
> > > > > build, only running the jobs that actually failed. It basically
> > behaves
> > > > > like the "Rerun failed jobs" button in the Azure UI.
> > > > > - various optimizations to increase responsiveness (primarily by
> > doing
> > > > > significantly less unnecessary work / requests to GH)
> > > > > - removed TravisCI support (since we no longer support a release
> that
> > > > > used Travis)
> > > > >
> > > > > Please ping me if you spot anything weird.
> > > >
> > >
> >
>


Re: [NOTICE] CiBot improvements

2021-10-10 Thread David Morávek
Nice! Thanks for the effort Chesnay, this is really a huge step forward!

Best,
D.

On Mon, Oct 11, 2021 at 6:02 AM Xintong Song  wrote:

> Thanks for the effort, @Chesnay. This is super helpful.
>
> @Jing,
> Every push to the PR branch should automatically trigger an entire new
> build. `@flinkbot run azure` should only be used when you want to re-run
> the failed stages without changing the PR. E.g., when running into known
> unstable cases that are unrelated to the PR.
>
> Thank you~
>
> Xintong Song
>
>
>
> On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG  wrote:
>
> > Hi Chesnay,
> > Thanks for the effort. It is a very useful improvement.
> > I have a minor question. Please forgive me if the question is too naive.
> > Since '@flinkbot run azure' now behaves like "Rerun failed jobs", is
> there
> > any way to trigger an entirely new build? Because some times I'm not sure
> > the passed cases in the last build could still success in the new build
> > because of introduced updates in new commit.
> >
> > Best,
> > JING ZHANG
> >
> >
> > Yangze Guo  于2021年10月11日周一 上午10:31写道:
> >
> > > Thanks for that great job, Chesnay! "Rerun failed jobs" will help a
> lot.
> > >
> > > Best,
> > > Yangze Guo
> > >
> > > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler 
> > > wrote:
> > > >
> > > > I made a number of changes to the CiBot over the weekend.
> > > >
> > > > - '@flinkbot run azure' previously triggered an entirely new build
> > based
> > > > on the last completed one. It now instead retries the last completed
> > > > build, only running the jobs that actually failed. It basically
> behaves
> > > > like the "Rerun failed jobs" button in the Azure UI.
> > > > - various optimizations to increase responsiveness (primarily by
> doing
> > > > significantly less unnecessary work / requests to GH)
> > > > - removed TravisCI support (since we no longer support a release that
> > > > used Travis)
> > > >
> > > > Please ping me if you spot anything weird.
> > >
> >
>


Re: [NOTICE] CiBot improvements

2021-10-10 Thread Xintong Song
Thanks for the effort, @Chesnay. This is super helpful.

@Jing,
Every push to the PR branch should automatically trigger an entire new
build. `@flinkbot run azure` should only be used when you want to re-run
the failed stages without changing the PR. E.g., when running into known
unstable cases that are unrelated to the PR.

Thank you~

Xintong Song



On Mon, Oct 11, 2021 at 11:45 AM JING ZHANG  wrote:

> Hi Chesnay,
> Thanks for the effort. It is a very useful improvement.
> I have a minor question. Please forgive me if the question is too naive.
> Since '@flinkbot run azure' now behaves like "Rerun failed jobs", is there
> any way to trigger an entirely new build? Because some times I'm not sure
> the passed cases in the last build could still success in the new build
> because of introduced updates in new commit.
>
> Best,
> JING ZHANG
>
>
> Yangze Guo  于2021年10月11日周一 上午10:31写道:
>
> > Thanks for that great job, Chesnay! "Rerun failed jobs" will help a lot.
> >
> > Best,
> > Yangze Guo
> >
> > On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler 
> > wrote:
> > >
> > > I made a number of changes to the CiBot over the weekend.
> > >
> > > - '@flinkbot run azure' previously triggered an entirely new build
> based
> > > on the last completed one. It now instead retries the last completed
> > > build, only running the jobs that actually failed. It basically behaves
> > > like the "Rerun failed jobs" button in the Azure UI.
> > > - various optimizations to increase responsiveness (primarily by doing
> > > significantly less unnecessary work / requests to GH)
> > > - removed TravisCI support (since we no longer support a release that
> > > used Travis)
> > >
> > > Please ping me if you spot anything weird.
> >
>


Re: [NOTICE] CiBot improvements

2021-10-10 Thread JING ZHANG
Hi Chesnay,
Thanks for the effort. It is a very useful improvement.
I have a minor question. Please forgive me if the question is too naive.
Since '@flinkbot run azure' now behaves like "Rerun failed jobs", is there
any way to trigger an entirely new build? Because some times I'm not sure
the passed cases in the last build could still success in the new build
because of introduced updates in new commit.

Best,
JING ZHANG


Yangze Guo  于2021年10月11日周一 上午10:31写道:

> Thanks for that great job, Chesnay! "Rerun failed jobs" will help a lot.
>
> Best,
> Yangze Guo
>
> On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler 
> wrote:
> >
> > I made a number of changes to the CiBot over the weekend.
> >
> > - '@flinkbot run azure' previously triggered an entirely new build based
> > on the last completed one. It now instead retries the last completed
> > build, only running the jobs that actually failed. It basically behaves
> > like the "Rerun failed jobs" button in the Azure UI.
> > - various optimizations to increase responsiveness (primarily by doing
> > significantly less unnecessary work / requests to GH)
> > - removed TravisCI support (since we no longer support a release that
> > used Travis)
> >
> > Please ping me if you spot anything weird.
>


Re: [NOTICE] CiBot improvements

2021-10-10 Thread Jingsong Li
Thanks! "only running the jobs that actually failed" is very useful.

A small problem may have been solved. After a commit, we cannot rerun
the failed test before the commit. (The test triggered by commit has
not finished yet. The test triggered again should run the full cases)

Best,
Jingsong

On Mon, Oct 11, 2021 at 10:31 AM Yangze Guo  wrote:
>
> Thanks for that great job, Chesnay! "Rerun failed jobs" will help a lot.
>
> Best,
> Yangze Guo
>
> On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler  wrote:
> >
> > I made a number of changes to the CiBot over the weekend.
> >
> > - '@flinkbot run azure' previously triggered an entirely new build based
> > on the last completed one. It now instead retries the last completed
> > build, only running the jobs that actually failed. It basically behaves
> > like the "Rerun failed jobs" button in the Azure UI.
> > - various optimizations to increase responsiveness (primarily by doing
> > significantly less unnecessary work / requests to GH)
> > - removed TravisCI support (since we no longer support a release that
> > used Travis)
> >
> > Please ping me if you spot anything weird.



-- 
Best, Jingsong Lee


Re: [NOTICE] CiBot improvements

2021-10-10 Thread Yangze Guo
Thanks for that great job, Chesnay! "Rerun failed jobs" will help a lot.

Best,
Yangze Guo

On Sun, Oct 10, 2021 at 4:56 PM Chesnay Schepler  wrote:
>
> I made a number of changes to the CiBot over the weekend.
>
> - '@flinkbot run azure' previously triggered an entirely new build based
> on the last completed one. It now instead retries the last completed
> build, only running the jobs that actually failed. It basically behaves
> like the "Rerun failed jobs" button in the Azure UI.
> - various optimizations to increase responsiveness (primarily by doing
> significantly less unnecessary work / requests to GH)
> - removed TravisCI support (since we no longer support a release that
> used Travis)
>
> Please ping me if you spot anything weird.


[NOTICE] CiBot improvements

2021-10-10 Thread Chesnay Schepler

I made a number of changes to the CiBot over the weekend.

- '@flinkbot run azure' previously triggered an entirely new build based 
on the last completed one. It now instead retries the last completed 
build, only running the jobs that actually failed. It basically behaves 
like the "Rerun failed jobs" button in the Azure UI.
- various optimizations to increase responsiveness (primarily by doing 
significantly less unnecessary work / requests to GH)
- removed TravisCI support (since we no longer support a release that 
used Travis)


Please ping me if you spot anything weird.