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

prevent github OAuthToken from causing unwanted changes #47

Merged
merged 2 commits into from
Sep 12, 2020

Conversation

dknell
Copy link
Contributor

@dknell dknell commented Sep 12, 2020

what

  • Adding a lifecycle block to ignore changes in source stage

why

  • When using github OAuthToken, the aws_codepipeline resource doesn't save the token in the state file. This causes it to detect changes on every apply, regardless of a change in the terraform files.

references

@dknell dknell requested a review from a team as a code owner September 12, 2020 20:06
@dknell dknell requested review from nitrocode and SweetOps and removed request for a team September 12, 2020 20:06
@joe-niland
Copy link
Member

/test all

@joe-niland
Copy link
Member

/rebuild-readme

@joe-niland
Copy link
Member

/test all

@joe-niland joe-niland self-requested a review September 12, 2020 23:57
@joe-niland joe-niland merged commit 5a4abb7 into cloudposse:master Sep 12, 2020
@joe-niland
Copy link
Member

Thanks @dknell !

@dknell
Copy link
Contributor Author

dknell commented Sep 13, 2020

Thanks @joe-niland ! Curious if your modules that depend on this (like ecs-web-app) automatically update dependencies or do I need to submit a PR for that too?

@dknell dknell deleted the github-OAuthToken-lifecycle-hook branch September 13, 2020 00:09
@joe-niland
Copy link
Member

@dknell good question! At the moment it's manual, but others are looking into dependabot or similar.

If you have time, another PR on dependent modules would be great. If not, I think I could do ecs-web-app tomorrow.

@dknell
Copy link
Contributor Author

dknell commented Sep 13, 2020

on it!

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.

3 participants