Re: Duplicate Requests in autopkgtest-cloud

2023-08-16 Thread Tim Andersson
Hi Athos,

Sorry, we overwrote the hotfix the other day with a deployment, sorry. I've
reinstated it now. Please let me know if you have any issues!

Regards,
Tim


On Wed, 16 Aug 2023 at 16:54, Athos Ribeiro 
wrote:

> On Fri, Jul 28, 2023 at 10:01:50AM +0100, Tim Andersson wrote:
> >Hi Steve,
> >
> >This is something I missed in the initial implementation, but there's now
> >an MP for a fix ready to go into master. Right now, however, I've hotfixed
> >prod so that if you pass `all-proposed`, the duplicate request check is
> >disabled. I made this quick change to unblock ginggs
>
> Hi Tim,
>
> is the hotfix still up?
>
> I just got a request with all-proposed blocked because a request without
> it was queued.
>
> >
> >On Fri, Jul 28, 2023 at 5:19 AM Steve Langasek  >
> >wrote:
> >
> >> Hi Tim,
> >>
> >> On Thu, Jul 27, 2023 at 11:10:05AM +0100, Tim Andersson wrote:
> >> > Hi all,
> >>
> >> > In the Ubuntu QA team we recently made and deployed a change which now
> >> > makes it impossible to queue duplicate requests.
> >>
> >> > If a request is currently in the queue, or is currently running, and
> you
> >> > request the same test, you will be taken to an error page which tells
> you
> >> > the test details and whether it is currently queued or currently
> running.
> >> > It looks like this:
> >> > ```
> >> >
> >> > You submitted an invalid request:
> >> >
> >> > Test already queued:
> >> >
> >> > release: lunar
> >> >
> >> > pkg: gzip
> >> >
> >> > arch: arm64
> >> >
> >> > triggers: gzip/1.12-1ubuntu1
> >> >
> >> > ```
> >> > This is to try and ease the load on autopkgtest-cloud.
> >>
> >> > If you experience any bugs or unexpected functionality, please file a
> bug
> >> > against `autopkgtest-cloud` and let us know. We expect it to work
> >> > seamlessly but always expect the unexpected right :)
> >>
> >> Does the code also properly distinguish between tests queued with
> >> proposed=1
> >> and those without, so that it's possible to queue both ways in parallel?
> >>
> >> Thanks,
>
> --
> Athos Ribeiro
>
> --
> ubuntu-devel mailing list
> ubuntu-devel@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel
>
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-08-16 Thread Athos Ribeiro

On Fri, Jul 28, 2023 at 10:01:50AM +0100, Tim Andersson wrote:

Hi Steve,

This is something I missed in the initial implementation, but there's now
an MP for a fix ready to go into master. Right now, however, I've hotfixed
prod so that if you pass `all-proposed`, the duplicate request check is
disabled. I made this quick change to unblock ginggs


Hi Tim,

is the hotfix still up?

I just got a request with all-proposed blocked because a request without
it was queued.



On Fri, Jul 28, 2023 at 5:19 AM Steve Langasek 
wrote:


Hi Tim,

On Thu, Jul 27, 2023 at 11:10:05AM +0100, Tim Andersson wrote:
> Hi all,

> In the Ubuntu QA team we recently made and deployed a change which now
> makes it impossible to queue duplicate requests.

> If a request is currently in the queue, or is currently running, and you
> request the same test, you will be taken to an error page which tells you
> the test details and whether it is currently queued or currently running.
> It looks like this:
> ```
>
> You submitted an invalid request:
>
> Test already queued:
>
> release: lunar
>
> pkg: gzip
>
> arch: arm64
>
> triggers: gzip/1.12-1ubuntu1
>
> ```
> This is to try and ease the load on autopkgtest-cloud.

> If you experience any bugs or unexpected functionality, please file a bug
> against `autopkgtest-cloud` and let us know. We expect it to work
> seamlessly but always expect the unexpected right :)

Does the code also properly distinguish between tests queued with
proposed=1
and those without, so that it's possible to queue both ways in parallel?

Thanks,


--
Athos Ribeiro

--
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-08-09 Thread Tim Andersson
Hey Utkarsh,

Thanks for reporting this! Yeah we've recently become aware of this. It
takes a few seconds for a request to hit the queue, so if you spam refresh
or re-queue you can queue multiple times. Not really sure how to amend this
just yet, but we'll look to amend it in the future.

Thanks,
Tim

On Tue, Aug 8, 2023 at 3:50 PM Utkarsh Gupta 
wrote:

> Hi Tim,
>
> On Thu, Jul 27, 2023 at 4:01 PM Tim Andersson
>  wrote:
> > In the Ubuntu QA team we recently made and deployed a change
> > which now makes it impossible to queue duplicate requests.
>
> Super, this is great stuff!
>
> However, this isn't working as advertised. Or perhaps I've gotten it
> wrong. But I wanted to trigger 'livecd-rootfs/23.10.12' for
> mantic/ppc64el and I did but just to try the whole duplicating thing,
> I did another time, and it did again. On the third time, however, it
> said it's already queued. Upon checking
> https://autopkgtest.ubuntu.com/running, I can confirm that the
> autopkgtest is running twice, triggered 9 seconds apart. Is that
> anyhow expected?
>
>
> - u
>
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-08-08 Thread Utkarsh Gupta
Hi Tim,

On Thu, Jul 27, 2023 at 4:01 PM Tim Andersson
 wrote:
> In the Ubuntu QA team we recently made and deployed a change
> which now makes it impossible to queue duplicate requests.

Super, this is great stuff!

However, this isn't working as advertised. Or perhaps I've gotten it
wrong. But I wanted to trigger 'livecd-rootfs/23.10.12' for
mantic/ppc64el and I did but just to try the whole duplicating thing,
I did another time, and it did again. On the third time, however, it
said it's already queued. Upon checking
https://autopkgtest.ubuntu.com/running, I can confirm that the
autopkgtest is running twice, triggered 9 seconds apart. Is that
anyhow expected?


- u

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-07-29 Thread Steve Langasek
On Fri, Jul 28, 2023 at 10:01:50AM +0100, Tim Andersson wrote:
> Hi Steve,

> This is something I missed in the initial implementation, but there's now
> an MP for a fix ready to go into master. Right now, however, I've hotfixed
> prod so that if you pass `all-proposed`, the duplicate request check is
> disabled. I made this quick change to unblock ginggs

Excellent, thank you!

> On Fri, Jul 28, 2023 at 5:19 AM Steve Langasek 
> wrote:
> 
> > Hi Tim,
> >
> > On Thu, Jul 27, 2023 at 11:10:05AM +0100, Tim Andersson wrote:
> > > Hi all,
> >
> > > In the Ubuntu QA team we recently made and deployed a change which now
> > > makes it impossible to queue duplicate requests.
> >
> > > If a request is currently in the queue, or is currently running, and you
> > > request the same test, you will be taken to an error page which tells you
> > > the test details and whether it is currently queued or currently running.
> > > It looks like this:
> > > ```
> > >
> > > You submitted an invalid request:
> > >
> > > Test already queued:
> > >
> > > release: lunar
> > >
> > > pkg: gzip
> > >
> > > arch: arm64
> > >
> > > triggers: gzip/1.12-1ubuntu1
> > >
> > > ```
> > > This is to try and ease the load on autopkgtest-cloud.
> >
> > > If you experience any bugs or unexpected functionality, please file a bug
> > > against `autopkgtest-cloud` and let us know. We expect it to work
> > > seamlessly but always expect the unexpected right :)
> >
> > Does the code also properly distinguish between tests queued with
> > proposed=1
> > and those without, so that it's possible to queue both ways in parallel?
> >
> > Thanks,
> > --
> > Steve Langasek   Give me a lever long enough and a Free OS
> > Debian Developer   to set it on, and I can move the world.
> > Ubuntu Developer   https://www.debian.org/
> > slanga...@ubuntu.com vor...@debian.org
> > --
> > ubuntu-devel mailing list
> > ubuntu-devel@lists.ubuntu.com
> > Modify settings or unsubscribe at:
> > https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel
> >

-- 
Steve Langasek   Give me a lever long enough and a Free OS
Debian Developer   to set it on, and I can move the world.
Ubuntu Developer   https://www.debian.org/
slanga...@ubuntu.com vor...@debian.org


signature.asc
Description: PGP signature
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-07-28 Thread Tim Andersson
Hi Steve,

This is something I missed in the initial implementation, but there's now
an MP for a fix ready to go into master. Right now, however, I've hotfixed
prod so that if you pass `all-proposed`, the duplicate request check is
disabled. I made this quick change to unblock ginggs

Regards,
Tim

On Fri, Jul 28, 2023 at 5:19 AM Steve Langasek 
wrote:

> Hi Tim,
>
> On Thu, Jul 27, 2023 at 11:10:05AM +0100, Tim Andersson wrote:
> > Hi all,
>
> > In the Ubuntu QA team we recently made and deployed a change which now
> > makes it impossible to queue duplicate requests.
>
> > If a request is currently in the queue, or is currently running, and you
> > request the same test, you will be taken to an error page which tells you
> > the test details and whether it is currently queued or currently running.
> > It looks like this:
> > ```
> >
> > You submitted an invalid request:
> >
> > Test already queued:
> >
> > release: lunar
> >
> > pkg: gzip
> >
> > arch: arm64
> >
> > triggers: gzip/1.12-1ubuntu1
> >
> > ```
> > This is to try and ease the load on autopkgtest-cloud.
>
> > If you experience any bugs or unexpected functionality, please file a bug
> > against `autopkgtest-cloud` and let us know. We expect it to work
> > seamlessly but always expect the unexpected right :)
>
> Does the code also properly distinguish between tests queued with
> proposed=1
> and those without, so that it's possible to queue both ways in parallel?
>
> Thanks,
> --
> Steve Langasek   Give me a lever long enough and a Free OS
> Debian Developer   to set it on, and I can move the world.
> Ubuntu Developer   https://www.debian.org/
> slanga...@ubuntu.com vor...@debian.org
> --
> ubuntu-devel mailing list
> ubuntu-devel@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel
>
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-07-27 Thread Steve Langasek
Hi Tim,

On Thu, Jul 27, 2023 at 11:10:05AM +0100, Tim Andersson wrote:
> Hi all,

> In the Ubuntu QA team we recently made and deployed a change which now
> makes it impossible to queue duplicate requests.

> If a request is currently in the queue, or is currently running, and you
> request the same test, you will be taken to an error page which tells you
> the test details and whether it is currently queued or currently running.
> It looks like this:
> ```
> 
> You submitted an invalid request:
> 
> Test already queued:
> 
> release: lunar
> 
> pkg: gzip
> 
> arch: arm64
> 
> triggers: gzip/1.12-1ubuntu1
> 
> ```
> This is to try and ease the load on autopkgtest-cloud.

> If you experience any bugs or unexpected functionality, please file a bug
> against `autopkgtest-cloud` and let us know. We expect it to work
> seamlessly but always expect the unexpected right :)

Does the code also properly distinguish between tests queued with proposed=1
and those without, so that it's possible to queue both ways in parallel?

Thanks,
-- 
Steve Langasek   Give me a lever long enough and a Free OS
Debian Developer   to set it on, and I can move the world.
Ubuntu Developer   https://www.debian.org/
slanga...@ubuntu.com vor...@debian.org


signature.asc
Description: PGP signature
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: Duplicate Requests in autopkgtest-cloud

2023-07-27 Thread Sergio Durigan Junior
On Thursday, July 27 2023, Tim Andersson wrote:

> In the Ubuntu QA team we recently made and deployed a change which now
> makes it impossible to queue duplicate requests.
[...]

Thank you very much, Tim.  This is a much appreciated feature.

-- 
Sergio
GPG key ID: E92F D0B3 6B14 F1F4 D8E0  EB2F 106D A1C8 C3CB BF14

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Duplicate Requests in autopkgtest-cloud

2023-07-27 Thread Tim Andersson
Hi all,

In the Ubuntu QA team we recently made and deployed a change which now
makes it impossible to queue duplicate requests.

If a request is currently in the queue, or is currently running, and you
request the same test, you will be taken to an error page which tells you
the test details and whether it is currently queued or currently running.
It looks like this:
```

You submitted an invalid request:

Test already queued:

release: lunar

pkg: gzip

arch: arm64

triggers: gzip/1.12-1ubuntu1

```
This is to try and ease the load on autopkgtest-cloud.

If you experience any bugs or unexpected functionality, please file a bug
against `autopkgtest-cloud` and let us know. We expect it to work
seamlessly but always expect the unexpected right :)

Thanks,
Ubuntu QA
-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel