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
change image references to reference the new organization #1200
Comments
Thank you so much for curating this list @ilithanos 💯 |
i'll start going through this list and fixing what i find. Will do it in multiple smaller PR's so reviewing and testing is easier. |
@chanwit When going over the helm chart how much rebranding to tofu controller is wanted? For now my thought is fixing the gcr urls and the helm repo urls and leaving it at that. But if you want every tf-controller string/reference to be changed to tofu-controller i'll do that while i'm at it. |
Hi @ilithanos agreed |
So far i've chosen to leave configuration parameter names containing the tf-controller and tf namings alone for backwards compatibility. As changing these will invalidate current configurations used by users. |
I haven't gone through the documentation again yet, to evaluate if we are some with that part. Tilt is still missing a few things. I have no experience in using tilt, but I'll see what I can come up with the following days. |
Tilt was very useful to us when debugging a complex controller + runner behaviour. |
Good to know, I'll have to try it out. It's just one of the tools I haven't run into yet. @kasper-leFevre could I get you to verify that the documentation references are all fixed? @akselleirv I've noticed you are digging into the Tilt setup, would you mind taking on the registry references etc while you are at it? Just trying to get an overview of what we still need to get done on this one so we can get it finished up. Thanks to everyone for helping out ❤️ |
@ilithanos I made the Tilt setup functionally work in https://github.com/flux-iac/tofu-controller/pull/1224/files. I can see that there are some references to tf-controller. I can change them to tofu-controller. |
ping @ilithanos cc other maintainers @akselleirv @yitsushi |
Hello @chanwit, I’m at KubeCon this week so a bit busy, but I’ll take a look at this next week if that is fine. |
@chanwit since i havn't heard back from the others i tagged, i'll go through it myself as soon as i can and update it. I'll leave the tilt part for @akselleirv |
Thank you @ilithanos 👍 |
@ilithanos all is good with the Tiltfile. Only thing left is to update the Helm chart references after it has been renamed from tf-controller to tofu-controller. |
There are still a few places where the name is wrong in the docs @chanwit i'll look into it when i'm done with the helm chart + ci stuff related to that. |
Update: I've updated and renamed helmchart to fit with rc.4. I think i saw a few places in the documentation that still needs fixing, but i havn't had the to go through it. |
Tilt file has been updated. (Thanks @akselleirv) Still need to find time to look through the documentation. @chanwit Since versions are tied to the tofu-controller versioning, are we able to actually release the new chart without releasing a new controller rc? |
Cutting a new RC would be fine for me. |
When the build pipeline fix is merged, the next logical step would be to update the toolings and deployments to pull images from this organization instead of the weaveworks organization.
Current places i've found so far:
The text was updated successfully, but these errors were encountered: