Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Old version of node_exporter (1.6.1) #16085

Closed
avelanarius opened this issue Nov 17, 2023 · 11 comments
Closed

Old version of node_exporter (1.6.1) #16085

avelanarius opened this issue Nov 17, 2023 · 11 comments

Comments

@avelanarius
Copy link
Member

The current version of node_exporter used in ScyllaDB images is 1.6.1 (as of 6bcf3ac). However, this version is flagged by security scanners with HIGH-severity CVE-2023-39325 (it uses old version of golang.org/x/net).

node_exporter should be updated to a later version 1.7.0 that fixed this problem.

avelanarius added a commit to avelanarius/scylla that referenced this issue Nov 17, 2023
Update node_exporter to 1.7.0. 

The previous version (1.6.1) was flagged by security scanners (such as 
Trivy) with HIGH-severity CVE-2023-39325. 1.7.0 release fixed that
problem.

Fixes scylladb#16085
@avelanarius avelanarius added Backport candidate backport/5.2 Issues that should be backported to 5.2 branch once they'll be fixed Requires-Backport-to-5.1 backport/5.4 Issues that should be backported to 5.4 branch once they'll be fixed labels Nov 17, 2023
denesb pushed a commit to denesb/scylla that referenced this issue Nov 17, 2023
Update node_exporter to 1.7.0.

The previous version (1.6.1) was flagged by security scanners (such as
Trivy) with HIGH-severity CVE-2023-39325. 1.7.0 release fixed that
problem.

[Botond: regenerate frozen toolchain]

Fixes scylladb#16085

Closes scylladb#16086
@avikivity
Copy link
Member

We could backport it, but I'd like a tiny run that proves that node_exporter starts correctly and exports metrics.

@mykaul
Copy link
Contributor

mykaul commented Nov 21, 2023

We could backport it, but I'd like a tiny run that proves that node_exporter starts correctly and exports metrics.

Will be part of next sprint - https://github.com/scylladb/qa-tasks/issues/1562

@roydahan
Copy link

@avikivity is it still waiting for verification before backports?
If so, I can try doing it now, we want to expedite it so we can release older releases without vulnerabilities

@mykaul
Copy link
Contributor

mykaul commented Nov 23, 2023

@avikivity is it still waiting for verification before backports? If so, I can try doing it now, we want to expedite it so we can release older releases without vulnerabilities

Please test.

@roydahan
Copy link

Tested with "5.5.0~dev-0.20231122.65e42e4166ce" and node_exporter works fine.

scyllaadm@longevity-parallel-topology-schema--db-node-dac3cbcd-1:/opt/scylladb$ /opt/scylladb/node_exporter/node_exporter --version
node_exporter, version 1.7.0 (branch: HEAD, revision: 7333465abf9efba81876303bb57e6fadb946041b)
  build user:       root@35918982f6d8
  build date:       20231112-23:53:35
  go version:       go1.21.4
  platform:         linux/amd64
  tags:             netgo osusergo static_build

Output Example:
curl http://localhost:9100/metrics

# HELP process_cpu_seconds_total Total user and system CPU time spent in seconds.
# TYPE process_cpu_seconds_total counter
process_cpu_seconds_total 7.93
# HELP process_max_fds Maximum number of open file descriptors.
# TYPE process_max_fds gauge
process_max_fds 524288
# HELP process_open_fds Number of open file descriptors.
# TYPE process_open_fds gauge
process_open_fds 10
# HELP process_resident_memory_bytes Resident memory size in bytes.
# TYPE process_resident_memory_bytes gauge
process_resident_memory_bytes 1.765376e+07
# HELP process_start_time_seconds Start time of the process since unix epoch in seconds.
# TYPE process_start_time_seconds gauge
process_start_time_seconds 1.70073227504e+09
# HELP process_virtual_memory_bytes Virtual memory size in bytes.
# TYPE process_virtual_memory_bytes gauge
process_virtual_memory_bytes 1.271443456e+09
# HELP process_virtual_memory_max_bytes Maximum amount of virtual memory available in bytes.
# TYPE process_virtual_memory_max_bytes gauge
process_virtual_memory_max_bytes 1.8446744073709552e+19
# HELP promhttp_metric_handler_errors_total Total number of internal errors encountered by the promhttp metric handler.
# TYPE promhttp_metric_handler_errors_total counter
promhttp_metric_handler_errors_total{cause="encoding"} 271
promhttp_metric_handler_errors_total{cause="gathering"} 0
# HELP promhttp_metric_handler_requests_in_flight Current number of scrapes being served.
# TYPE promhttp_metric_handler_requests_in_flight gauge
promhttp_metric_handler_requests_in_flight 1
# HELP promhttp_metric_handler_requests_total Total number of scrapes by HTTP status code.
# TYPE promhttp_metric_handler_requests_total counter
promhttp_metric_handler_requests_total{code="200"} 324
promhttp_metric_handler_requests_total{code="500"} 0
promhttp_metric_handler_requests_total{code="503"} 0

@roydahan
Copy link

Can we do the backports today?
We would like to have it on branch-2022.1 hopefully by Sunday.

@roydahan
Copy link

ping @avikivity

@avikivity
Copy link
Member

I'll regenerate now.

@avikivity
Copy link
Member

In 5.2 we have node_exporter 1.4.0. Is it vulnerable? Or was the vulnerability introduced later?

avikivity pushed a commit that referenced this issue Nov 27, 2023
Update node_exporter to 1.7.0.

The previous version (1.6.1) was flagged by security scanners (such as
Trivy) with HIGH-severity CVE-2023-39325. 1.7.0 release fixed that
problem.

[Botond: regenerate frozen toolchain]

Fixes #16085

Closes #16086

Closes #16090

(cherry picked from commit 321459e)

[avi: regenerate frozen toolchain]
avikivity pushed a commit that referenced this issue Nov 27, 2023
Update node_exporter to 1.7.0.

The previous version (1.6.1) was flagged by security scanners (such as
Trivy) with HIGH-severity CVE-2023-39325. 1.7.0 release fixed that
problem.

[Botond: regenerate frozen toolchain]

Fixes #16085

Closes #16086

Closes #16090

(cherry picked from commit 321459e)

[avi: regenerate frozen toolchain]
@roydahan
Copy link

In 5.2 we have node_exporter 1.4.0. Is it vulnerable? Or was the vulnerability introduced later?

@yaronkaikov ?

avikivity pushed a commit that referenced this issue Nov 27, 2023
Update node_exporter to 1.7.0.

The previous version (1.6.1) was flagged by security scanners (such as
Trivy) with HIGH-severity CVE-2023-39325. 1.7.0 release fixed that
problem.

[Botond: regenerate frozen toolchain]

Fixes #16085

Closes #16086

Closes #16090

(cherry picked from commit 321459e)

[avi: regenerate frozen toolchain]
[avi: update build script to work around https://users.rust-lang.org/t/cargo-uses-too-much-memory-being-run-in-qemu/76531]
@avikivity
Copy link
Member

Backported to 5.1, 5.2, 5.4.

@mykaul mykaul removed Backport candidate backport/5.2 Issues that should be backported to 5.2 branch once they'll be fixed backport/5.4 Issues that should be backported to 5.4 branch once they'll be fixed labels Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants