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 2.x] Temporarily hardcode chromedriver to 112.0.0 to enable all ftr tests #4037

Merged
merged 1 commit into from
May 16, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 5ea0cbe from #3976.

…3976)

The latest version of chromedriver is 112.0.1 which does not support
node 14. This PR hardcodes chromedriver to 112.0.0 temporarily. Pls
revert it once we bump to node 18.

Issue Resolved
#3975

Signed-off-by: ananzh <[email protected]>
(cherry picked from commit 5ea0cbe)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

# Conflicts:
#	CHANGELOG.md
@kavilla kavilla added the Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry label May 15, 2023
@kavilla
Copy link
Member

kavilla commented May 15, 2023

Test will fail because this will also need to be merged in: https://github.com/opensearch-project/OpenSearch-Dashboards/pull/4013/files.

But that PR is blocked by this PR not getting merged in.

@codecov
Copy link

codecov bot commented May 16, 2023

Codecov Report

Merging #4037 (56013df) into 2.x (62e7ea6) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              2.x    #4037      +/-   ##
==========================================
- Coverage   66.45%   66.45%   -0.01%     
==========================================
  Files        3229     3229              
  Lines       62045    62045              
  Branches     9597     9597              
==========================================
- Hits        41232    41230       -2     
- Misses      18513    18514       +1     
- Partials     2300     2301       +1     
Flag Coverage Δ
Linux 66.39% <ø> (-0.01%) ⬇️
Windows 66.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

@abbyhu2000 abbyhu2000 merged commit 9985bb2 into 2.x May 16, 2023
@github-actions github-actions bot deleted the backport/backport-3976-to-2.x branch May 16, 2023 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip-Changelog PRs that are too trivial to warrant a changelog or release notes entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants