Skip to content

ci(docs-infra): allow aio_local builds to fail on Travis - #26366

Closed
petebacondarwin wants to merge 1 commit into
angular:masterfrom
petebacondarwin:aio-allow-aio-local-failures
Closed

petebacondarwin wants to merge 1 commit into
angular:masterfrom
petebacondarwin:aio-allow-aio-local-failures

Conversation

@petebacondarwin

Copy link
Copy Markdown
Contributor

This job is flaky (up to 50%!) so let's allow it to fail while
we investigate the reason.

This job is flaky (up to 50%!) so let's allow it to fail while
we investigate the reason.
@petebacondarwin petebacondarwin added area: build & ci Related the build and CI infrastructure of the project severity2: inconvenient target: major This PR is targeted for the next major release labels Oct 10, 2018
@petebacondarwin petebacondarwin added this to the Backlog milestone Oct 10, 2018
@ngbot ngbot Bot modified the milestones: Backlog, needsTriage Oct 10, 2018
@pkozlowski-opensource

Copy link
Copy Markdown
Member

This would be much appreciated - currently many PRs are flaky on the PWA score test - it was failing > 50% of time on my PRs in 2-3 last days.

@IgorMinar IgorMinar added the action: merge The PR is ready for merge by the caretaker label Oct 10, 2018
@ngbot

ngbot Bot commented Oct 10, 2018

Copy link
Copy Markdown

I see that you just added the PR action: merge label, but the following checks are still failing:
    failure status "ci/circleci: integration_test" is failing
    pending status "code-review/pullapprove" is pending
    pending status "continuous-integration/travis-ci/pr" is pending

If you want your PR to be merged, it has to pass all the CI checks.

If you can't get the PR to a green state due to flakes or broken master, please try rebasing to master and/or restarting the CI job. If that fails and you believe that the issue is not due to your change, please contact the caretaker and ask for help.

@mhevery mhevery closed this in 7d0e175 Oct 11, 2018
@petebacondarwin
petebacondarwin deleted the aio-allow-aio-local-failures branch October 15, 2018 15:10
@mhevery mhevery added target: patch This PR is targeted for the next patch release and removed target: major This PR is targeted for the next major release labels Oct 15, 2018
@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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants