Skip to content
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

Update terraform-libvirt-provider #432

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rjmateus
Copy link
Member

@rjmateus rjmateus commented Dec 27, 2021

Follow up PR of

Do not merge until the above PR got merged.

This PR will update all jobs to use the latest version of terraform libvirt provider version.

@srbarrios
Copy link
Member

Fine, but it doesn't seems to include the build validation 4.3 tf, check if it needs some care too please.

@srbarrios srbarrios force-pushed the master branch 24 times, most recently from bd9faef to 816f63a Compare March 1, 2022 15:50
@nodeg
Copy link
Member

nodeg commented Jul 20, 2022

This PR needs a rebase, an update for libvirt 0.6.14 and the 4.3 files have to be adjusted, too as Oscar mentioned above.

@rjmateus rjmateus force-pushed the terraform_libvirt_0_6_12 branch 2 times, most recently from 436155d to 6cf27ca Compare July 22, 2022 11:37
@rjmateus
Copy link
Member Author

@nodeg @srbarrios this PR is ready to merge. If you agree I propose to merge it on monday, so we can have time to update all CI and have a closer look on them

@nodeg nodeg changed the title update terraform libvirt plugin to version 0.6.12 update terraform libvirt plugin to version 0.6.14 Jul 27, 2022
@ktsamis
Copy link
Member

ktsamis commented Jul 27, 2022

@rjmateus We discussed this in QE, let's wait for the submission of 4.3.1 to be done first (not released, just submitted), same with the uyuni submission and then we can merge this.

@srbarrios
Copy link
Member

Let's merge on Monday.

@nodeg nodeg changed the title update terraform libvirt plugin to version 0.6.14 Update terraform-libvirt-provider Dec 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants