-
Notifications
You must be signed in to change notification settings - Fork 1.1k
[windows] Publish fluxctl on chocolatey #1363
Comments
Not to step on anyone toes here, but I ran across this and decided to give it a go. The package is currently pending a review. https://chocolatey.org/packages/fluxctl/1.8.0 |
Nice work, @JimPruitt! Please let us know how things go! I have a couple of questions. Do you know more about this? What's the process for
|
Thanks @dholbach! Sorry if this is too verbose... I should explain how this works. The choco package I built downloads the latest build from Weavworks and then installs that binary with a 'shim' that allows the command to be added to the PATH environment variable. So what I uploaded to chocolatey wasn't so much the fluxctl.exe, but instead was a .nupkg file that has the logic on where to download the file, check it against a sha256 checksum and then 'shim' the downloaded executable so that it's available from the command line. Uploading Packages You're more than welcome to take what I've done here and host the code out of Weaveworks Github project if you wish. (My intention here was just to contribute to this really cool project!) I can do a PR for this if that helps? Downloading/Installing
Automating/Testing |
This is great stuff - thanks again @JimPruitt. |
It looks like https://chocolatey.org/packages/fluxctl/1.8.0 was approved - great job @JimPruitt! What do we want to do next? AFAICS there's:
I'm not sure if all of this makes sense and if we want to do this all as part of this issue. Please let me know what you think. |
Sorry for my delayed response. (was away from home over the holiday weekend) I'm in agreement with everything you had stated above. I can start by creating a PR for the above (although I'm not sure how to organize my code offering in the flux repository). I'll start with a top level folder and get feedback on the PR from there. |
Thanks Jim! Hope you had a great weekend. I'm not quite sure what to respond in terms of the final place where the build files should live, but I started a discussion about it here: https://weave-community.slack.com/archives/C4U5ATZ9S/p1543305536162900 |
Just bumping this old thread here. I've had to step away from this for a while to catch up on other efforts. I have updated the Chocolatey package that's published to Fluxctl V 1.9.0. (It's currently pending review) I'm still working on a way to automatically test this (using BoxStarter) such that my Github repo can be incorporated into the Weaveworks namespace. I'll keep working on this as time allows. |
Ran across this when checking if someone was planning to update the install doco. https://chocolatey.org/packages/fluxctl is a trusted package now. Can we update the doco to make it easier for windows users to install? |
Sure. Can somebody send a PR to https://github.com/fluxcd/flux/blob/master/docs/references/fluxctl.md with the relevant steps and links? |
No worries @dholbach - I'll take care of that today. |
I can see that https://chocolatey.org/packages/fluxctl/ lists the nearly-latest version of fluxctl and is being updated somehow, although it does not reflect the latest release from a few days ago. This is not a high priority to fix, as there are still zero changes in fluxctl 1.21.2 itself; I just want to understand the release process for chocolatey a bit and whether there is something I should be doing to ensure chocolatey gets updated as well? (The fluxctl binary in homebrew by comparison is apparently updated by CI, or by some other process that feeds from it.) |
Currently, the Chocolatey package itself is updated via a k8s job that is run manually. The job itself merely detects whether or not a new fluxctl release has been posted, then updates the Chocolatey package code to reference the new release, and finally submits a pull request to the Fluxctl Chocolatey Github project. The process of building the .nupkg file and testing it is still a manual process. At one point there was talk about automating this entire process as part of the fluxctl build; however, it was never seen through to completion. While, building the new chocolatey package is trivial enough, a build validation would need to occur test the installation of that package. Chocolatey has released a test environment that could be used to test the package locally (before submission to Chocolatey.org); however, the following discussion we had on the matter seemed to indicate that using Vagrant (as required by the test platform) might not be compatible with the existing build process for FluxCtl. I am certainly happy to reopen the discussion on how to incorporate the Chocolatey package update process (and test process) into the Fluxctl build and code-line (as it would make more sense to have this maintained by the project itself - as opposed to an independent maintainer). Alternately, I don't mind maintaining the package during the interim. |
That's fascinating. Thank you for explaining the situation. I don't have the bandwidth to pursue this myself at the moment though I'd be happy to review and merge a PR for it, but as the process is not fully automated for now (sounds like it is mostly automated) and as someone must remember to do it I'm going to keep this issue open to memorialize the state of affairs and this decision (at least until we can say it's finished.) |
I just noticed this: https://community.chocolatey.org/packages/fluxctl It's current and up to date, and the modern Flux v2 is also being maintained thanks to @JimPruitt, with ongoing discussion about the maintenance and upkeep at: (https://community.chocolatey.org/packages/flux) Thanks so much for this! |
As
fluxctl
builds on Windows (cf #1214), and chocolatey.org was mentioned multiple times, it'd be great if we could publish it there as well to make it more accessible.@geofflamrock indicated interest in taking a look. Anyone else?
The text was updated successfully, but these errors were encountered: