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

helm: add env variable for session inactivity to server #718

Merged

Conversation

giuseppe-steduto
Copy link
Member

Adds an env variable that contains the maximum number of days for which an interactive session can stay inactive before being automaticallly closed.

Closes reanahub/reana-client#657

@codecov-commenter
Copy link

codecov-commenter commented Jun 9, 2023

Codecov Report

Merging #718 (78cb8b1) into master (0a0c4c6) will not change coverage.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #718   +/-   ##
=======================================
  Coverage   18.90%   18.90%           
=======================================
  Files          26       26           
  Lines        2195     2195           
=======================================
  Hits          415      415           
  Misses       1780     1780           

Copy link
Member

@mdonadoni mdonadoni left a comment

Choose a reason for hiding this comment

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

All good, but a rebase is needed!

CHANGES.rst Outdated
@@ -5,6 +5,7 @@ Version 0.9.1 (UNRELEASED)
--------------------------

- Administrators:
- Adds configuration environment variable ``reana_server.environment.REANA_INTERACTIVE_SESSION_MAX_INACTIVITY_PERIOD`` to set a limit in days for the maximum inactivity period of interactive sessions after which they will be closed.
Copy link
Member

Choose a reason for hiding this comment

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

I think this can be skipped, it is just an internal change so I feel like there is no need to add it to the CHANGES. In any case, CHANGES.rst are checked and fixed before any stable release.

@giuseppe-steduto giuseppe-steduto force-pushed the server-max-inactivity-time-env branch 2 times, most recently from e605356 to 78cb8b1 Compare July 4, 2023 08:00
Adds an env variable that contains the maximum number of days for which
an interactive session can stay inactive before being automatically
closed.

Closes reanahub/reana-client#657
@mdonadoni mdonadoni force-pushed the server-max-inactivity-time-env branch from 78cb8b1 to 6f86e14 Compare July 12, 2023 11:27
@mdonadoni mdonadoni merged commit 6f86e14 into reanahub:master Jul 12, 2023
11 checks passed
@mdonadoni
Copy link
Member

Looking good! I have rebased the PR and merged it!

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.

interactive session: notify user of auto-closure
3 participants