search

flux-iac / tofu-controller

A GitOps OpenTofu and Terraform controller for Flux
https://flux-iac.github.io/tofu-controller/
Apache License 2.0
1.29k stars 138 forks source link

`tfctl` is no longer managed as part of this repo, so it should be removed + docs should be updated #516

Closed Gowiem closed 11 months ago

Gowiem commented 1 year ago

~It would be great to be able to install tfctl via brew install tfctl. I'm sure others would appreciate other package manager installation methods as well.~

tfctl is now merged in the gitops CLI tool, but there are references in the code (e.g. https://github.com/weaveworks/tf-controller/tree/main/tfctl) and documentation (e.g. https://docs.gitops.weave.works/docs/terraform/tfctl/#installation) that mention otherwise. Those should be cleaned up to decrease confusion for new folks trying out the project.

Nalum commented 1 year ago

tfctl has been merged into the Weave GitOps CLI, see weaveworks/weave-gitops#3190 and https://docs.gitops.weave.works/docs/installation/weave-gitops/

Gowiem commented 1 year ago

@Nalum ah I didn't realize, thanks for pointing that out! There are docs that point to installing it through releases here: https://docs.gitops.weave.works/docs/terraform/tfctl/#installation

I'll update this ticket to be about updating docs.

Nalum commented 1 year ago

Ah good catch! I'll try tackling this as soon as I get time.

artem-nefedov commented 1 year ago

What is the weave gitops cli analog for tfctl reconcile command?

aristapimenta commented 1 year ago

What is the weave gitops cli analog for tfctl reconcile command?

yeah, I don't think this migration is complete yet. I don't see the functionality of tfctl install in weave gitops either

chanwit commented 1 year ago

No it's not 100% yet. We'll need to do more on this part. For the time being, please still feel free to continue with tfctl.

squaremo commented 11 months ago

tfctl stayed as its own thing, in the end.