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

Upgrade Terraform Mixing to 1.0.0-rc.1 #2677

Merged
merged 12 commits into from
Oct 2, 2022

Conversation

tamirkamara
Copy link
Collaborator

@tamirkamara tamirkamara commented Oct 2, 2022

Resolves #2659

What is being addressed

The terraform mixin in porter now includes mirroring of providers internally so we can remove the process we did in our dockerfile.tmpl in our templates.

How is this addressed

  • Resource processor image used the 1.0.0.rc.1 of the terraform mixin
  • Remove terraform mixin commands from all templates

@tamirkamara
Copy link
Collaborator Author

/test-extended

@github-actions
Copy link

github-actions bot commented Oct 2, 2022

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/3168109791 (with refid 0d4b011d)

(in response to this comment from @tamirkamara)

@github-actions
Copy link

github-actions bot commented Oct 2, 2022

Unit Test Results

0 tests   - 2   0 ✔️  - 2   0s ⏱️ - 1h 31m 5s
0 suites  - 1   0 💤 ±0 
0 files    - 1   0 ±0 

Results for commit fb947ea. ± Comparison against base commit 4d25270.

♻️ This comment has been updated with latest results.

@tamirkamara tamirkamara marked this pull request as ready for review October 2, 2022 12:00
@tamirkamara tamirkamara changed the title Upgrade Terraform Mixing to 1.0.0.rc1 Upgrade Terraform Mixing to 1.0.0-rc.1 Oct 2, 2022
@tamirkamara
Copy link
Collaborator Author

/test-force-approve
just a changelog update

@github-actions
Copy link

github-actions bot commented Oct 2, 2022

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit fb947ea)

(in response to this comment from @tamirkamara)

@tamirkamara tamirkamara enabled auto-merge (squash) October 2, 2022 12:15
@tamirkamara tamirkamara merged commit 1433311 into main Oct 2, 2022
@tamirkamara tamirkamara deleted the tamirkamara/2659-tf-mixin-upgrade branch October 2, 2022 12:29
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.

Terraform mixin version mismatch
2 participants