You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Daan Hoogland <da...@gmail.com> on 2020/10/21 08:09:13 UTC
recurring error in marvin tests for redundant VPCs
Please all keep an eye out. Something was merged (probably by me) that
broke the tests for VPCs [1] I have seen it in at least three smoke test
runs for different PRs and I don't know what to revert/fix. I'll be looking
at it but do appreciate help.
[1] https://github.com/apache/cloudstack/pull/4200#issuecomment-713227303
--
Daan
Re: recurring error in marvin tests for redundant VPCs
Posted by Wei ZHOU <us...@gmail.com>.
Hi Daan,
I checked some recent trillian test results, the issue should be fixed.
-Wei
On Wed, 25 Nov 2020 at 18:56, Daan Hoogland <da...@gmail.com> wrote:
> thanks, but 4413 was merged a month ago, so i wonder still...
>
> On Wed, Nov 25, 2020 at 12:06 PM Wei ZHOU <us...@gmail.com> wrote:
>
> > Hi Daan and Andrija,
> >
> > This should be caused by issue described in
> > https://github.com/apache/cloudstack/pull/4386
> > it has been fixed by https://github.com/apache/cloudstack/pull/4413
> >
> > -Wei
> >
> >
> >
> >
> >
> > On Wed, 25 Nov 2020 at 12:23, Andrija Panic <an...@gmail.com>
> > wrote:
> >
> > > These have been consistently unreliable for many releases, nothing to
> do
> > > with you, afaik.
> > >
> > > On Wed, 21 Oct 2020 at 10:09, Daan Hoogland <da...@gmail.com>
> > > wrote:
> > >
> > > > Please all keep an eye out. Something was merged (probably by me)
> that
> > > > broke the tests for VPCs [1] I have seen it in at least three smoke
> > test
> > > > runs for different PRs and I don't know what to revert/fix. I'll be
> > > looking
> > > > at it but do appreciate help.
> > > >
> > > > [1]
> > > https://github.com/apache/cloudstack/pull/4200#issuecomment-713227303
> > > >
> > > > --
> > > > Daan
> > > >
> > >
> > >
> > > --
> > >
> > > Andrija Panić
> > >
> >
>
>
> --
> Daan
>
Re: recurring error in marvin tests for redundant VPCs
Posted by Daan Hoogland <da...@gmail.com>.
thanks, but 4413 was merged a month ago, so i wonder still...
On Wed, Nov 25, 2020 at 12:06 PM Wei ZHOU <us...@gmail.com> wrote:
> Hi Daan and Andrija,
>
> This should be caused by issue described in
> https://github.com/apache/cloudstack/pull/4386
> it has been fixed by https://github.com/apache/cloudstack/pull/4413
>
> -Wei
>
>
>
>
>
> On Wed, 25 Nov 2020 at 12:23, Andrija Panic <an...@gmail.com>
> wrote:
>
> > These have been consistently unreliable for many releases, nothing to do
> > with you, afaik.
> >
> > On Wed, 21 Oct 2020 at 10:09, Daan Hoogland <da...@gmail.com>
> > wrote:
> >
> > > Please all keep an eye out. Something was merged (probably by me) that
> > > broke the tests for VPCs [1] I have seen it in at least three smoke
> test
> > > runs for different PRs and I don't know what to revert/fix. I'll be
> > looking
> > > at it but do appreciate help.
> > >
> > > [1]
> > https://github.com/apache/cloudstack/pull/4200#issuecomment-713227303
> > >
> > > --
> > > Daan
> > >
> >
> >
> > --
> >
> > Andrija Panić
> >
>
--
Daan
Re: recurring error in marvin tests for redundant VPCs
Posted by Wei ZHOU <us...@gmail.com>.
Hi Daan and Andrija,
This should be caused by issue described in
https://github.com/apache/cloudstack/pull/4386
it has been fixed by https://github.com/apache/cloudstack/pull/4413
-Wei
On Wed, 25 Nov 2020 at 12:23, Andrija Panic <an...@gmail.com> wrote:
> These have been consistently unreliable for many releases, nothing to do
> with you, afaik.
>
> On Wed, 21 Oct 2020 at 10:09, Daan Hoogland <da...@gmail.com>
> wrote:
>
> > Please all keep an eye out. Something was merged (probably by me) that
> > broke the tests for VPCs [1] I have seen it in at least three smoke test
> > runs for different PRs and I don't know what to revert/fix. I'll be
> looking
> > at it but do appreciate help.
> >
> > [1]
> https://github.com/apache/cloudstack/pull/4200#issuecomment-713227303
> >
> > --
> > Daan
> >
>
>
> --
>
> Andrija Panić
>
Re: recurring error in marvin tests for redundant VPCs
Posted by Andrija Panic <an...@gmail.com>.
These have been consistently unreliable for many releases, nothing to do
with you, afaik.
On Wed, 21 Oct 2020 at 10:09, Daan Hoogland <da...@gmail.com> wrote:
> Please all keep an eye out. Something was merged (probably by me) that
> broke the tests for VPCs [1] I have seen it in at least three smoke test
> runs for different PRs and I don't know what to revert/fix. I'll be looking
> at it but do appreciate help.
>
> [1] https://github.com/apache/cloudstack/pull/4200#issuecomment-713227303
>
> --
> Daan
>
--
Andrija Panić