glpi icon indicating copy to clipboard operation
glpi copied to clipboard

fix(inventory): check if link need to be updated are added (prevent d…

Open stonebuzz opened this issue 3 years ago • 1 comments

At each inventory, GLPI delete all links (and infocom too) between asset and device (item_devicexxxx)

Check if link need to be updated are added (prevent bulk deletion)

Best regards

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #12852

stonebuzz avatar Oct 03 '22 07:10 stonebuzz

@orthagh

Before this fix :

Computer:
	prepare:
		Execution time:      0.0003 seconds
		Memory usage:        5.69 Mio
		Memory usage (real): 10 Mio
		Memory peak:         8.99 Mio

	handle:
		Execution time:      2.12774 seconds
		Memory usage:        10.36 Mio
		Memory usage (real): 18 Mio
		Memory peak:         13.23 Mio

	full:
		Execution time:      3.46368 seconds
		Memory usage:        10.35 Mio
		Memory usage (real): 18 Mio
		Memory peak:         13.23 Mio

Inventoried assets: Computer #1

After :

Computer:
	prepare:
		Execution time:      0.00029 seconds
		Memory usage:        5.69 Mio
		Memory usage (real): 10 Mio
		Memory peak:         8.99 Mio

	handle:
		Execution time:      2.06414 seconds
		Memory usage:        10.4 Mio
		Memory usage (real): 18 Mio
		Memory peak:         13.28 Mio

	full:
		Execution time:      3.30758 seconds
		Memory usage:        10.39 Mio
		Memory usage (real): 18 Mio
		Memory peak:         13.28 Mio

Inventoried assets: Computer #1

stonebuzz avatar Oct 03 '22 08:10 stonebuzz

I close in favour of https://github.com/glpi-project/glpi/pull/12904

stonebuzz avatar Oct 07 '22 11:10 stonebuzz