Project

General

Profile

Bug #13887

Option description for "File download (Rudder server)" technique is misleading

Added by François ARMAND 22 days ago. Updated 17 days ago.

Status:
Released
Priority:
N/A
Category:
Techniques
Target version:
Severity:
User visibility:
Effort required:
Priority:
0

Description

'File download (Rudder server)' technique lets you copy things from `/var/rudder/configuration-repository/shared-files/` to somewhere else.

You have three option to choose what to copy:

- Only this file
- The whole content of the folder
- The folder, its content, and all the subfolders

What I undestand, if I have:

.../configuration-repository/shared-files/
└── dir
    ├── dir2
    │   └── foo.txt
    └── hollymolly.txt

And that I set "dir" in source path, "/tmp" in destination path, is that the content will be:

- Only this file

/tmp/
└── dir/

But it actually does nothing.

- The whole content of the folder

/tmp/
├── dir2 (empty)
└── hollymolly.txt

And it's that!

- The folder, its content, and all the subfolders

/tmp/
└── dir
    ├── dir2
    │   └── foo.txt
    └── hollymolly.txt

But it actually skips "dir" (only content is copied)

Moreover, folder is used a couple of time, when `directory` is used in (many more) other places.

Change into:

- Only this file (or nothing for a directory)
- The direct content of the directory (one level)
- The whole content of the directory (recursively)

(and: Apply the sticky bit to the whole folder? (caution, use with care) => Apply the sticky bit to the whole directory? (caution, use with care)

(even so, that does not seems very consistent. Why the sub-directory is created in second case if skipped in the first? But it can be ok)


Subtasks

Bug #13888: Option description for "dowload for shared files" is mileading and mixed use of folder/directory (rudder 4.3)RejectedFrançois ARMAND

Associated revisions

Revision 2bf9f07b (diff)
Added by François ARMAND 22 days ago

Fixes #13887: Option descrition for \"dowload for shared files\" is mileading and mixed use of folder/directory

History

#1 Updated by François ARMAND 22 days ago

  • Description updated (diff)

#2 Updated by François ARMAND 22 days ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND

#3 Updated by François ARMAND 22 days ago

  • Description updated (diff)

#4 Updated by François ARMAND 22 days ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Alexis MOUSSET
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1374

#5 Updated by François ARMAND 22 days ago

  • Subject changed from Option descrition for "dowload for shared files" is mileading and mixed use of folder/directory to Option description for "dowload for shared files" is mileading and mixed use of folder/directory

#6 Updated by Normation Quality Assistant 22 days ago

  • Assignee changed from Alexis MOUSSET to François ARMAND

#7 Updated by François ARMAND 22 days ago

  • Status changed from Pending technical review to Pending release

#8 Updated by Vincent MEMBRÉ 18 days ago

  • Subject changed from Option description for "dowload for shared files" is mileading and mixed use of folder/directory to Option description for "File download (Rudder server)" technique is misleading

#9 Updated by Vincent MEMBRÉ 17 days ago

  • Status changed from Pending release to Released
This bug has been fixed in Rudder 4.1.17, 4.3.7 and 5.0.3 which were released today.
Changelog
Changelog
Changelog

Also available in: Atom PDF