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

[CI] Bump chromedriver to 102 #1811

Merged
merged 1 commit into from
Jul 7, 2022

Conversation

kavilla
Copy link
Member

@kavilla kavilla commented Jun 27, 2022

Description

GitHub env runs on Chrome 102. This is done to get the CI
working again on 1.x however there is a follow-up issue here:
#1746

However, there's an issue with Node existing on an a user that
the GitHub CI doesn't have permissions too.

Signed-off-by: Kawika Avilla [email protected]

Issues Resolved

n/a

Check List

  • New functionality includes testing.
    • All tests pass
      • yarn test:jest
      • yarn test:jest_integration
      • yarn test:ftr
  • New functionality has been documented.
  • Commits are signed per the DCO using --signoff

GitHub env runs on Chrome 102. This is done to get the CI
working again on 1.x however there is a follow-up issue here:
opensearch-project#1746

However, there's an issue with Node existing on an a user that
the GitHub CI doesn't have permissions too.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
@kavilla
Copy link
Member Author

kavilla commented Jun 29, 2022

Expected maps failure has a follow up.

@kavilla kavilla merged commit e27b40b into opensearch-project:1.x Jul 7, 2022
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 7, 2022
GitHub env runs on Chrome 102. This is done to get the CI
working again on 1.x however there is a follow-up issue here:
#1746

However, there's an issue with Node existing on an a user that
the GitHub CI doesn't have permissions too.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit e27b40b)
@kavilla kavilla deleted the avillk/1.x/bump_chrome branch July 7, 2022 20:46
Copy link
Contributor

@seanneumann seanneumann left a comment

Choose a reason for hiding this comment

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

LGTM

kavilla added a commit that referenced this pull request Jul 7, 2022
GitHub env runs on Chrome 102. This is done to get the CI
working again on 1.x however there is a follow-up issue here:
#1746

However, there's an issue with Node existing on an a user that
the GitHub CI doesn't have permissions too.

Issue:
n/a

Signed-off-by: Kawika Avilla <[email protected]>
(cherry picked from commit e27b40b)

Co-authored-by: Kawika Avilla <[email protected]>
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.

4 participants