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

Examples contain misleading version restrictions #207

Closed
OurFriendIrony opened this issue Jul 20, 2023 · 7 comments
Closed

Examples contain misleading version restrictions #207

OurFriendIrony opened this issue Jul 20, 2023 · 7 comments
Assignees

Comments

@OurFriendIrony
Copy link

Some of your examples reference the sts module terraform-redhat/rosa-sts/aws.
This module has a restriction to only support version 4.x.x of terraform, not >4.
Whilst that is not related to this provider directly, this projects examples show >= 4.x.x as being allowed.

I am currently using 5.x.x but cannot follow the examples due to the restriction in the sts module. I have raised an issue last week on that repo but have not met a response yet. terraform-redhat/terraform-aws-rosa-sts#23

@nirarg
Copy link
Member

nirarg commented Aug 6, 2023

Hi @OurFriendIrony ,
Thank you for the feedback
We work on improving the examples, targeting this as well

@nirarg nirarg self-assigned this Aug 6, 2023
@OurFriendIrony
Copy link
Author

Has any progress been made on this? I believe the module issue still hasn't received any responses highlighting any future changes

@nirarg
Copy link
Member

nirarg commented Sep 19, 2023

Hi @OurFriendIrony ,

We are working on new module which also is going to contain all examples with improvements.
This is the new repository opened for this purpose:
https://github.com/terraform-redhat/terraform-rhcs-rosa
We will update here when this new module will be published

@OurFriendIrony
Copy link
Author

Will this deprecate terraform-redhat/terraform-aws-rosa-sts#23 ?

@nirarg
Copy link
Member

nirarg commented Sep 20, 2023

Hi @OurFriendIrony
Yes, finally the old module will be deprecated

If you need it for now, I can create quick fix changes the AWS version to ">=4.x.x", and release special module tag for this
You will have to change the module source version manually in the examples you use
In case you want this change, what is the module version you use?

@OurFriendIrony
Copy link
Author

OurFriendIrony commented Sep 20, 2023 via email

@nirarg
Copy link
Member

nirarg commented Oct 19, 2023

This was fixed in rosa-sts module in version 0.0.14
Also new module is in work in progress already with this issue solved

Closing this issue

@nirarg nirarg closed this as completed Oct 19, 2023
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

No branches or pull requests

2 participants