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

Drop unused permissions from preconditions script #143

Merged

Conversation

adrienthebo
Copy link
Contributor

@adrienthebo adrienthebo commented Feb 21, 2019

Work in progress.

  • Verify that CI passes with the dropped permission requirements
  • Update README with reduced CI requirements

@adrienthebo adrienthebo force-pushed the adrienthebo/maint/preconditions-drop-unused-requirements branch 2 times, most recently from 9072f39 to 926009c Compare February 21, 2019 22:19
@adrienthebo adrienthebo changed the title (WIP) Drop unused permissions from preconditions script Drop unused permissions from preconditions script Feb 21, 2019
morgante
morgante previously approved these changes Feb 22, 2019
aaron-lane
aaron-lane previously approved these changes Feb 22, 2019
The service account running Terraform does not need
`roles/iam.serviceAccountAdmin` on the entire organization. This commit
updates the README accordingly.
@adrienthebo adrienthebo dismissed stale reviews from aaron-lane and morgante via 977fc93 February 22, 2019 17:53
@adrienthebo adrienthebo force-pushed the adrienthebo/maint/preconditions-drop-unused-requirements branch from 2d13558 to 977fc93 Compare February 22, 2019 17:53
@morgante morgante self-requested a review February 22, 2019 17:54
@morgante morgante merged commit 8c7db39 into master Feb 22, 2019
@aaron-lane aaron-lane deleted the adrienthebo/maint/preconditions-drop-unused-requirements branch February 22, 2019 18:06
adrienthebo added a commit that referenced this pull request Feb 22, 2019
adrienthebo added a commit that referenced this pull request Feb 22, 2019
aaron-lane added a commit that referenced this pull request Feb 22, 2019
…nt/update-changelog

Add CHANGELOG entries for PR #143
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