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

if "latest" for revision, grab latest deployment available for proxy #24

Merged
merged 3 commits into from
Mar 15, 2019

Conversation

zricethezav
Copy link
Collaborator

This PR includes a change that allows for users to set revision="latest" in their .tf files. During the Update stage, terraform will fetch the latest valid revision from the api proxy and deploy that revision.

@zambien
Copy link
Owner

zambien commented Mar 15, 2019

Nice. This looks like a step in the direction of fixing issue #8 too. Does that make sense?

@zricethezav
Copy link
Collaborator Author

Yep that is exactly what this does. If the revision is 'latest' terraform will grab the most recent deployed revision for some api proxy and use that as the target revision for the deployment resource. It's also nice if you dont want to depend on the apgiee_api_proxy resource

@zricethezav
Copy link
Collaborator Author

I took a look at the issue you opened hashicorp/terraform#15857 and this is exactly what we were running into. This is a bit hacky, but gets the job done.

@zambien
Copy link
Owner

zambien commented Mar 15, 2019

Nice! Thanks for the contribution.

@zricethezav
Copy link
Collaborator Author

@zambien great, mind giving it an approval then ill push out a new release?

@zambien
Copy link
Owner

zambien commented Mar 15, 2019

you got it!

@aessaid
Copy link

aessaid commented Mar 17, 2021

hello,
we still need to double apply to deploy the last revision.

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