Dear developers,
Do you think we could hide the head chunk PR under a feature flag for
now?
I observe that to run some of the test it creates in my tmpfs
directories of 16GB, then tests just fail.
As far as I know Prometheus should only require Posix compliant file
systems and to the best of my knowledge, tmpfs is Posix compliant.
$ go test -run TestSubquerySelector -v
=== RUN TestSubquerySelector
TestSubquerySelector: test.go:536: closing test storage: sync
/tmp/test_storage431942557/chunks_head/000028: file already closed
--- FAIL: TestSubquerySelector (1.23s)
FAIL
exit status 1
FAIL github.com/prometheus/prometheus/promql 1.237s
Chaning TMPDIR to ext4 make the test work again.
$ du -hs
/tmp/test_storage431942557/*
14G /tmp/test_storage431942557/chunks_head
0 /tmp/test_storage431942557/lock
128K /tmp/test_storage431942557/wal
I think that overall that change might have many other blind spots, so I
am wondering if that would be possible to somehow hide it under a
feature flag, disabled by default?
--
Julien Pivotto
@roidelapluie
--
You received this message because you are subscribed to the Google Groups
"Prometheus Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to prometheus-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/prometheus-developers/20200522215908.GA459437%40oxygen.
