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

Don't take into account node number when check license usage legality #1959

Merged
merged 1 commit into from
Mar 20, 2017

Conversation

dmytro-ndp
Copy link
Contributor

@dmytro-ndp dmytro-ndp commented Mar 20, 2017

Signed-off-by: Dmytro Nochevnov dnochevnov@codenvy.com

What does this PR do?

It removes redundant getting number of nodes from the swarm connector, to avoid throwing next exception in case of swarm container is not accessible:

HTTP Status 500 - org.eclipse.che.api.core.ServerException: Failed to check if Codenvy usage matches system License constraints.

Selenium testing result.

What issues does this PR fix or reference?

This request is linked with #1928.

Changelog

Don't take into account node number when check license usage legality

Release Notes

(there are no significant changes for release notes)

Docs PR

(there are no significant changes for Docs)

Signed-off-by: Dmytro Nochevnov <dnochevnov@codenvy.com>
@dmytro-ndp dmytro-ndp requested a review from tolusha March 20, 2017 10:10
@dmytro-ndp dmytro-ndp merged commit de7ee54 into master Mar 20, 2017
@dmytro-ndp dmytro-ndp deleted the 1928 branch March 20, 2017 10:17
@JamesDrummond JamesDrummond added this to the 5.6.0 milestone Mar 30, 2017
@JamesDrummond JamesDrummond mentioned this pull request Apr 2, 2017
8 tasks
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.

None yet

3 participants