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

Improve adf-pipelines CodeBuild permissions kickstarting SFN #569

Merged

Conversation

sbkok
Copy link
Collaborator

@sbkok sbkok commented Dec 14, 2022

Why?

With the move to Step Functions, the majority of the workload for the adf-pipelines CodeBuild process was moved to the ADF Pipeline Management State Machine in AWS Step Functions.

However, the policies attached to the original CodeBuild environment were not updated yet. Additionally, the machine type used for this process could be reduced in size too.

What?

  • Changed the name from pipeline-provisioner* to pipeline-generation-provisioner*, as it no longer privisions the pipelines but ensures the Step Function has the required input to do it instead.
  • Removed the original provisioner role from AssumeRole permissions that it no longer requires.
  • Removed environment variables that are not used by the process to provision the pipeline generator.
  • Removed policy statements that the pipeline-generation provisioner does not require.
  • Applied the /adf-automation/ path to the pipeline-generation provisioner role.

By submitting this pull request, I confirm that you can use, modify, copy, and
redistribute this contribution, under the terms of your choice.

**Why?**

With the move to Step Functions, the majority of the workload for the
adf-pipelines CodeBuild process was moved to the ADF Pipeline Management State
Machine in AWS Step Functions.

However, the policies attached to the original CodeBuild environment were not
updated yet. Additionally, the machine type used for this process could be
reduced in size too.

**What?**

* Changed the name from `pipeline-provisioner*` to
  `pipeline-generation-provisioner*`, as it no longer privisions the pipelines
  but ensures the Step Function has the required input to do it instead.
* Removed the original provisioner role from AssumeRole permissions that it no
  longer requires.
* Removed environment variables that are not used by the process to provision
  the pipeline generator.
* Removed policy statements that the pipeline-generation provisioner does not
  require.
* Applied the `/adf-automation/` path to the pipeline-generation provisioner
  role.
@sbkok sbkok added the enhancement New feature or request label Dec 14, 2022
@sbkok sbkok added this to the v3.2.0 milestone Dec 14, 2022
@StewartW StewartW merged commit c890a7c into awslabs:master Dec 15, 2022
@sbkok sbkok deleted the refactor/codebuild-kickstart-sfn-permissions branch December 20, 2022 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants