-
-
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
traefik: ship webui #41378
Comments
Here's a workaround inspired by #65166 (for #42614) for the time being: { stdenv, fetchurl }:
let
version = "2.0.2";
sources = let
base = "https://github.com/containous/traefik/releases/download/v${version}";
in {
"x86_64-linux" = fetchurl {
url = "${base}/traefik_v${version}_linux_amd64.tar.gz";
sha256 = "f9adf514fc7ae0e90b88680213d44d59df429d8b68978b4a7ed4f9408fb0ad67";
};
};
in stdenv.mkDerivation rec {
pname = "traefik-bin";
inherit version;
src = sources."${stdenv.hostPlatform.system}" or (throw "unsupported system: ${stdenv.hostPlatform.system}");
sourceRoot = ".";
installPhase = ''
runHook preInstall
mkdir -p $bin/bin $out/bin
mv traefik $bin/bin
runHook postInstall
'';
meta = with stdenv.lib; {
homepage = https://traefik.io/;
description = "A modern reverse proxy";
platforms = [ "x86_64-linux" ];
license = licenses.mit;
maintainers = with maintainers; [ hamhut1066 vdemeester ];
};
outputs = [ "bin" "out" ];
} I'm not too eager to send a PR since I have no idea what I'm doing, but it seems to work with EDIT: This should work for v1.7 as well. |
Thank you for your contributions.
|
Building javascript frontends with Nix unfortunately isn't quite nice, and the tooling we have isn't too good either. I doubt we'll be able to solve this any time soon. @ljani, in the meantime traefik has been bumped to 2.x (in #76723) - would you be up to create a PR adding a Users that want it could then set |
@flokli: It seems I do not have enough time to play at this point, sorry. Also, I've moved to Debian on WSL2 for the time being. I should try that NixOS for WSL2 package first. |
Actually I have a working version of this, but it's not very maintainable: people would have to manually do the version bump for the frontend. Would anyone be interested to make a PR for that? |
@jokogr I'm not sure if I quite follow - why would |
@flokli I meant to say that I have already managed to package the frontend inside the Should I create a PR, so that you could have a look? |
@jokogr it probably depends on how much code and how maintainable it is. If we need to jump through a lot of hoops to update the package, maybe it's better to ship it without the webui, and provide the binary package for those who want the webui. |
Hi any updates on this? anyway to continue this work? |
@eljojo I think getting #41378 (comment) would be a good start :-) |
I marked this as stale due to inactivity. → More info |
Issue description
Copied over from #41208:
/cc @vdemeester @ryantm @xeji
The text was updated successfully, but these errors were encountered: