Skip to content

Conversation

@bunnichx
Copy link
Contributor

@bunnichx bunnichx commented May 20, 2025

This setting allows to disable TCP timestamps on host.
OS user can use 'sysctl net.ipv4.tcp_timestamps' command to
check the setting.

Merge Checklist

All boxes should be checked before merging the PR

  • The changes in the PR have been built and tested
  • [] cgmanifest file has been updated if required
  • Ready to merge

Description

Added kernel command in systemd service to disable TCP timestamps.

Any Newly Introduced Dependencies

No.

How Has This Been Tested?

First systemd rpm has built with modified systemd spec file.
With above systemd binaries, created iso and installed on system. verified as below.
image

@bunnichx bunnichx requested a review from a team as a code owner May 20, 2025 12:09
@bunnichx bunnichx force-pushed the disable-tcp-timestamp branch from 6a7da8c to f79caac Compare May 20, 2025 12:30
  This setting allows to disable TCP timestamps disclosure on host.
  OS user can use 'sysctl net.ipv4.tcp_timestamps' command to
  check the setting.

Signed-off-by: Unniche, BasavarajX <[email protected]>
@bunnichx bunnichx force-pushed the disable-tcp-timestamp branch from f79caac to 3ab92ff Compare May 20, 2025 16:36
@anujm1 anujm1 merged commit fa44382 into open-edge-platform:3.0-dev May 21, 2025
10 of 11 checks passed
cheeyanglee added a commit to cheeyanglee/edge-microvisor-toolkit that referenced this pull request May 22, 2025
fix mis-match conf file signature introduced in commit
"systemd: disable tcp timestamps. (open-edge-platform#110)"

Signed-off-by: Lee Chee Yang <[email protected]>
@cheeyanglee cheeyanglee mentioned this pull request May 22, 2025
3 tasks
anujm1 pushed a commit that referenced this pull request May 22, 2025
fix mis-match conf file signature introduced in commit
"systemd: disable tcp timestamps. (#110)"

Signed-off-by: Lee Chee Yang <[email protected]>
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.

3 participants