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

Disable selenium tests #19509

Merged
merged 1 commit into from
Apr 6, 2021
Merged

Disable selenium tests #19509

merged 1 commit into from
Apr 6, 2021

Conversation

skabashnyuk
Copy link
Contributor

@skabashnyuk skabashnyuk commented Apr 6, 2021 •
edited
Loading

What does this PR do?

Disable selenium tests

Screenshot/screencast of this PR

n/a

What issues does this PR fix or reference?

Maven 3.8.1 not able to build

ERROR] Failed to execute goal on project che-selenium-core: Could not resolve dependencies for project org.eclipse.che.selenium:che-selenium-core:jar:7.29.0-SNAPSHOT: Failed to collect dependencies at org.kohsuke:github-api:jar:1.90 -> com.infradna.tool:bridge-method-annotation:jar:1.17 -> org.jenkins-ci:annotation-indexer:jar:1.4: Failed to read artifact descriptor for org.jenkins-ci:annotation-indexer:jar:1.4: Could not transfer artifact org.jenkins-ci:jenkins:pom:1.26 from/to maven-default-http-blocker (http://0.0.0.0/): Blocked mirror for repositories: [repo.jenkins-ci.org (http://repo.jenkins-ci.org/public/, default, releases+snapshots)] -> [Help 1]

Reason https://maven.apache.org/docs/3.8.1/release-notes.html#how-to-fix-when-i-get-a-http-repository-blocked

Related to #19190

How to test this PR?

  • mvn clean install

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

@skabashnyuk
Disable selenium tests
e472dd9 
Signed-off-by: Sergii Kabashniuk <skabashniuk@redhat.com>
dmytro-ndp
dmytro-ndp approved these changes Apr 6, 2021
View reviewed changes
Copy link
Contributor

@dmytro-ndp dmytro-ndp left a comment

Choose a reason for hiding this comment

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

Looks good to me.
Related issue: #18376

@che-bot che-bot added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. labels Apr 6, 2021
@skabashnyuk skabashnyuk merged commit 3c5fe0c into eclipse-che:master Apr 6, 2021
@skabashnyuk skabashnyuk deleted the disableselenium branch April 6, 2021 12:13
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Apr 6, 2021
@che-bot che-bot added this to the 7.29 milestone Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SkorikSergey SkorikSergey SkorikSergey approved these changes

@dmytro-ndp dmytro-ndp dmytro-ndp approved these changes

@nickboldt nickboldt Awaiting requested review from nickboldt

@sparkoo sparkoo Awaiting requested review from sparkoo

@Katka92 Katka92 Awaiting requested review from Katka92

Assignees
No one assigned
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Milestone
7.29
Development

Successfully merging this pull request may close these issues.
4 participants
@skabashnyuk @dmytro-ndp @SkorikSergey @che-bot