User story #4590: Only copy "tools" that are required for each platform (implementation) - Rudder - Issue Tracker

Custom queries

8.1 beta
Bugs triage
My TR
Need technical review
Open bugs (all)
Open bugs (UI - UX)
Open issues 8.0
Open UX defects (all)
Opened by users

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

Status:

Released

Priority:

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/tools	Rejected		2015-03-05			Actions
	Related to Rudder - User story #7142: Avoid copying unnecessary tools (.exe .msi .bat .rpm .vbs .dll on Linux Debian for example)	Released	Benoît PECCATTE	2015-08-28			Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Matthieu CERDA about 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

Updated by Vincent MEMBRÉ about 10 years ago

Target version changed from 2.8.4 to 133

Actions

Copy link

Updated by Vincent MEMBRÉ about 10 years ago

Target version changed from 133 to 2.9.5

Actions

Copy link

Updated by Cédric Cabessa about 10 years ago

updated version for 2.10 https://github.com/Normation/rudder-techniques/pull/338

Actions

Copy link

Updated by Vincent MEMBRÉ almost 10 years ago

Target version changed from 2.9.5 to 2.9.6

Actions

Copy link

Updated by Jonathan CLARKE almost 10 years ago

Target version changed from 2.9.6 to 2.9.7

Actions

Copy link

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

Updated by Jonathan CLARKE almost 10 years ago

Parent task set to #5108

Actions

Copy link

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