python-terraform icon indicating copy to clipboard operation
python-terraform copied to clipboard

WIP: support lastest terraform, v1.1.0

Open niejn opened this issue 3 years ago • 8 comments

I am using python-terraform in my projects, and I found out that python-terraform lib is not supporting latest terraform, such ass v1.1.0. So I want to contribute to add terraform v1.1.0 support

niejn avatar Jan 20 '22 08:01 niejn

We have already done the work on our fork here: https://gitlab.com/domaindrivenarchitecture/python-terraform

As we moved to gitlab (doe to nicer ci-support) a PullRequest will be slightly more work. But we will help to reunite the changes :-)

jerger avatar Jan 20 '22 17:01 jerger

Hey guys! Is there any extra help needed to be able to merge those changes? The support for the version 1.x would be pretty awsome !

Th0masL avatar Jul 09 '22 01:07 Th0masL

At https://gitlab.com/domaindrivenarchitecture/python-terraform tf > 1 is working. But we've done more changes than just the lift to 1.0. So extra help would be very apreciated :-)

We can do a mob-pr session to setup a PR - in case of interest you can contact me at [email protected]

jerger avatar Jul 12 '22 11:07 jerger

Sadly its not very convenient to deal with changes/PRs between github and gitlab.

My main question is: does this PR contains everything you guys have added to your gitlab repo ?

If the answer is no, I'm happy to make a new PR that includes more stuff, just let me know what you think is the best approach @jerger, thanks !

Th0masL avatar Jul 14 '22 07:07 Th0masL

No, we've not contributed to this PR - our solution would produce conflicts.

jerger avatar Jul 14 '22 07:07 jerger

Thanks for your answer @jerger! I've had a look at your changes and I've created a new branch based on the content of your gitlab repo, and I also tried to resolve most of the minor differences to try to make the reunion of the two repos as easy as posisble.

Here is the branch: https://github.com/Th0masL/python-terraform/tree/1.0.4-dev-dda

Is there more things we need to pair on before this would eventually get merged ?

Th0masL avatar Jul 15 '22 21:07 Th0masL

No let's create a PR as work in progress & discuss about our changes while seeing the diffs.

jerger avatar Jul 16 '22 10:07 jerger

https://github.com/beelit94/python-terraform/pull/123

Th0masL avatar Jul 17 '22 07:07 Th0masL