quickwit: 0.8.0 -> 0.8.1, default config file

[nh2]

• Based on quickwit: Fix outdated Cargo.lock file #300659, merge that first
• https://github.com/quickwit-oss/quickwit/releases/tag/v0.8.1
• Also fix:
  • quickwit: Include and reference default config file #289000

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[nh2]

Building this still fails on my 128 GB RAM machine with:

thread 'actors::indexer::tests::test_indexer_triggers_commit_on_timeout' panicked at quickwit-indexing/src/actors/indexer.rs:1039:9:
assertion `left == right` failed
  left: MemoryLimit
 right: Timeout

See: #289000 (comment)

[happysalada]

I observed the same failure when building 0.8.0
Let's skip this test if that's ok with you.

[nh2]

Let's skip this test if that's ok with you.

Fine by me, I don't know yet how to disable specific Rust tests though. Feel free to push into my PR here in case you know it!

[happysalada]

I'm not close to a keyboard for long enough to push a commit, but here is an example.

  checkFlags = [
    # tries to make a network access
    "--skip=sinks::loki::tests::healthcheck_grafana_cloud"

so it shoulb be something like --skip='actors::indexer::tests::test_indexer_triggers_commit_on_timeout

I should have time this weekend, I'm just posting this in case you have time before.

[tcheronneau]

Hey guys I've been trying to build quickwit with the UI (and finally managed to do it).
I've also added the recommended build-features. Maybe it would worth to add that with this PR ?

[happysalada]

That would indeed be appreciated. Would you he okay with pushing to the branch ?

[nh2]

@tcheronneau I already left some comments on your PR #307808

[nh2]

Hey guys I've been trying to build quickwit with the UI (and finally managed to do it).
I've also added the recommended build-features

@tcheronneau Where are those changes, do you have a branch? Feel free to push them into this PR as additional commits for me to review, if you have permission to do that. Otherwise, let me know and I'll pull them.

[tcheronneau]

Hey guys I've been trying to build quickwit with the UI (and finally managed to do it).
I've also added the recommended build-features

@tcheronneau Where are those changes, do you have a branch? Feel free to push them into this PR as additional commits for me to review, if you have permission to do that. Otherwise, let me know and I'll pull them.

@nh2 I don't really know how to come in your PR (not used to github).
But you can find the change there : be916fc.

[tcheronneau]

I've prepared a PR but I cannot push to your repo.
In the end the build features does not work for now.
I'll try to debug I got a dev from quickwit I could maybe contact to help.
For now the working package with UI is there : https://gist.github.com/tcheronneau/8c0417705e8acc28e37289255dbb6b45

[jpds]

Sorry folks - I didn't see this one when I pushed up #317152.

[wegank]

Done in #317152.