-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
open-webui: init at 0.2.4 #316248
open-webui: init at 0.2.4 #316248
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi!
Thanks for submitting this!
I left some comments, could you also please pass the new files through nixfmt-rfc-style
?
v0.2.3 is out! |
@TanvirOnGH There is a problem after To be specific, they are fetching some dependencies in build-time: https://github.com/open-webui/open-webui/blob/be08dbf00d9ed894602f92955355eb2aa8374ec5/package.json#L7C13-L7C34 |
@TanvirOnGH I have disabled Now the package inits at |
I see that you've updated to 0.2.0, does that mean that the aforementioned issue has been dealt with? If yes, would it be possible to update to the very latest version then? |
I have disabled the feature that’s causing the issue for the time-being: 39227d2
will do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there,
Upgrade to latest version and squash the relevant some commits together now so that it make sense?
Can you please squash the relevant commits together? I think in the end there should be 2 commits, 1 for introducing the pkg |
Done! |
Nice! I'm wondering if adding something similar to nixpkgs/pkgs/servers/honk/default.nix Line 53 in 3e1b36a
open-webui pkg would be a good idea. WDYT ?
|
Thank you @shivaraj-bh ! |
I quickly tested it locally and I noticed 2 things:
|
I was about to point this out as well. I just noticed the log while running the VM test. |
Doesn't seem to be working
/var/lib/open-webui/cache is not a read only directory(i can manually create files in that dir) |
First off -- thank you all for the work on this! I was pleasantly surprised to see it in nixpkgs and it saves me from having to add another docker container. I ended up running into the same issue as @Celibistrial . Per @drupol 's comment above, I got it up and running by adding a few more environment variables. I don't think these locations can be arbitrarily created and used due to service permissions, so I kept them to the runtime directory (this may not the ideal place, I was just focused on getting up and running)
|
I also had the issue and planning to fix the issue, here it is: #318099 |
ported from: https://github.com/shivaraj-bh/ollama-flake/blob/main/services/open-webui.nix This was also recently upstreamed to nixpkgs: NixOS/nixpkgs#316248
@natsukium Thanks for the review! I will address them in a different PR and cross-reference it here. |
For more info, see: - Disable hatch custom hook: NixOS#316248 (comment) - why `makeWrapperArgs`?: NixOS#316248 (comment) - `pythonRelaxDepsHook` is a nativeBuildInput: NixOS#316248 (comment)
Please backport this to stable, i wanna use it Thanks <3 |
ported from: https://github.com/shivaraj-bh/ollama-flake/blob/main/services/open-webui.nix This was also recently upstreamed to [nixpkgs](https://github.com/NixOS/nixpkgs/tree/master): NixOS/nixpkgs#316248 --------- Co-authored-by: Pol Dellaiera <pol.dellaiera@protonmail.com> Co-authored-by: Sridhar Ratnakumar <3998+srid@users.noreply.github.com>
ported from: https://github.com/shivaraj-bh/ollama-flake/blob/main/services/open-webui.nix This was also recently upstreamed to [nixpkgs](https://github.com/NixOS/nixpkgs/tree/master): NixOS/nixpkgs#316248 --------- Co-authored-by: Pol Dellaiera <pol.dellaiera@protonmail.com> Co-authored-by: Sridhar Ratnakumar <3998+srid@users.noreply.github.com>
Successfully created backport PR for |
For more info, see: - Disable hatch custom hook: #316248 (comment) - why `makeWrapperArgs`?: #316248 (comment) - `pythonRelaxDepsHook` is a nativeBuildInput: #316248 (comment) (cherry picked from commit 75d7ee7)
Description of changes
Resolves #309567
open-webui package
open-webui service
open-webui service with ollama
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.