Skip to content

ci(docs-infra): increase wait for SW on all origins to avoid CI flakes - #29988

Closed
gkalpak wants to merge 1 commit into
angular:masterfrom
gkalpak:ci-aio-fix-pwa-flakes
Closed

gkalpak wants to merge 1 commit into
angular:masterfrom
gkalpak:ci-aio-fix-pwa-flakes

Conversation

@gkalpak

@gkalpak gkalpak commented Apr 19, 2019 •

Copy link
Copy Markdown
Member

In #29953, the wait period for SW on localhost was increased to avoid CI flakes for the PWA score tests.

This commit expands the fix to non-localhost origins to avoid flakes in the aio_monitoring job, when CircleCI VMs/network are slow.
(For reference, example failures: 289127, 289238)

@gkalpak gkalpak added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project effort1: hours comp: docs-infra target: patch This PR is targeted for the next patch release risk: low labels Apr 19, 2019
@gkalpak gkalpak added this to the docs-infra-aio-app milestone Apr 19, 2019
@gkalpak
gkalpak requested a review from a team April 19, 2019 10:06
In angular#29953, the wait period for SW on localhost was increased to avoid CI
flakes for the PWA score tests.

This commit expands the fix to non-localhost origins to avoid flakes in
the `aio_monitoring` job, when CircleCI VMs/network are slow.
(For reference, example failures: [289127], [289238])

[289127]: https://circleci.com/gh/angular/angular/289127
[289238]: https://circleci.com/gh/angular/angular/289238
@gkalpak
gkalpak force-pushed the ci-aio-fix-pwa-flakes branch from 8b0187d to ea10345 Compare April 19, 2019 10:11
@mary-poppins

Copy link
Copy Markdown

You can preview 8b0187d at https://pr29988-8b0187d.ngbuilds.io/.

@mary-poppins

Copy link
Copy Markdown

You can preview ea10345 at https://pr29988-ea10345.ngbuilds.io/.

benlesh pushed a commit that referenced this pull request Apr 19, 2019
#29988)

In #29953, the wait period for SW on localhost was increased to avoid CI
flakes for the PWA score tests.

This commit expands the fix to non-localhost origins to avoid flakes in
the `aio_monitoring` job, when CircleCI VMs/network are slow.
(For reference, example failures: [289127], [289238])

[289127]: https://circleci.com/gh/angular/angular/289127
[289238]: https://circleci.com/gh/angular/angular/289238

PR Close #29988
@benlesh benlesh closed this in 1f8325d Apr 19, 2019
@gkalpak
gkalpak deleted the ci-aio-fix-pwa-flakes branch April 19, 2019 18:08
BioPhoton pushed a commit to BioPhoton/angular that referenced this pull request May 21, 2019
angular#29988)

In angular#29953, the wait period for SW on localhost was increased to avoid CI
flakes for the PWA score tests.

This commit expands the fix to non-localhost origins to avoid flakes in
the `aio_monitoring` job, when CircleCI VMs/network are slow.
(For reference, example failures: [289127], [289238])

[289127]: https://circleci.com/gh/angular/angular/289127
[289238]: https://circleci.com/gh/angular/angular/289238

PR Close angular#29988
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project cla: yes effort1: hours risk: low target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants