Skip to content
This repository was archived by the owner on Nov 12, 2025. It is now read-only.

Set max packet size to 256KiB for SshjTool#405

Closed
grkvlt wants to merge 2 commits intoapache:masterfrom
grkvlt:fix/ssh-max-packet
Closed

Set max packet size to 256KiB for SshjTool#405
grkvlt wants to merge 2 commits intoapache:masterfrom
grkvlt:fix/ssh-max-packet

Conversation

@grkvlt
Copy link
Copy Markdown
Member

@grkvlt grkvlt commented Dec 18, 2014

Fix for BROOKLYN-106

@asfbot
Copy link
Copy Markdown

asfbot commented Dec 18, 2014

incubator-brooklyn-pull-requests #517 SUCCESS
This pull request looks good

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

would it make sense to have it as a configKey?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Could do, yes. One of the SSH_BLAH keys?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

yeah, something like SSH_MAX_PACKET_SIZE, wdyt?

@asfbot
Copy link
Copy Markdown

asfbot commented Dec 18, 2014

incubator-brooklyn-pull-requests #521 SUCCESS
This pull request looks good

@grkvlt
Copy link
Copy Markdown
Member Author

grkvlt commented Dec 18, 2014

I made the default 256KiB but this could be changed to 32KiB to match existing behaviour?

I think the buffer size will have an impact on the total heap used by Brooklyn, as it increases the required memory allocation per SSH session. If there are many entities, and thus many SSH connections, this could be an issue. We need to do some memory profiling runs before deciding on an appropriate setting for this value.

@grkvlt
Copy link
Copy Markdown
Member Author

grkvlt commented Dec 19, 2014

Since this doesn't seem to resolve the underlying issue, I'm closing this.

@grkvlt grkvlt closed this Dec 19, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants