alibaba/sentinel-golang

[BUG] getProcessCpuStat return value (in percentage) breaks compatibility

Aperta

#435 aperta il 10 nov 2021

Vedi su GitHubGuida contributor
 (0 commenti) (1 reazione) (0 assegnatari)Go (453 fork)github user discovery
area/metricsgood first issuekind/bug

Metriche repository

Star
 (2957 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Issue Description

Type: bug report

Describe what happened

getProcessCpuStat return value (in percentage) breaks compatibility. In previous implementation, the CPU usage representation is the value between [0.0, 1.0]. Since 1.0.x, the CPU usage from getProcessCpuStat was changed to percentage value (i.e. 0-100), which breaks compatibility and affects system adaptive rules.

Describe what you expected to happen

getProcessCpuStat keeps compatibility and unifys the CPU usage value to [0.0, 1.0]

Tell us your environment

Sentinel Go v1.0.3

Guida contributor

Direzione di ricerca
Confronta l'implementazione di getProcessCpuStat tra le versioni per identificare la modifica che ha trasformato il valore restituito da frazione a percentuale. Ripristina il formato della frazione o aggiungi un'opzione configurabile.
Tech stack
go
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GoGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65