Skip to content
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.

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

[8.0] Use copy+remove in PilotSync agent to avoid SELinux problems #7304

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

sfayer
Copy link
Member

@sfayer sfayer commented Nov 23, 2023

Hi,

We've been switching to using an external webserver to serve the pilot files as things were getting overloaded, but ran into SELinux problems. This small tweak changes the deploying of the pilot files from move to copy and delete; this causes the SELinux context of the files to get reset to the correct values for the destination.

Regards,
Simon

BEGINRELEASENOTES
*WorkloadManagement
FIX: Use copy+remove in PilotSync agent to avoid SELinux problems
ENDRELEASENOTES

@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Nov 23, 2023
@fstagni fstagni merged commit 0eafb3a into DIRACGrid:rel-v8r0 Nov 24, 2023
24 checks passed
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Nov 24, 2023
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Nov 24, 2023
@DIRACGridBot
Copy link

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/6978662719

Successful:

  • integration

@chaen
Copy link
Contributor

chaen commented Nov 24, 2023

maybe in the __init__.py of DIRAC we should have subprocess.run("setenforce permissive") ? 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alsoTargeting:integration Cherry pick this PR to integration after merge sweep:done All sweeping actions have been done for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants