Skip to content

Prevent openQA from cleaning up CURRENT repositories #321

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 18, 2025
Merged

Prevent openQA from cleaning up CURRENT repositories #321

merged 1 commit into from
Sep 18, 2025

Conversation

Martchus
Copy link
Contributor

@Martchus Martchus added the do not merge Proposed changes that require further discussion label Sep 17, 2025
@andrii-suse
Copy link
Collaborator

Oh, before deployment we probably need to tweak local apparmor (then also update it in this repo for reference profiles/apparmor.d/opt.openqa-trigger-from-obs.script.rsync.sh)

@Martchus
Copy link
Contributor Author

Regarding AppArmor: It looks like we already use the double-asterisks like /var/lib/openqa/share/factory/repo/** rwl in all occurrences of repo in the profile in this repo and in /etc/apparmor.d/opt.openqa-trigger-from-obs.script.rsync.sh on o3. So I think we already cover granting permissions in subdirectories like repo/fixed.

@Martchus Martchus merged commit 5ea66e4 into os-autoinst:master Sep 18, 2025
5 checks passed
@Martchus Martchus removed the do not merge Proposed changes that require further discussion label Sep 23, 2025
@Martchus Martchus deleted the mv-current branch September 23, 2025 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@okurz okurz okurz approved these changes

@andrii-suse andrii-suse andrii-suse approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Martchus @andrii-suse @okurz