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

Backport 4K x 4K windows to 1.3.x #111

Open
wants to merge 4 commits into
base: 1.3.x
Choose a base branch
from

Conversation

Rotonen
Copy link
Contributor

@Rotonen Rotonen commented May 15, 2019

Backport of #110

@mister-roboto
Copy link

@Rotonen thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

@Rotonen
Copy link
Contributor Author

Rotonen commented May 15, 2019

@jenkins-plone-org please run jobs

@Rotonen
Copy link
Contributor Author

Rotonen commented May 15, 2019

I'm currently not figuring out what is different about the Travis build on this branch vs. the master. The relevant files in this repository do not seem to have changed and the issue is a Java stack trace at test setup, where it tries to connect to some (probably Windows based) remote browser.

Could it be something like the Travis-side credentials / config of Saucelabs for this branch?

@datakurre
Copy link
Member

This project has Travis setup for running Selenium tests with Sauce Labs as an example. But there is no other real reason for that. Maybe it is time to tear that down? By removing ROBOT_REMOTE_URL and the build matrix and sauce add-on from travis configuration?

@Rotonen
Copy link
Contributor Author

Rotonen commented May 16, 2019

Would that break the testing on IE we have going on? Astonishingly that's the only thing green on Travis currently.

@datakurre
Copy link
Member

Yes it would 🙈

@Rotonen
Copy link
Contributor Author

Rotonen commented May 16, 2019

I'd rather figure out what's up and fix it than throw that away and I'm not in a particular hurry either.

@Rotonen
Copy link
Contributor Author

Rotonen commented May 21, 2019

I've now also cherry-picked 2c6ad4b onto this branch, which has rejuvenated the main Travis builds, but now the IE builds get stuck and time out.

@Rotonen
Copy link
Contributor Author

Rotonen commented May 21, 2019

Upon inspecting the history of the IE builds, they're just flaky. So they're actually fine as they now are.

@jenkins-plone-org please run jobs

@Rotonen Rotonen requested review from datakurre and tisto May 21, 2019 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants