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.

Sign up for GitHub

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

Jump to bottom

CAMEL-21452: decouple the infrastructure from the testing API #16322

Merged
merged 26 commits into from
Jan 9, 2025
Merged

CAMEL-21452: decouple the infrastructure from the testing API #16322

merged 26 commits into from
Jan 9, 2025

Conversation

Croway
Copy link
Contributor

@Croway Croway commented Nov 19, 2024

common test-infra refactor plus following components:

  • arangodb

@Croway Croway requested a review from orpiske November 19, 2024 10:54
@github-actions GitHub Actions
Copy link
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run

  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

  • You can label PRs using build-all, build-dependents, skip-tests and test-dependents to fine-tune the checks executed by this PR.

  • Build and test logs are available in the Summary page. Only Apache Camel committers have access to the summary.

  • ⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@Croway Croway requested a review from cunningt November 20, 2024 12:46
@Croway Croway force-pushed the CAMEL-21452 branch 2 times, most recently from 135c708 to d2c6a0a Compare November 20, 2024 14:01
@Croway Croway force-pushed the CAMEL-21452 branch 2 times, most recently from 677fa78 to 7f3d56c Compare November 28, 2024 15:35
@orpiske
Copy link
Contributor

orpiske commented Nov 29, 2024

Folks, just let me know once you want me to start running tests for this on my own build nodes.

@Croway
Copy link
Contributor Author

Croway commented Nov 29, 2024

Folks, just let me know once you want me to start running tests for this on my own build nodes.

Sure thing I'll let you know, right now this branch is not building, as soon as we finish the refactoring I'll let you know. Thanks!

@Croway Croway marked this pull request as ready for review January 7, 2025 10:39
@Croway Croway force-pushed the CAMEL-21452 branch 4 times, most recently from b2d33be to c930ade Compare January 9, 2025 14:17
@orpiske
Copy link
Contributor

orpiske commented Jan 9, 2025

@oscerd @davsclaus I'm wondering if we still have time to get this one in shape for 4.10. Any thoughts?

@oscerd
Copy link
Contributor

oscerd commented Jan 9, 2025

I guess we have time for including it

@orpiske
Copy link
Contributor

orpiske commented Jan 9, 2025

Great, thanks. I am running a few more tests on my own CI, but once everything is ready I can merge this one

@orpiske orpiske merged commit b69f248 into apache:main Jan 9, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@orpiske orpiske orpiske left review comments

@oscerd oscerd oscerd approved these changes

@cunningt cunningt Awaiting requested review from cunningt

Assignees
No one assigned
Labels
camel-4 components components-ai components-aws components-azure components-jms components-kafka test-infra
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Croway @orpiske @oscerd @cunningt