Explicit order (#19216)

[cincuranet]

Fixes #19216.

[smitpatel]

This broke cosmos tests. Build is broken!

[cincuranet]

It's green in ADO.

[roji]

Oh shoot, I'm guessing the Cosmos tests don't run on PRs? I'll take a look at this a later today (or first thing tomorrow morning, European time).

We should discuss if this is a good situation CI-wise...

[ajcvickers]

@roji I have a bit of time available now, so I'll send something out to fix the build.

We should discuss if this is a good situation CI-wise...

We have been discussing this, and @AndriySvyryd spent considerable effort trying to get them to run reliably, with very little luck.

[AndriySvyryd]

We wouldn't run Cosmos tests on fork PRs anyway due to security concerns.

[roji]

And still no way to get the emulator onto the image somehow? How about trying to install it as part of the build itself? I know it sounds weird but it's commonly done at least in Linux CI scripts.

[roji]

Oh, and thanks for the help @ajcvickers

[smitpatel]

We wouldn't run Cosmos tests on fork PRs anyway due to security concerns.

Does it imply that any external PR which can affect Cosmos build has to be verified locally before merging too?

[ajcvickers]

@smitpatel @AndriySvyryd @roji Let's discuss this on an internal channel.