New and motivated project member
Hey all!
This is not an issue, but me reaching out to the rest of the project (such as it is). @wonderhoss was nice enough to add me to the project after I voiced my wish to contribute.
I am not here to step on anyone's toes, but I am motivated to push this project forward.
This is my preliminary plan:
- merge a number of auto-closed PRs (RBAC, k8s version, some small features) :heavy_check_mark:
- migrate from
deptogo mod(https://www.cockroachlabs.com/blog/dep-go-modules/) :heavy_check_mark: - update all dependencies
- disable the autoclose-on-stale :heavy_check_mark: (https://github.com/wave-k8s/wave/pull/140) and re-open all issues+PRs closed by it (https://drewdevault.com/2021/10/26/stalebot.html) – as long as they still seem relevant after the previous steps; label open Issues with
FeatureRequest,Bug, … :heavy_check_mark: - Set up some forum for discussing things, e.g. GitHub discussions
- Add some tests, especially end-to-end tests with different Kubernetes versions (https://github.com/wave-k8s/wave/pull/139) :heavy_check_mark:
I will start doing these things in the upcoming weeks if nobody complains first 😃 (see stepping on toes).
Cheers, Philipp
Appreciate it! We would be migrating off otherwise since current version does not play well with recent kubernetes versions anymore.
Hey @toelke - we're 'attempting' to do the migration from 'dep' to go.mod. This is our work so far:
- https://github.com/wave-k8s/wave/compare/master...mamccorm:wave:master
Full disclosure - i'm not a go developer so largely trial and error and piecing bits together. api and webhook build fine, but controller and core do not (yet).
Hey.
There is also https://github.com/wave-k8s/wave/pull/105 and https://github.com/wave-k8s/wave/pull/107.
I tested #107. The build works and creates a working binary. The tests fail, however, as the Makefile does a lot of go install / go get where the sematics have changed or the old versions are no longer available (I found that for go generate, for example).
Thanks @toelke, dropped a comment over in https://github.com/wave-k8s/wave/pull/107 with a possible suggestion to fix the failing CI tests.
Sorry, yes. It's in my queue. Expect a reaction until the weekend.
Hey @toelke, how's the above going? Does it look any closer? :)
I replied here: https://github.com/starkers/wave/pull/1#issuecomment-2025929876