Skip to content

Use /var/lib/agama/scripts to save scripts#3371

Merged
imobachgs merged 3 commits intoSLE-16from
bsc-1261787
Apr 10, 2026
Merged

Use /var/lib/agama/scripts to save scripts#3371
imobachgs merged 3 commits intoSLE-16from
bsc-1261787

Conversation

@imobachgs
Copy link
Copy Markdown
Contributor

@imobachgs imobachgs commented Apr 10, 2026

Problem

  • The scripts cannot be executed from /var/log/agama-installation on a hardened system.

Solution

Use /var/lib/agama/scripts.

Testing

  • Adapted the unit test.
  • Tested manually.

* The scripts cannot be executed from /var/log/agama-installation in a hardened system.
@imobachgs imobachgs marked this pull request as ready for review April 10, 2026 10:00
Copy link
Copy Markdown
Contributor

@joseivanlopez joseivanlopez left a comment

Choose a reason for hiding this comment

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

LGTM

@imobachgs imobachgs merged commit 1005e25 into SLE-16 Apr 10, 2026
16 checks passed
@imobachgs imobachgs deleted the bsc-1261787 branch April 10, 2026 10:36
Copy link
Copy Markdown
Contributor

@mvidner mvidner left a comment

Choose a reason for hiding this comment

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

@imobachgs Do we now more about what kind of "a hardened system" that is? If it is a common and reasonable method, we could add it to our tests

@imobachgs
Copy link
Copy Markdown
Contributor Author

No, to be honest I do not have more details (just that the /var/log mount point does not have exec permissions, which makes sense).

imobachgs added a commit that referenced this pull request Apr 10, 2026
## Problem

The scripts cannot be executed from /var/log/agama-installation on a
hardened system.

- [*bsc1261787*](https://bugzilla.suse.com/show_bug.cgi?id=1261787)
- #3371


## Solution

change /var/log/agama-installation to /var/lib/agama/


## Testing

- Adapted the unit test.
@imobachgs imobachgs mentioned this pull request Apr 14, 2026
imobachgs added a commit that referenced this pull request Apr 14, 2026
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