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

[cli-sessions] #99

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[cli-sessions] #99

wants to merge 5 commits into from

Conversation

i-davydenko
Copy link

@i-davydenko i-davydenko commented Dec 26, 2023

Add support for cli-sessions feature

What I did:
Added handlers for new SSH_SERVER and SERIAL_CONSOLE attributes.

Why I did it:

Give ability to:
configure limit for active login sessions.
configure ssh-server / serial console autologout timeout
configure sysrq-capabilities (enable / disable)

How I did it

Add new service that responsible for serial configuration;
Update existing flows for extended ssh-server configurations in hostcfgd;
Add YANG model to support new configuration.

How I verified it:

Configure, reconfigure, and reset all new parameters, check if applicapple parameters was updated in the system.
Added unittests.

Signed-off-by: Ivan Davydenko ivanda@nvidia.com

Copy link

linux-foundation-easycla bot commented Dec 26, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

 update hostcfgd: add serial-console and extend ssh-server modules,
 update existing tests,
 introduce serial-console test
@qiluo-msft qiluo-msft requested a review from liuh-80 May 15, 2024 17:02
@qiluo-msft
Copy link
Contributor

Could you refine a meaningful PR title?

self.render_conf_file()

# Read max_sessions config
def read_max_sessions_config(self, ssh_server_policies):
if ssh_server_policies is not None:
Copy link
Contributor

Choose a reason for hiding this comment

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

if not ssh_server_policies:

Copy link
Author

Choose a reason for hiding this comment

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

this will break logic :(

@liat-grozovik
Copy link
Collaborator

@i-davydenko please handle missing coverage

@i-davydenko
Copy link
Author

@i-davydenko please handle missing coverage

Done

@fastiuk fastiuk closed this May 27, 2024
@fastiuk fastiuk reopened this May 27, 2024
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

5 participants