commit a36e5c25a12c13e371769f89a2746708e616df96
author Yuping Zhai <yupingz@fuchsia.infra.roller.google.com> Mon Aug 21 21:09:53 2023 +0000
committer Copybara-Service <copybara-worker@google.com> Mon Aug 21 14:11:00 2023 -0700
tree 05900d41ea7938453b1f3328d61954d99b0f1da1
parent f641bf23f95ffe34bae0c467263c64f2626ba798

[roll] Roll infra/recipes Reland "Reland "Reland "Reland "[checkout] Add step to ensure submodules are update to date""""

This reverts commit 25e276b9de2dcde5ed8ca85f35cee95c31347aac.

Reason for revert: Initial CL to remove ensure files (cl/558847381) did not land. Hence, the ensure files were still showing when the revert happened. The CL is now merged. We can start another reland now.

Original change's description:
> Revert "Reland "Reland "Reland "[checkout] Add step to ensure submodules are update to date""""
>
> This reverts commit b3ccda327b88de9a79a0614a7829f25fdc72f020.
>
> Reason for revert: https://logs.chromium.org/logs/fuchsia/buildbucket/cr-buildbucket/8772580491063375777/+/u/build/get_changed_files/git_diff-tree/files is showing superproject affected files
>
> Original change's description:
> > Reland "Reland "Reland "[checkout] Add step to ensure submodules are update to date"""
> >
> > This reverts commit 29ca2dec431359c5bda3a646688ab1b6eec68357.
> >
> > Reason for revert: Removed the extra ensure files that affected tests are picking up. Updated affected tests so that the right diff is shown instead of the last commit.
> >
> > Original change's description:
> > > Revert "Reland "Reland "[checkout] Add step to ensure submodules are update to date"""
> > >
> > > This reverts commit 0f4845886c88c905f53d38fb0291e766ade14c80.
> > >
> > > Reason for revert: creating a new commit causes a different set of
> > > files to be considered affected, breaking affected file analysis in
> > > presubmit
> > >
> > > Original-Bug: 131929
> > >
> > > Original change's description:
> > > > Reland "Reland "[checkout] Add step to ensure submodules are update to date""
> > > >
> > > > This reverts commit d444a612038cc6ceb446f2ddea15e73de0fc66d9.
> > > >
> > > > Reason for revert: We have now fixed the issue where internal projects are skipped for the submodule tool.
> > > >
> > > > Original change's description:
> > > > > Revert "Reland "[checkout] Add step to ensure submodules are update to date""
> > > > >
> > > > > This reverts commit 31f020630a31c3612afedea56a72ba6bc05b2f1a.
> > > > >
> > > > > Reason for revert: Observing infra failures related to submodule updates on bringup.arm64-hwasan and bringup.x64-asan-submodule e.g. go/bbid/8775179096962942129, go/bbid/8775179101078604225
> > > > >
> > > > > Original change's description:
> > > > > > Reland "[checkout] Add step to ensure submodules are update to date"
> > > > > >
> > > > > > This reverts commit 0213ec5e992d7d63e0e8ef7aafe47fddf3a16a47.
> > > > > >
> > > > > > Reason for revert: Internal projects are not being excluded in submodule-update tool, which caused many submodules to be created. The tool has now being updated.
> > > > > >
> > > > > > Original change's description:
> > > > > > > Revert "[checkout] Add step to ensure submodules are update to date"
> > > > > > >
> > > > > > > This reverts commit 8a48b2351c59c7098869409f60f24544eabfadc4.
> > > > > > >
> > > > > > > Reason for revert: infra failure
> > > > > > >
> > > > > > > Original change's description:
> > > > > > > > [checkout] Add step to ensure submodules are update to date
> > > > > > > >
> > > > > > > > When submodules are enabled, submodules are the source of truth. We need to
> > > > > > > > ensure submodule revisions are matching project pins at integration.
> > > > > > > >
> > > > > > > > Original-Bug: b/289291544
> > > > > > > > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/883173
> > > > > > >
> > > > > > > Original-Bug: b/289291544
> > > > > > > No-Presubmit: true
> > > > > > > No-Tree-Checks: true
> > > > > > > No-Try: true
> > > > > > > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/885891
> > > > > >
> > > > > > Original-Bug: b/289291544
> > > > > > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/886554
> > > > >
> > > > > Original-Bug: b/289291544
> > > > > No-Presubmit: true
> > > > > No-Tree-Checks: true
> > > > > No-Try: true
> > > > > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/886555
> > > >
> > > > Original-Bug: b/289291544
> > > > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/894139
> > >
> > > Original-Bug: b/289291544
> > > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/897160
> >
> > Original-Bug: 131929
> > Original-Bug: b/289291544
> > Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/897783
>
> Original-Bug: 131929
> Original-Bug: b/289291544
> Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/904825

Original-Bug: 131929
Original-Bug: b/289291544
Original-Reviewed-on: https://fuchsia-review.googlesource.com/c/infra/recipes/+/905393
Original-Revision: 37d5755ff0ac3324e6e0e5aef18607731024e079
GitOrigin-RevId: e533fd7a154639f391221a5379e3b9b10cb63067
Change-Id: I8fb53ce013f1c26aa0cd6a649f7d8a45ab9a1188