Skip to content

Service hardenings #1436

Merged
brianmcgillion merged 2 commits intotiiuae:mainfrom
enesoztrk:fix/battery
Oct 1, 2025
Merged

Service hardenings #1436
brianmcgillion merged 2 commits intotiiuae:mainfrom
enesoztrk:fix/battery

Conversation

@enesoztrk
Copy link
Copy Markdown
Contributor

@enesoztrk enesoztrk commented Sep 29, 2025 •
edited
Loading

Description of Changes

Service hardenings to address the battery draining issue caused by restoring the suspend state, while also enhancing security.

Type of Change

  • New Feature
  • Bug Fix
  • Improvement / Refactor

Related Issues / Tickets

tiiuae/ghafpkgs#87 must be merged first

Checklist

  • Clear summary in PR description
  • Detailed and meaningful commit message(s)
  • Commits are logically organized and squashed if appropriate
  • Contribution guidelines followed
  • Ghaf documentation updated with the commit - https://tiiuae.github.io/ghaf/
  • Author has run make-checks and it passes
  • All automatic GitHub Action checks pass - see actions
  • Author has added reviewers and removed PR draft status

Testing Instructions

Applicable Targets

  • Orin AGX aarch64
  • Orin NX aarch64
  • Lenovo X1 x86_64
  • Dell Latitude x86_64
  • System 76 x86_64

Installation Method

  • Requires full re-installation
  • Can be updated with nixos-rebuild ... switch
  • Other:

Test Steps To Verify:

  1. Follow the test steps in SSRCSP-7293
  2. Chromecast should still be working

@enesoztrk enesoztrk added the Needs Testing CI Team to pre-verify label Sep 29, 2025
@milva-unikie
Copy link
Copy Markdown

milva-unikie commented Sep 30, 2025

Tested on Lenovo-X1 (new installation)

  • Battery issue has been fixed
  • Chromecast works

@milva-unikie milva-unikie added Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon and removed Needs Testing CI Team to pre-verify labels Sep 30, 2025
brianmcgillion
brianmcgillion reviewed Sep 30, 2025
View reviewed changes
@enesoztrk enesoztrk marked this pull request as ready for review September 30, 2025 08:14
@enesoztrk
fix: nw-packet-forwarder service improvements
886ed02 
* security hardening
* cli arg changes
* service bind to nic devices

Signed-off-by: Enes Öztürk <enes.ozturk@unikie.com>
@enesoztrk
bump: ghafpkgs/nw-packet-forwarder
f6ddf00 
Signed-off-by: Enes Öztürk <enes.ozturk@unikie.com>
@brianmcgillion brianmcgillion merged commit 5f2fdd2 into tiiuae:main Oct 1, 2025
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brianmcgillion brianmcgillion brianmcgillion approved these changes

@vunnyso vunnyso vunnyso approved these changes

Assignees

No one assigned

Labels

Tested on Lenovo X1 Carbon This PR has been tested on Lenovo X1 Carbon

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@enesoztrk @milva-unikie @brianmcgillion @vunnyso