Skip to content

test: Update module used for testing init's -from-module feature, so that the test can be run on darwin_arm64 platforms#38015

Merged
SarahFrench merged 1 commit intomainfrom
update-e2e-init-from-module
Dec 17, 2025
Merged

test: Update module used for testing init's -from-module feature, so that the test can be run on darwin_arm64 platforms#38015
SarahFrench merged 1 commit intomainfrom
update-e2e-init-from-module

Conversation

@SarahFrench
Copy link
Member

@SarahFrench SarahFrench commented Dec 17, 2025

If you run TestInit_fromModule on an a machine using an Apple chip like M1 then the test will fail like this:

    init_test.go:325: unexpected error: exit status 1
    init_test.go:330: unexpected stderr output:
        ╷
        │ Error: Incompatible provider version
        │ 
        │ Provider registry.terraform.io/hashicorp/template v2.2.0 does not have a
        │ package available for your current platform, darwin_arm64.
        │ 
        │ Provider releases are separate from Terraform CLI releases, so not all
        │ providers are available for all platforms. Other versions of this provider
        │ may have different platforms supported.
        ╵
--- FAIL: TestInit_fromModule (11.28s)

This PR updates the test to use another HashiCorp-owned module that usable on Macbooks etc.

The test checks that a module can be downloaded when initialising a working directory and it doesn't matter which specific module is in use (as long as it's one we own).

Target Release

N/A

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

… so that the test can be run on darwin_arm64 platforms.
@SarahFrench SarahFrench added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Dec 17, 2025
Copy link
Contributor

@mildwonkey mildwonkey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works on my machine!!

@SarahFrench SarahFrench marked this pull request as ready for review December 17, 2025 16:06
@SarahFrench SarahFrench requested a review from a team as a code owner December 17, 2025 16:06
@SarahFrench SarahFrench enabled auto-merge (squash) December 17, 2025 16:07
@SarahFrench SarahFrench merged commit 33d54b9 into main Dec 17, 2025
22 of 23 checks passed
@SarahFrench SarahFrench deleted the update-e2e-init-from-module branch December 17, 2025 16:12
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants