Обсуждение: cfbot is listing committed patches?

cfbot [1] is listing some already committed patches under the "Needs
Review" category. For example here are some of mine [1][2]. And
because they are already committed, the 'apply'  fails, so they get
flagged by cfbot as needed rebase.

Something seems broken.

------
[1] http://cfbot.cputube.org/next.html
[2] https://commitfest.postgresql.org/43/4246/
[3] https://commitfest.postgresql.org/43/4266/

Kind Regards,
Peter Smith.
Fujitsu Australia

On Tue, Apr 11, 2023 at 6:16 PM Peter Smith <smithpb2250@gmail.com> wrote:
> cfbot [1] is listing some already committed patches under the "Needs
> Review" category. For example here are some of mine [1][2]. And
> because they are already committed, the 'apply'  fails, so they get
> flagged by cfbot as needed rebase.
>
> Something seems broken.

Thanks, fixed.  It was confused because after CF #42 was recently
closed, #43 became "current" but there is not yet a "next" commitfest.
I never noticed before, but I guess those are manually created.

On Tue, Apr 11, 2023 at 4:36 PM Thomas Munro <thomas.munro@gmail.com> wrote:
>
> On Tue, Apr 11, 2023 at 6:16 PM Peter Smith <smithpb2250@gmail.com> wrote:
> > cfbot [1] is listing some already committed patches under the "Needs
> > Review" category. For example here are some of mine [1][2]. And
> > because they are already committed, the 'apply'  fails, so they get
> > flagged by cfbot as needed rebase.
> >
> > Something seems broken.
>
> Thanks, fixed.  It was confused because after CF #42 was recently
> closed, #43 became "current" but there is not yet a "next" commitfest.
> I never noticed before, but I guess those are manually created.

Oh, right. And I was mistakenly looking at the cfbot page of the
"next" commitfest even though there is no such thing. Thanks for the
fix/explanation.

------
Kind Regards,
Peter Smith.
Fujitsu Australia