Skip to content

Issue 47 notification on failure#56

Merged
juanmatias merged 32 commits intomasterfrom
ISSUE-47-notification-on-failure
Jun 1, 2023
Merged

Issue 47 notification on failure#56
juanmatias merged 32 commits intomasterfrom
ISSUE-47-notification-on-failure

Conversation

@juanmatias
Copy link
Contributor

what

  • Retry on merge failure
    • when git pull from upstream fails git pull --rebase is run instead
    • if this success then the git push with the adding of flag -f is run
  • Notification to Slack
    • When the process fails a Slack notif is sent
    • Also is possible to send notifications on success

why

references

@juanmatias juanmatias added enhancement New feature or request patch labels Mar 31, 2023
@juanmatias juanmatias self-assigned this Mar 31, 2023
@juanmatias juanmatias requested a review from a team as a code owner March 31, 2023 18:12
@exequielrafaela exequielrafaela requested a review from a team March 31, 2023 19:13
@juanmatias
Copy link
Contributor Author

Reports:

  • binbashar/terraform-aws-config switched master to main
  • binbashar/terraform-aws-cloudfront-s3-cdn switched master to main
  • binbashar/terraform-terraform-label switched master to main

@juanmatias
Copy link
Contributor Author

  • binbashar/terraform-null-label/branches switched master to main
  • binbashar/terraform-aws-vpc-base switched master to main

Regarding terraform-aws-transit-gateway is being synced from two different repos:

"terraform-aws-transit-gateway","cloudposse/terraform-aws-transit-gateway","master" \
"terraform-aws-transit-gateway","terraform-aws-modules/terraform-aws-transit-gateway","master" \

Due to this order in the action file, the second one won´t succeed (when the script tries to add an upstream remote it fails the second time since it already exists). So I will comment out the second line and change the default branch on our fork to main.

@juanmatias
Copy link
Contributor Author

  • binbashar/terraform-aws-sso switched master to main
  • binbashar/terraform-aws-secure-baseline switched master to main
  • binbashar/terraform-aws-sns-topic switched master to main
  • binbashar/terraform-aws-s3-log-storage switched master to main

@juanmatias
Copy link
Contributor Author

  • binbashar/terraform-aws-route53-query-logs switched master to main
  • binbashar/terraform-aws-route53-alias switched master to main
  • binbashar/terraform-aws-root-login-notifications switched master to main

@juanmatias
Copy link
Contributor Author

  • binbashar/terraform-aws-multi-az-subnets switched master to main
  • binbashar/terraform-aws-msk-apache-kafka-cluster switched master to main
  • binbashar/terraform-aws-logs switched master to main
  • binbashar/terraform-aws-kms-key switched master to main
  • binbashar/terraform-aws-firewall-manager switched master to main

@juanmatias
Copy link
Contributor Author

  • binbashar/terraform-aws-elasticache-redis switched master to main
  • binbashar/terraform-aws-dynamodb switched master to main
  • binbashar/terraform-aws-cloudtrail-s3-bucket switched master to main
  • binbashar/terraform-aws-cloudtrail-cloudwatch-alarms switched master to main
  • binbashar/terraform-aws-cloudtrail/branches switched master to main

@juanmatias
Copy link
Contributor Author

  • binbashar/terraform-aws-ses switched master to main

@juanmatias juanmatias merged commit 08770bb into master Jun 1, 2023
@juanmatias juanmatias deleted the ISSUE-47-notification-on-failure branch June 1, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Divergent branches found when syncing repos

3 participants