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 over 10 years ago. Updated over 9 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 over 10 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É over 10 years ago

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

Updated by Vincent MEMBRÉ over 10 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 over 9 years ago

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

Updated by Jonathan CLARKE about 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 almost 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