Skip to content

Add AWS CloudTrail native extension and integration tests#8787

Merged
jamesnetherton merged 1 commit into
apache:mainfrom
JinyuChen97:camel-quarkus-5566
Jun 24, 2026
Merged

Add AWS CloudTrail native extension and integration tests#8787
jamesnetherton merged 1 commit into
apache:mainfrom
JinyuChen97:camel-quarkus-5566

Conversation

@JinyuChen97

@JinyuChen97 JinyuChen97 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Add native support for the AWS CloudTrail component, enabling users to interact with AWS CloudTrail service in native mode.

This implementation includes:

  • Runtime and deployment modules for native compilation
  • Integration tests with WireMock for testing CloudTrail operations
  • Consumer support for polling CloudTrail events
  • Documentation and extension metadata

Co-Authored-By: Jinyu Chen
Co-Authored-By: Claude Sonnet 4.5 noreply@anthropic.com

fixes #5566

@aldettinger

Copy link
Copy Markdown
Contributor

So this extension is directly added as native. Maybe we need to add complete tooling/scripts/test-categories.yaml ?

@JinyuChen97

Copy link
Copy Markdown
Contributor Author

So this extension is directly added as native. Maybe we need to add complete tooling/scripts/test-categories.yaml ?

the aws cloudtrail is under the aws2 folder, I saw the aws2 has been defined in group02, do I need to add aws cloudtrail alone to the test-categories?

@aldettinger

Copy link
Copy Markdown
Contributor

So this extension is directly added as native. Maybe we need to add complete tooling/scripts/test-categories.yaml ?

the aws cloudtrail is under the aws2 folder, I saw the aws2 has been defined in group02, do I need to add aws cloudtrail alone to the test-categories?

Ah ok, it should be fine then.

@aldettinger aldettinger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Failures are unrelated. Tests in group 02 and 12 looks good. Well donz 👏

Comment on lines +47 to +52
<exclusions>
<exclusion>
<groupId>commons-logging</groupId>
<artifactId>commons-logging</artifactId>
</exclusion>
</exclusions>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this? The same exclusion is already applied to camel-aws-cloudtrail.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes you are right, the aws sdk cloudtrail should not be added here either. Removed this already.

Comment on lines +66 to +70
<dependency>
<groupId>org.apache.camel.quarkus</groupId>
<artifactId>camel-quarkus-integration-wiremock-support</artifactId>
<scope>test</scope>
</dependency>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess WireMock is used because LocalStack does not support Cloudtrail?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, it's not supported in localstack community version.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok - then we should revisit after #8760 is done because I think Floci does have support for Cloudtrail.

Add native support for the AWS CloudTrail component, enabling
users to interact with AWS CloudTrail service in native mode.

This implementation includes:
- Runtime and deployment modules for native compilation
- Integration tests with WireMock for testing CloudTrail operations
- Consumer support for polling CloudTrail events
- Documentation and extension metadata

Co-Authored-By: Jinyu Chen
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@jamesnetherton jamesnetherton merged commit 3cf470e into apache:main Jun 24, 2026
22 checks passed
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.

Create an AWS Cloudtrail Extension

3 participants