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.

Sign up for GitHub

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

Jump to bottom

Browserstack integration breaks our CI #1742

Closed
mlewand opened this issue May 8, 2019 · 6 comments
Closed

Browserstack integration breaks our CI #1742

mlewand opened this issue May 8, 2019 · 6 comments
Assignees
@pomek
Labels
type:task This issue reports a chore (non-production change) and other types of "todos".
Milestone
iteration 25

Comments

@mlewand
Copy link
Contributor

mlewand commented May 8, 2019 •
edited
Loading

Is this a bug report or feature request? (choose one)

Other

📋 Steps to reproduce

Recently browserstack integration doesn't work great and causes our CI tests to fail rendering it unreliable.

One example is https://travis-ci.org/ckeditor/ckeditor5-utils/builds/529690643 (build was restarted later on, and is no longer broken)

Even though tests passes, there are plenty of errors reported.
@mlewand mlewand added the type:task This issue reports a chore (non-production change) and other types of "todos". label May 8, 2019
@mlewand mlewand added this to the iteration 24 milestone May 8, 2019
@pomek
Copy link
Member

pomek commented May 8, 2019

  • Did BS change anything on its side? (cc: @wwalc)
  • Can it be related to Travis itself?
  • Restarting the build usually helps but it does not seem to be a solution.

@wwalc
Copy link
Member

wwalc commented May 8, 2019

Did BS change anything on its side? (cc: @wwalc)

I have no idea.

Did you try changing captureTimeout to something more than 60000 ms?

@pomek
Copy link
Member

pomek commented May 9, 2019

I increased BrowserStack limits (ckeditor/ckeditor5-dev@master...t/ckeditor5/1742) and executed 10 builds:

  1. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530193037
  2. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530195877
  3. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530198261
  4. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530199802
  5. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530201883
  6. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530204677
  7. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530208272
  8. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530212249
  9. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/530214758
  10. https://travis-ci.org/ckeditor/ckeditor5-ui/builds/522126220

Everything is green.

The problem that the ticket describes is not too often and unfortunately, it's hard to debug. I guess we had no luck. We can stay with new limits but if the problem will occur again, (TBH) I have no idea what we can do more.

@mlewand mlewand modified the milestones: iteration 24, iteration 25 May 27, 2019
@pomek pomek mentioned this issue May 28, 2019
Closed
@pomek
Copy link
Member

pomek commented May 28, 2019

As specified here – #1778 (comment) – let's close this topic.

@pomek pomek closed this as completed May 28, 2019
@pomek pomek added the resolution:duplicate This issue is a duplicate of another issue and was merged into it. label May 28, 2019
@mlewand mlewand reopened this May 28, 2019
@mlewand
Copy link
Contributor Author

mlewand commented May 28, 2019 •
edited
Loading

Reopening this issue as #1778 was a dup of this issue, and we had some coments here already.

@mlewand mlewand added status:confirmed and removed resolution:duplicate This issue is a duplicate of another issue and was merged into it. labels May 28, 2019
@pomek
Copy link
Member

pomek commented Jun 6, 2019

Due to problems last time, we decided to disable integration with BrowserStack.

This was referenced Jun 6, 2019
Merged
Merged
Merged
Merged
Merged
ma2ciek added a commit to ckeditor/ckeditor5-react that referenced this issue Jun 6, 2019
@ma2ciek
Merge pull request #95 from ckeditor/t/ckeditor5/1742
2379fbf 
Other: Removed the BrowserStack integration from this repository. See ckeditor/ckeditor5#1742.
ma2ciek added a commit to ckeditor/ckeditor5-dev that referenced this issue Jun 6, 2019
@ma2ciek
Merge pull request #524 from ckeditor/t/ckeditor5/1742
a753590 
Other: Removed the BrowserStack integration from `ckeditor5-dev` repository. See ckeditor/ckeditor5#1742.
@pomek pomek self-assigned this Jun 7, 2019
@oleq oleq closed this as completed in 79b67e9 Jun 13, 2019
oleq added a commit to ckeditor/ckeditor5-autoformat that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
4ed446e 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-autosave that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
048b58b 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-alignment that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
33b02f1 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-adapter-ckfinder that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
151a052 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-editor-classic that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
6852559 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-editor-decoupled that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
e723a31 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-editor-inline that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
e06878d 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-essentials that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
f430ad9 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-enter that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
0891a1d 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-font that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
03c1c99 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-engine that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
e143f2f 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-heading that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
af6b967 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-highlight that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
bc616c8 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-image that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
2c588e3 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-link that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
5009117 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-list that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
aa86913 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-media-embed that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
3b1fbed 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-markdown-gfm that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
4314290 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-mention that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
ae4a251 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-paragraph that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
2648344 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-remove-format that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
7a96c2b 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-paste-from-office that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
e8da49c 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-table that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
215680e 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-typing that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
34f1dc5 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-theme-lark that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
e6c1bda 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-ui that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
1ea310d 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-undo that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
8ac683f 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-upload that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
d917b64 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-widget that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
79ec161 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-utils that referenced this issue Jun 13, 2019
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
b8041e5 
…tor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-inspector that referenced this issue Jun 13, 2019
@oleq
Merge pull request #51 from ckeditor/t/ckeditor5/1742
32c5613 
Tests: Removed the BrowserStack integration from this repository (see ckeditor/ckeditor5#1742).
oleq added a commit to ckeditor/ckeditor5-vue that referenced this issue Jun 13, 2019
@oleq
Merge pull request #68 from ckeditor/t/ckeditor5/1742
79d5e32 
Tests: Removed the BrowserStack integration from this repository (see ckeditor/ckeditor5#1742).
@pomek pomek mentioned this issue Jun 15, 2019
Closed
JDinABox pushed a commit to JDinABox/ckeditor5-build-markdown that referenced this issue Sep 6, 2021
@oleq
Tests: Removed BrowserStack integration in this repository (see ckedi…
7011ac5 
…tor/ckeditor5#1742).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pomek pomek

Labels
type:task This issue reports a chore (non-production change) and other types of "todos".
Projects
None yet
Milestone
iteration 25
Development

No branches or pull requests
3 participants
@wwalc @pomek @mlewand