On Fri, 22 May 2020 at 22:59, Julien Pivotto <roidelapl...@prometheus.io> wrote:
> 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? > If we've this sort of issue I think we should be looking to fix that prior to release, rather than putting out buggy code. -- Brian Brazil www.robustperception.io -- 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/CAHJKeLryQ9SWog82cpX_4g_MQhzyzRBtKmsYi4d9xjX85-AmaQ%40mail.gmail.com.
