Skip to content

ENT-11988: Fixed bug where remote file copy always preserves source perms#5837

Merged
larsewi merged 1 commit into
cfengine:masterfrom
larsewi:preserve
Jul 24, 2025
Merged

ENT-11988: Fixed bug where remote file copy always preserves source perms#5837
larsewi merged 1 commit into
cfengine:masterfrom
larsewi:preserve

Conversation

@larsewi

@larsewi larsewi commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

Remote file copy with the 'copy_from' attribute now only preserves
source file permissions if the 'preserve' attribute in 'body copy_from'
is true. Otherwise it will use the permissions of the destination file
if it already exists and default permissions if it does not.

Ticket: ENT-11988
Changelog: Commit
Signed-off-by: Lars Erik Wik lars.erik.wik@northern.tech

Back ported to:

Remote file copy with the 'copy_from' attribute now only preserves
source file permissions if the 'preserve' attribute in 'body copy_from'
is true. Otherwise it will use the permissions of the destination file
if it already exists and default permissions if it does not.

Ticket: ENT-11988
Changelog: Commit
Signed-off-by: Lars Erik Wik <lars.erik.wik@northern.tech>
@olehermanse

Copy link
Copy Markdown
Member

@cf-bottom jenkins, please

@cfengine cfengine deleted a comment from cf-bottom Jul 16, 2025
@cfengine cfengine deleted a comment from cf-bottom Jul 16, 2025
@cfengine cfengine deleted a comment from cf-bottom Jul 16, 2025
@cf-bottom

Copy link
Copy Markdown

@craigcomstock

Copy link
Copy Markdown
Contributor

@cf-bottom jenkins please after fixing up the build host setup some more...

@cf-bottom

Copy link
Copy Markdown

@larsewi

larsewi commented Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

@cf-bottom Jenkins please :)

@cf-bottom

cf-bottom commented Jul 23, 2025

Copy link
Copy Markdown

@larsewi larsewi merged commit 69efbd8 into cfengine:master Jul 24, 2025
39 of 41 checks passed
@larsewi larsewi deleted the preserve branch July 25, 2025 10:54
@nickanderson nickanderson added the cherry-pick? Fixes which may need to be cherry-picked to LTS branches label Aug 15, 2025
@aleksandrychev

Copy link
Copy Markdown
Contributor

cherry-pick label removed, because I see that it cp-ed

@aleksandrychev aleksandrychev removed the cherry-pick? Fixes which may need to be cherry-picked to LTS branches label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants