Project

General

Profile

Actions
Copy link

User story #4590

closed

User story #5108: Only copy "tools" that are required for each platform

Only copy "tools" that are required for each platform (implementation)

Added by Cédric Cabessa almost 11 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
2
Assignee:
Nicolas CHARLES
Category:
Techniques
Target version:
2.11.0~beta2
Pull Request:
https://github.com/Normation/rudder-t...
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Until now every files in rudder_dependencies are copied, for example .exe files are copied on linux node.
This can be a problem for low bandwith / low disk space devices.
This patch create a list for each platform in order to copy only useful classes.

Related issues 2 (0 open — 2 closed)

Related to Rudder - Bug #6348: Do not download file that are not for you in /var/rudder/toolsRejected2015-03-05Actions
Related to Rudder - User story #7142: Avoid copying unnecessary tools (*.exe *.msi *.bat *.rpm *.vbs *.dll on Linux Debian for example)ReleasedBenoît PECCATTE2015-08-28Actions
Actions
Copy link
 #1

Updated by Matthieu CERDA almost 11 years ago

  • Status changed from New to Pending technical review
  • Assignee set to Jonathan CLARKE
  • Priority changed from N/A to 2
  • % Done changed from 0 to 100

The PR looks cool, and I love the idea, it will make things much less bandwidth-greedy.

Actions
Copy link
 #2

Updated by Vincent MEMBRÉ almost 11 years ago

  • Target version changed from 2.8.4 to 133
Actions
Copy link
 #3

Updated by Vincent MEMBRÉ almost 11 years ago

  • Target version changed from 133 to 2.9.5
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ over 10 years ago

  • Target version changed from 2.9.5 to 2.9.6
Actions
Copy link
 #6

Updated by Jonathan CLARKE over 10 years ago

  • Target version changed from 2.9.6 to 2.9.7
Actions
Copy link
 #7

Updated by Jonathan CLARKE over 10 years ago

  • Tracker changed from Bug to 6
  • Subject changed from filter rudder_dependencies on classes to Only copy "tools" that are required for each platform (implementation)
  • Status changed from Pending technical review to In progress
  • Target version changed from 2.9.7 to 2.11.0~beta2

We've reviewed this and all agree it's going to be a great change! It will be included in 2.11.

However, I need to make a couple of changes first, to fix one error and make "linux" support more-widely available to all UNICES (!windows.!android). I will create a new Pull Request based on yours, Cédric.

Cédric, thank you very much for this highly valuable contribution!

Actions
Copy link
 #8

Updated by Jonathan CLARKE over 10 years ago

  • Parent task set to #5108
Actions
Copy link
 #9

Updated by Jonathan CLARKE over 10 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Jonathan CLARKE to Nicolas CHARLES
  • Pull Request changed from https://github.com/Normation/rudder-techniques/pull/316 to https://github.com/Normation/rudder-techniques/pull/421
Actions
Copy link
 #10

Updated by Cédric Cabessa over 10 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:95b866c9580273a1c51ee9a95b7014b1daa38d0b.

Actions
Copy link
 #11

Updated by Vincent MEMBRÉ over 10 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.0~beta2 (announcement , changelog), which were released today.

Actions
Copy link
 #12

Updated by Benoît PECCATTE almost 10 years ago

  • Tracker changed from 6 to User story
Actions
Copy link
 #13

Updated by Jonathan CLARKE over 9 years ago

  • Related to Bug #6348: Do not download file that are not for you in /var/rudder/tools added
Actions
Copy link
 #14

Updated by Jonathan CLARKE about 9 years ago

  • Related to User story #7142: Avoid copying unnecessary tools (*.exe *.msi *.bat *.rpm *.vbs *.dll on Linux Debian for example) added
Actions
Copy link

Also available in: Atom PDF