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

bump aws provider version to fix conflict #11

Open
wants to merge 3 commits into
base: release/v0.1.0
Choose a base branch
from

Conversation

pnadolny13
Copy link
Contributor

@pnadolny13 pnadolny13 commented Oct 12, 2022

@kgpayne @aaronsteers I can't init the module because of conflicting version constraints:

╷
│ Error: Failed to query available provider packages
│ 
│ Could not retrieve the list of available versions for provider hashicorp/aws: locked provider registry.terraform.io/hashicorp/aws 3.73.0 does not match configured
│ version constraint >= 2.49.0, >= 3.0.0, >= 3.1.0, >= 3.13.0, >= 3.40.0, >= 3.56.0, >= 3.59.0, 3.65.0, >= 3.73.0, < 4.0.0; must use terraform init -upgrade to allow
│ selection of new versions

The squared module references the v0.1.0 release and main has a bunch of name changes that we dont want to apply to squared right now. Could we create a new patch release to fix this?

Additionally it looks like the db versions were auto upgraded so when running an apply I get warnings that it cant downgrade my db from 13.7 to 13.4. This feels like the wrong place to do it since its specific to my deployment but wanted to see what you thought.

@pnadolny13 pnadolny13 requested a review from kgpayne October 12, 2022 15:28
@magreenbaum
Copy link

@pnadolny13 we can use >= constraint on versions within re-usable modules to prevent conflict errors also in the future.

Regarding db engine version, using a variable might be a good option here. We can set the default to 13.4 then you could pass your version in as a parameter in the workspace where you're calling the module from.

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.

2 participants