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

feat: freeze browser version to 112 #1408

Merged

Conversation

SuZhou-Joe
Copy link
Member

@SuZhou-Joe SuZhou-Joe commented Jun 13, 2024

Description

Github CI is using the latest chromium to run the Cypress test, which leads to forever hang in some test cases.
According to #4241 , we will freeze the Chromium version to 112, exactly same as what Jenkins CI uses.

Issues Resolved

[List any issues this PR will resolve]

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@@ -127,7 +139,11 @@ jobs:
- run: npx cypress cache path
- name: Cypress tests
uses: cypress-io/github-action@v2
env:
# remove success video to save space and memory
CYPRESS_DELETE_SUCCESS_VIDEOS: true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but we want to keep the video if the specs failed.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And even if we downgrade chromium version to 112, the security test will still fail. I am trying to reduce the memory our test cases are using.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Got it.

@@ -19,7 +19,7 @@
// ***********************************************************

// Import commands.js using ES2015 syntax:
import '../utils/commands';
import { currentBackendEndpoint } from '../utils/commands';
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a lint warning in this file that we import '../utils/commands' twice, so I combine the two import into one.

@SuZhou-Joe
Copy link
Member Author

SuZhou-Joe commented Jun 14, 2024

Convert to draft as it is still WIP and under verification.

@SuZhou-Joe SuZhou-Joe marked this pull request as draft June 14, 2024 02:28
@SuZhou-Joe SuZhou-Joe marked this pull request as ready for review June 15, 2024 15:47
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
@SuZhou-Joe SuZhou-Joe force-pushed the feature/freeze-chromium-version branch from fb436b2 to dc48b55 Compare June 16, 2024 08:09
@SuZhou-Joe SuZhou-Joe merged commit 03548a6 into opensearch-project:2.x Jun 18, 2024
54 of 58 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 18, 2024
* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add comment

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add comment

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

---------

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
(cherry picked from commit 03548a6)
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 18, 2024
* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add comment

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add comment

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

---------

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
(cherry picked from commit 03548a6)
SuZhou-Joe added a commit that referenced this pull request Jun 18, 2024
* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add comment

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add comment

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: freeze browser version

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: optimize test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

---------

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
(cherry picked from commit 03548a6)

Co-authored-by: SuZhou-Joe <suzhou@amazon.com>
peterzhuamazon added a commit that referenced this pull request Jun 19, 2024
peterzhuamazon added a commit that referenced this pull request Jun 19, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jun 19, 2024
This reverts commit 03548a6.

(cherry picked from commit deb070e)
peterzhuamazon added a commit that referenced this pull request Jun 19, 2024
This reverts commit 03548a6.

(cherry picked from commit deb070e)

Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants