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

Devops/adding eventbridge #255

Closed
wants to merge 31 commits into from

Conversation

Mhart12
Copy link

@Mhart12 Mhart12 commented Nov 21, 2024

what

This adds IAM and EventBridge resources in order for CodeDeploy to trigger new deployment based on the default task definition in ECS.

why

So we don't have to manually trigger CodeDeploys

references

https://kininsurance.atlassian.net/browse/DK-824

Mhart12 and others added 30 commits December 20, 2023 14:48
updating to use var instead of id
* adding autoscaling capabilites

* removing ecs cluster name var
* adding autoscaling capabilites

* removing ecs cluster name var

* fixing resource instance key

* fixing resource instance key
* adding autoscaling capabilites

* removing ecs cluster name var

* fixing resource instance key

* fixing resource instance key

* adding count attribute for role
* adding autoscaling capabilites

* removing ecs cluster name var

* fixing resource instance key

* fixing resource instance key

* adding count attribute for role

* adding count attribute
…resource (#7)

* adding autoscaling capabilites

* removing ecs cluster name var

* fixing resource instance key

* fixing resource instance key

* adding count attribute for role

* adding count attribute

* removing role arn, not needed

* changing back to name
* adding autoscaling capabilites

* removing ecs cluster name var

* fixing resource instance key

* fixing resource instance key

* adding count attribute for role

* adding count attribute

* removing role arn, not needed

* changing back to name

* updating to use ecs_service_name instead of mod name
* adding autoscaling capabilites

* removing ecs cluster name var

* fixing resource instance key

* fixing resource instance key

* adding count attribute for role

* adding count attribute

* removing role arn, not needed

* changing back to name

* updating to use ecs_service_name instead of mod name

* remove weird formatting for task policy arns

* testing new family setup and removing weird formatting

* testing tag updates

* updating container def tags

* testing attributes

* just going to use tags

* reverting formatting
* adding lifecycle event to ignore task def, desired count, and load balancer changes

* Revert "adding lifecycle event to ignore task def, desired count, and load balancer changes"

This reverts commit 683eccc.

* asking terraform to ignore changes to load balancer when blue/green deploys happen
* updating resource id service name ref

* using try to catch false and true situations for service name
* adding sg_name for dot-com

* changing empty string to null

---------

Co-authored-by: Monica Hart <[email protected]>
Co-authored-by: Monica Hart <[email protected]>
@Mhart12 Mhart12 requested review from a team as code owners November 21, 2024 16:55
@Mhart12 Mhart12 requested review from Gowiem and jamengual November 21, 2024 16:55
Copy link

mergify bot commented Nov 21, 2024

💥 This pull request now has conflicts. Could you fix it @Mhart12? 🙏

@mergify mergify bot added conflict This PR has conflicts triage Needs triage labels Nov 21, 2024
@Mhart12 Mhart12 closed this Nov 21, 2024
@mergify mergify bot removed conflict This PR has conflicts triage Needs triage labels Nov 21, 2024
@Mhart12 Mhart12 deleted the devops/adding-eventbridge branch November 21, 2024 17:15
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