Skip to content

Conversation

codgician
Copy link
Member

@codgician codgician commented Aug 12, 2025

Diff: open-webui/open-webui@refs/tags/v0.6.18...refs/tags/v0.6.22

Changelog: https://github.com/open-webui/open-webui/blob/v0.6.22/CHANGELOG.md

Also fixed the update script so it could get the correct npmDepsHash.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@74k1
Copy link
Contributor

74k1 commented Aug 12, 2025

Hi @codgician

Did you test if the issue from #430433 is happening here as well?

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Aug 12, 2025
@codgician
Copy link
Member Author

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 433087
Commit: b7588ce8c2fb7501d1097ebc9c81726b0ff62251


x86_64-linux

✅ 2 packages built:
  • open-webui
  • open-webui.dist

@codgician
Copy link
Member Author

codgician commented Aug 12, 2025

Hi @codgician

Did you test if the issue from #430433 is happening here as well?

Hi @74k1, thanks for bringing this to attention!

From my understanding above issue is caused by the open-webui module and not the open-webui package itself. I personally use PostgreSQL for my locally hosted Open-WebUI instance so I am not directly affected.

I'd happy to help looking into whether we can add this scenario into the tests of open-webui in a few days if available. Not sure if we could first focus on version bumping specifically for this PR?

@codgician codgician marked this pull request as ready for review August 12, 2025 13:07
@nix-owners nix-owners bot requested review from shivaraj-bh and drupol August 12, 2025 13:09
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages. labels Aug 12, 2025
@codgician
Copy link
Member Author

codgician commented Aug 12, 2025

P.S. GitHub API is experiencing issues at the moment and affecting Eval job: https://www.githubstatus.com

image

@priegger
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 433087
Commit: 9a10f0dc0ee01983429250a263d764da797a14ad


x86_64-linux

✅ 2 packages built:
  • open-webui
  • open-webui.dist

@priegger
Copy link
Contributor

I installed the package in my 25.05 system, ran into the #430433 auth issue and used the config from #430433 (comment), which made it work again (with losing my local state). I could start a chat with a local ollama model.

So I guess it's fine and, as far as I understand, all the issues are already present in 0.6.18 or worked around already or there are open tickets/prs to fix them and none of this has anything to do with this specific version bump. I just had the issues because I skipped some previous updates.

@74k1
Copy link
Contributor

74k1 commented Aug 12, 2025

Thanks @priegger

--

I have just tested this Package (from 0.6.18 to 0.6.22) along with the changes from #431395 and #432897 already configured and I did not suffer any data loss.

This update looks good to me. It'd be helpful if anyone could also take a look at my PR for any improvements #432897 (related to the nixos module for open-webui).

@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 2 This PR was reviewed and approved by two persons. and removed 12.approvals: 1 This PR was reviewed and approved by one person. labels Aug 12, 2025
@drupol drupol enabled auto-merge August 12, 2025 20:02
@drupol drupol disabled auto-merge August 12, 2025 20:02
@drupol drupol closed this Aug 12, 2025
@drupol drupol reopened this Aug 12, 2025
@drupol drupol merged commit e7a02c9 into NixOS:master Aug 12, 2025
48 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 12.approvals: 2 This PR was reviewed and approved by two persons. 12.approved-by: package-maintainer This PR was reviewed and approved by a maintainer listed in any of the changed packages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants