Skip to content

feat: Use eventbus data for event archive when create_bus is set to false #176

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yuntac
Copy link

@yuntac yuntac commented Jul 25, 2025

Description

Allow to use eventbus data for event archive when create_bus is set to false

Motivation and Context

In case of create_bus is set to false, we can't use the aws_cloudwatch_event_bus data in aws_cloudwatch_event_archive resource. we have to set a new data in root module and use it in archives.event_source_arn. This feature allow to directly use the data of the child module if it's desired

Breaking Changes

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

@yuntac yuntac force-pushed the feat-use-event-bus-data-for-event-archives branch from 9309392 to 8bd6b6b Compare July 25, 2025 10:20
@yuntac yuntac force-pushed the feat-use-event-bus-data-for-event-archives branch from 8bd6b6b to 11f6db8 Compare July 25, 2025 10:22
@yuntac yuntac changed the title feat: allow to use eventbus data for event archive when create_bus is… feat: use eventbus data for event archive when create_bus is set to false Jul 25, 2025
@yuntac yuntac changed the title feat: use eventbus data for event archive when create_bus is set to false feat: Use eventbus data for event archive when create_bus is set to false Jul 25, 2025
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.

1 participant