Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Darwin diskstats reports operation counts divided by block size as sectors

Abierto Apto para principiantes
  • ¿Algo no está bien?
#3,850 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
go, prometheus
Área
observability

Línea de trabajo

Empieza en collector/diskstats_darwin.go, en las dos expresiones enlazadas en el issue, y compáralas con las exportaciones existentes de contadores de bytes en ese archivo. El issue propone eliminar ambas familias de métricas de sectores de Darwin y dirigir a los consumidores a los contadores de bytes; confirma las pruebas de métricas o las referencias del proyecto para esas familias. La tarea estará terminada cuando se eliminen los contadores de sectores incorrectos y pasen las comprobaciones afectadas; los consumidores deben tener en cuenta el cambio de unidades.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

The Darwin collector exports node_disk_read_sectors_total and node_disk_written_sectors_total using these expressions in v1.12.1, still present on master at 638ec72c:

float64(stat.NumRead) / float64(stat.BlockSize)
float64(stat.NumWrite) / float64(stat.BlockSize)

iostat.DriveStats defines NumRead and NumWrite as operation counts and BlockSize as bytes. Its Darwin bridge maps these separately from BytesRead and BytesWritten; the IOKit reader reads operation counts, byte counts, and preferred block size from their respective keys. The exported values therefore have units of operations divided by bytes, rather than sectors. The same collector already exports node_disk_read_bytes_total from BytesRead and node_disk_written_bytes_total from BytesWritten (source).

Reproduce the current arithmetic with Python 3:

cases = [("read", 3, 16384, 4096), ("write", 1000, 8192000, 4096)]
for name, operations, byte_count, block_size in cases:
    print(f"{name}: actual={operations / block_size:.12g}, sectors_if_512_bytes={byte_count / 512:g}")

read: actual=0.000732421875, sectors_if_512_bytes=32
write: actual=0.244140625, sectors_if_512_bytes=16000

The 512-byte sector value is a comparison convention, not a macOS API contract. Validation: arithmetic reproduction only; native macOS collector tests were not run.

I propose removing the two Darwin sector families and directing consumers to the existing byte counters, following Linux PR #787. Queries, rules, and panels must change their units and thresholds because the byte counters are not numeric replacements for the sector values.

Lenguaje dominante
Go
Estrellas
13.9k
Forks
2.7k
Merge medio
2 d 13 h
PR fusionados (30 d)
14

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de prometheus/node_exporter

Todos los issues de prometheus/node_exporter

Issues similares

Más issues de Go

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.