Peter Šurda PeterSurda
  • Joined on 2020-04-27
PeterSurda approved Sysdeploy/idlers-agent#15 2024-06-27 10:47:23 +02:00
disk space calculation using /sys/block/

I'm merging, please address my comments in the next PR.

PeterSurda pushed to main at Sysdeploy/idlers-agent 2024-06-27 00:57:59 +02:00
8f79919659 Updated ecc logic, added RAM configured speed
PeterSurda merged pull request Sysdeploy/idlers-agent#17 2024-06-27 00:57:58 +02:00
Updated ecc logic, added RAM configured speed
PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-27 00:57:30 +02:00
Added note and refactor code

Yes.

PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-27 00:57:12 +02:00
Added note and refactor code

It misreports on some DDR4 systems and I haven't found conclusive clarification. I have one system for example which reports TotalWidth 72 and DataWidth 72. But it doesn't influence the amount of…

PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-27 00:55:12 +02:00
Added note and refactor code

Yes looks like it's ok.

PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-27 00:54:46 +02:00
Added note and refactor code

one:

Handle 0x0001, DMI type 1, 27 bytes
System Information
	Manufacturer: To Be Filled By O.E.M.
	Product Name: X570D4U
	Version: To Be Filled By O.E.M.
	Serial Number: To Be Filled By…
PeterSurda pushed to main at Sysdeploy/idlers-agent 2024-06-26 01:47:21 +02:00
61909258a0 Added RAM info in notes
c5474b1f3c refactor - remove server_id from create_note_data
f10d0b53b0 minor fixes
3940cbdbde refactor: encaps create/update logic in upsert
fa67103b20 Added chachis and processor info in note
Compare 6 commits »
PeterSurda merged pull request Sysdeploy/idlers-agent#16 2024-06-26 01:47:20 +02:00
Added note and refactor code
PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-26 01:46:57 +02:00
Added note and refactor code

I need to verify how it works if there are multiple processors. I have a couple of dual-socket systems, but they always have the same processor model.

PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-26 01:46:57 +02:00
Added note and refactor code

I was looking at some systems, and sometimes both TotalWidth and DataWidth is 72. I'm not really sure what it means. I vaguely remember reading that that's a bug in some DDR4 modules or motherboards. For now I would only check for TotalWidth and ignore DataWidth.

PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-26 01:46:57 +02:00
Added note and refactor code

There are actually two speeds reported, one is the specification of the module and the other is configured speed. What you could do, is to report them both separated with "@". E.g. "1866@1333MHz".

PeterSurda commented on pull request Sysdeploy/idlers-agent#16 2024-06-26 01:46:57 +02:00
Added note and refactor code

We should also report "baseboard", and "system", if present. Traditional servers seem to report "system" and/or "chassis" whereas custom build machines report only "baseboard" (i.e. motherboard), as there is no way for the motherboard to find out what kind of chassis it is mounted into.

PeterSurda approved Sysdeploy/idlers-agent#16 2024-06-26 01:46:57 +02:00
Added note and refactor code

I'll merge it because the comments I have can be done in next iteration.

PeterSurda commented on pull request Sysdeploy/idlers-agent#15 2024-06-26 01:23:49 +02:00
disk space calculation using /sys/block/

Also please watching out for needing to rebase PRs.

PeterSurda commented on pull request Sysdeploy/idlers-agent#15 2024-06-26 01:21:01 +02:00
disk space calculation using /sys/block/

We also need to check for /sys/block/*/device as I explained in the chat. The logic is supposed to be like this.

PeterSurda suggested changes for Sysdeploy/idlers-agent#15 2024-06-26 01:21:01 +02:00
disk space calculation using /sys/block/
PeterSurda commented on pull request Sysdeploy/idlers-agent#15 2024-06-26 01:16:12 +02:00
disk space calculation using /sys/block/

Well, this logging.info we could keep but I would prefer if we got rid of f-strings due to #9

PeterSurda commented on pull request Sysdeploy/idlers-agent#15 2024-06-26 01:14:08 +02:00
disk space calculation using /sys/block/

We shouldn't use /proc/diskstats at all because it doesn't contain the information we need. We could use /proc/partitions if /sys/block isn't available. But I don't think I have such a…