Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

All new PRs are blocked on failing Chrome stability job #13112

Closed
foolip opened this issue Sep 20, 2018 · 8 comments
Closed

All new PRs are blocked on failing Chrome stability job #13112

foolip opened this issue Sep 20, 2018 · 8 comments

Comments

@foolip
Copy link
Member

foolip commented Sep 20, 2018

Lots of recently opened PRs are blocked on failing Travis checks.

This is the earliest I can find, from https://travis-ci.org/web-platform-tests/wpt/jobs/430913594:

Traceback (most recent call last):
  File "./wpt", line 5, in <module>
    wpt.main()
  File "/home/travis/build/web-platform-tests/wpt/tools/wpt/wpt.py", line 129, in main
    rv = script(*args, **kwargs)
  File "/home/travis/build/web-platform-tests/wpt/tools/ci/check_stability.py", line 276, in run
    wpt_kwargs = setup_wptrunner(venv, **wpt_kwargs)
  File "/home/travis/build/web-platform-tests/wpt/tools/wpt/run.py", line 458, in setup_wptrunner
    check_environ(kwargs["product"])
  File "/home/travis/build/web-platform-tests/wpt/tools/wpt/run.py", line 140, in check_environ
    raise WptrunError(message)
tools.wpt.run.WptrunError: Missing hosts file configuration. Run
./wpt make-hosts-file | sudo tee -a /etc/hosts

Affected PRs:
#13092
#13093
#13094
#13095
#13103
#13104
#13105
#13106
#13109
#13110

Failing status checks for other reasons:
#13107
#13108

Not affected because "No tests changed" in log:
#13096
#13097
#13098
#13099
#13100
#13101
#13102

@foolip
Copy link
Member Author

foolip commented Sep 20, 2018

I sent #13111 because #13091 was the PR immediately before the problems started happening, and it says "[ci skip]". I don't know if it is really the cause yet.

@foolip
Copy link
Member Author

foolip commented Sep 20, 2018

#13113 by @jgraham will probably fix this.

If that's right, then #13091 was the cause. @jdm, did that PR bypass Travis, is that what "[ci skip]" means? If so, can you file an issue for Servo syncs to not bypass checks?

@jdm
Copy link
Contributor

jdm commented Sep 20, 2018

Yes, this is the perfect moment to pull the switch on our new system that will not bypass them.

@foolip
Copy link
Member Author

foolip commented Sep 20, 2018

@jdm, sweet, is that ready for action right now?

@jdm
Copy link
Contributor

jdm commented Sep 20, 2018

As far as I know, yes.

@foolip foolip assigned jdm and unassigned gsnedders Sep 20, 2018
@foolip foolip closed this as completed in 1f8fa9e Sep 20, 2018
@foolip
Copy link
Member Author

foolip commented Sep 20, 2018

Reopening until it's clear that new PRs aren't affected, and the listed affected ones recovered.

@foolip foolip reopened this Sep 20, 2018
@foolip
Copy link
Member Author

foolip commented Sep 20, 2018

#13106 did pass the Travis check, with one affected test, so the original problem is indeed fixed. I'll comment in all the listed PRs.

@foolip
Copy link
Member Author

foolip commented Sep 21, 2018

OK, it's clear now the problem is fixed. @jdm, I'll close this, if you need another issue to track "ci skip" please file one :)

@foolip foolip closed this as completed Sep 21, 2018
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Sep 24, 2018
…d., a=testonly

Automatic update from web-platform-testsHandle case where mozlog is not installed. (#13113)

In case that mozlog is present we need to ensure any lock it holds is
released in subprocesses. In case it's not present we don't need to do
anything, so don't fail if we can't import it.

Fixes web-platform-tests/wpt#13112.
--

wpt-commits: 1f8fa9e610d934490538812d6121a77baf646bfc
wpt-pr: 13113
xeonchen pushed a commit to xeonchen/gecko-cinnabar that referenced this issue Sep 25, 2018
…d., a=testonly

Automatic update from web-platform-testsHandle case where mozlog is not installed. (#13113)

In case that mozlog is present we need to ensure any lock it holds is
released in subprocesses. In case it's not present we don't need to do
anything, so don't fail if we can't import it.

Fixes web-platform-tests/wpt#13112.
--

wpt-commits: 1f8fa9e610d934490538812d6121a77baf646bfc
wpt-pr: 13113
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 3, 2019
…d., a=testonly

Automatic update from web-platform-testsHandle case where mozlog is not installed. (#13113)

In case that mozlog is present we need to ensure any lock it holds is
released in subprocesses. In case it's not present we don't need to do
anything, so don't fail if we can't import it.

Fixes web-platform-tests/wpt#13112.
--

wpt-commits: 1f8fa9e610d934490538812d6121a77baf646bfc
wpt-pr: 13113

UltraBlame original commit: ba995f8b27019322f6b01edc29c7d061145fba02
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 3, 2019
…d., a=testonly

Automatic update from web-platform-testsHandle case where mozlog is not installed. (#13113)

In case that mozlog is present we need to ensure any lock it holds is
released in subprocesses. In case it's not present we don't need to do
anything, so don't fail if we can't import it.

Fixes web-platform-tests/wpt#13112.
--

wpt-commits: 1f8fa9e610d934490538812d6121a77baf646bfc
wpt-pr: 13113

UltraBlame original commit: ba995f8b27019322f6b01edc29c7d061145fba02
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 3, 2019
…d., a=testonly

Automatic update from web-platform-testsHandle case where mozlog is not installed. (#13113)

In case that mozlog is present we need to ensure any lock it holds is
released in subprocesses. In case it's not present we don't need to do
anything, so don't fail if we can't import it.

Fixes web-platform-tests/wpt#13112.
--

wpt-commits: 1f8fa9e610d934490538812d6121a77baf646bfc
wpt-pr: 13113

UltraBlame original commit: ba995f8b27019322f6b01edc29c7d061145fba02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants