commit fe3ecb802d6e2a967ddb762c97c8bf561b556e4e
author Matan Lurey <matan@lurey.org> Fri Aug 23 19:19:37 2024 +0000
committer CQ Bot Account <flutter-scoped@luci-project-accounts.iam.gserviceaccount.com> Fri Aug 23 19:19:37 2024 +0000
tree 59f43f68c9f4c14ac26d10653c0fe903e1fb8840
parent 57f92aba78607ab84cae8db1458b8461b95cf8f5

Reland "Reland "Use logs_util.upload_logs in the flutter_drone recipe.""

This reverts commit 2e9c17ce72ae354230b6942f355977e005b4f5a4.

Reason for revert: Fixed by https://github.com/flutter/flutter/pull/153966, hopefully the last one.

Original change's description:
> Revert "Reland "Use logs_util.upload_logs in the flutter_drone recipe.""
>
> This reverts commit fd459d0f5591fc06874b2a248b2d9c6de3ecc231.
>
> Reason for revert: Broke another test (https://ci.chromium.org/ui/p/flutter/builders/prod/Linux%20web_long_running_tests_5_5/18054/overview)
>
> Original change's description:
> > Reland "Use logs_util.upload_logs in the flutter_drone recipe."
> >
> > This reverts commit 69d6127512b217887ea632fa4c34776da7788e13.
> >
> > Reason for revert: The affected test (https://github.com/flutter/flutter/pull/153888) has been fixed to handle the variable.
> >
> > Original change's description:
> > > Revert "Use logs_util.upload_logs in the flutter_drone recipe."
> > >
> > > This reverts commit b111cd3ef2297e66905cc48c2cfedce0cf1ba46f.
> > >
> > > Reason for revert: https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_arm64%20framework_tests_misc/3191/overview
> > >
> > > Original change's description:
> > > > Use logs_util.upload_logs in the flutter_drone recipe.
> > > >
> > > > Bug: https://github.com/flutter/flutter/issues/152775
> > > > Change-Id: I3840d064a96e32e32c23c04422eea7802ddfdeeb
> > > > Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/59220
> > > > Reviewed-by: Christopher Fujino <fujino@google.com>
> > > > Reviewed-by: Zach Anderson <zra@google.com>
> > >
> > > TBR=zra@google.com,fujino@google.com,matanl@google.com
> > >
> > > Change-Id: I13bb70a1bcb0109e4e87639e0cc8b0964a97b932
> > > No-Presubmit: true
> > > No-Tree-Checks: true
> > > No-Try: true
> > > Bug: https://github.com/flutter/flutter/issues/152775
> > > Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/59341
> > > Reviewed-by: Zach Anderson <zra@google.com>
> > > Reviewed-by: Christopher Fujino <fujino@google.com>
> > > Reviewed-by: Chris Bracken <cbracken@google.com>
> >
> > # Not skipping CQ checks because this is a reland.
> >
> > Bug: https://github.com/flutter/flutter/issues/152775
> > Change-Id: I155c914a46e3dbe623578b6c327225cb736a479f
> > Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/59343
> > Reviewed-by: Chris Bracken <cbracken@google.com>
> > Reviewed-by: Christopher Fujino <fujino@google.com>
> > Reviewed-by: Matan Lurey <matanl@google.com>
>
> TBR=zra@google.com,fujino@google.com,cbracken@google.com,matanl@google.com
>
> Change-Id: I0bf451f142fd7ce785b8faea7bc6a5c78014431a
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: https://github.com/flutter/flutter/issues/152775
> Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/59347
> Reviewed-by: Matan Lurey <matanl@google.com>
> Commit-Queue: Matan Lurey <matanl@google.com>
> Reviewed-by: Christopher Fujino <fujino@google.com>


Bug: https://github.com/flutter/flutter/issues/152775
Change-Id: I109d7dead105aea7cb76ab93bb2b9f881215d004
Reviewed-on: https://flutter-review.googlesource.com/c/recipes/+/59349
Reviewed-by: Zach Anderson <zra@google.com>
Commit-Queue: Matan Lurey <matanl@google.com>
Reviewed-by: Matan Lurey <matanl@google.com>