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

CHE-274 - Improve idling implementation #5425

Closed
wants to merge 1 commit into from

Conversation

snjeza
Copy link
Contributor

@snjeza snjeza commented Jun 21, 2017

What does this PR do?

Enable workspace shutdown after a specified inactivity period.

What issues does this PR fix or reference?

https://issues.jboss.org/browse/CHE-274

Changelog

Added the che.machine.ws.agent.inactive.stop.timeout.ms=3600000 and stop.workspace.scheduler.period property

Release Notes

Per default, a workspace will be stopped after one hour of inactivity.

Docs PR

N/A

Signed-off-by: Snjezana Peco snjezana.peco@redhat.com

See #5386

@codenvy-ci
Copy link

Can one of the admins verify this patch?

@skabashnyuk skabashnyuk requested review from slemeur, a user, bmicklea and akorneta June 22, 2017 06:05
@skabashnyuk
Copy link
Contributor

skabashnyuk commented Jun 22, 2017

  1. Need to add docs, changes log, release notes section
  2. Remove these components from codenvy repo and reuse it from Eclipse Che.

@bmicklea
Copy link

@snjeza I've added the template for PR descriptions here with hidden HTML comments on what each section means. Please fill it in as well as you can and I will review.

@snjeza
Copy link
Contributor Author

snjeza commented Jun 22, 2017

@bmicklea I have updated the PR description.

@bmicklea
Copy link

bmicklea commented Jun 23, 2017

I think there's a typo in the changelog - it should be 3,600,000 not 360,000 right? And I noticed this is changed in che-properties, but is there an option added to che.env to allow users to change it?

Also, please add a label for kind/enhancement.

@sunix sunix added the kind/enhancement A feature request - must adhere to the feature request template. label Jun 23, 2017
* [2012] - [2017] Codenvy, S.A.
* All Rights Reserved.
*
* NOTICE: All information contained herein is, and remains
Copy link
Contributor

Choose a reason for hiding this comment

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

copyright seems to be wrong. it should be EPL

* is strictly forbidden unless prior written permission is obtained
* from Codenvy S.A..
*/
package org.eclipse.che.api.agent.server.activity;
Copy link
Contributor

Choose a reason for hiding this comment

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

in che, copyrights are EPL

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have updated the PR.

@benoitf
Copy link
Contributor

benoitf commented Jun 26, 2017

ci-build

@codenvy-ci
Copy link

Build # 2909 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2909/ to view the results.

@skabashnyuk
Copy link
Contributor

@snjeza we need to do appropriate changes in codenvy to reuse this code. please do not merge until we will be ready.

Signed-off-by: Snjezana Peco <snjezana.peco@redhat.com>
@benoitf
Copy link
Contributor

benoitf commented Jun 26, 2017

ci-build

@sunix sunix mentioned this pull request Jun 26, 2017
1 task
@codenvy-ci
Copy link

Build # 2920 - FAILED

Please check console output at https://ci.codenvycorp.com/job/che-pullrequests-build/2920/ to view the results.

@benoitf
Copy link
Contributor

benoitf commented Jun 27, 2017

@snjeza
Copy link
Contributor Author

snjeza commented Jun 27, 2017

@benoitf There is a newer build - https://ci.codenvycorp.com/job/che-pullrequests-build/2920
It has a test failure that I can't reproduce it locally.

@benoitf
Copy link
Contributor

benoitf commented Jun 27, 2017

let's restart the build on jenkins

@benoitf
Copy link
Contributor

benoitf commented Jun 27, 2017

ci-build

@codenvy-ci
Copy link

Copy link
Contributor

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

Will require changes in codenvy before merging

@sunix
Copy link
Contributor

sunix commented Jun 28, 2017

@benoitf

Will require changes in codenvy before merging

is someone working on this ?

@benoitf
Copy link
Contributor

benoitf commented Jun 28, 2017

@sunix : iI suppose it's @skabashnyuk team

@mkuznyetsov
Copy link
Contributor

Copied content from this PR into new one: #5512

@mkuznyetsov mkuznyetsov closed this Jul 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement A feature request - must adhere to the feature request template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants