envoy-mobile icon indicating copy to clipboard operation
envoy-mobile copied to clipboard

[DO NOT MERGE] Moving endpoint for running CI jobs in new EngFlow cluster

Open anfelbar opened this issue 3 years ago • 2 comments

Description: Engflow is making some internal changes to make the infrastructure more scalable and that requires using a new endpoint. This PR runs all necessary CI jobs in the new cluster. Risk Level: Low Testing: N/A Docs Changes: N/A Release Notes: N/A [Optional Fixes #Issue] [Optional Deprecated:]

Signed-off-by: Andrés Felipe Barco Santa [email protected]

anfelbar avatar Aug 29 '22 16:08 anfelbar

@lfpino I need a review for this.

anfelbar avatar Sep 19 '22 17:09 anfelbar

@lfpino we are ready to move to the new cluster. docs are failing in the circleci but not in our cluster.

anfelbar avatar Sep 22 '22 16:09 anfelbar

We have made some dns changes that are propagation now. Even without the propagation of dns, if this PR is approved then the cluster endpoint gets updated and the build can continue on the new cluster (each branch should merge the master branch).

anfelbar avatar Sep 23 '22 23:09 anfelbar

Some of the workflows are failing, can you look into that?

@jpsim could you please review this?

lfpino avatar Sep 24 '22 00:09 lfpino

could you please review this?

What are you suggesting I review? I don't see any changes in this branch.

image

jpsim avatar Sep 26 '22 18:09 jpsim

could you please review this?

What are you suggesting I review? I don't see any changes in this branch.

@jpsim actually we just changed DNS as to be able to work with the same envoy.cluster.engflow.com endpoint (that now points to the new modularized cluster). The only thing remaining is to test releases that are not triggered by the normal PR. Is it possible for you to trigger the releases building process? Once we know that part is working then we can close this PR without merging it. This is the last step.

anfelbar avatar Sep 26 '22 18:09 anfelbar

The only thing remaining is to test releases that are not triggered by the normal PR. Is it possible for you to trigger the releases building process?

Running the release jobs now: https://github.com/envoyproxy/envoy-mobile/actions/runs/3128074078

The previous run failed yesterday with SSL errors: https://github.com/envoyproxy/envoy-mobile/actions/runs/3128074078/jobs/5075452741

jpsim avatar Sep 27 '22 14:09 jpsim

/retest

jpsim avatar Sep 27 '22 14:09 jpsim

The release jobs passed: https://github.com/envoyproxy/envoy-mobile/actions/runs/3128074078

jpsim avatar Sep 27 '22 14:09 jpsim

The release jobs passed: https://github.com/envoyproxy/envoy-mobile/actions/runs/3128074078

Wonderful. So right now the migration is complete and the same endpoint envoy.cluster.engflow.com is used. I'm closing this testing PR. Thank you @jpsim and @lfpino

The only thing remaining is to test releases that are not triggered by the normal PR. Is it possible for you to trigger the releases building process?

Running the release jobs now: https://github.com/envoyproxy/envoy-mobile/actions/runs/3128074078

The previous run failed yesterday with SSL errors: https://github.com/envoyproxy/envoy-mobile/actions/runs/3128074078/jobs/5075452741

Most likely it failed because DNS propagation.

anfelbar avatar Sep 27 '22 15:09 anfelbar

The new cluster is then in use with the same endpoint envoy.cluster.engflow.com. I am closing this PR. Thank you @jpsim and @lfpino

anfelbar avatar Sep 27 '22 15:09 anfelbar