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

Add Makefile to run YAKS tests #1625

Merged
merged 2 commits into from Sep 7, 2023
Merged

Add Makefile to run YAKS tests #1625

merged 2 commits into from Sep 7, 2023

Conversation

christophd
Copy link
Contributor

  • Adding Makefile as a wrapper for the YAKS binary
  • Makes local YAKS test execution more comfortable
  • Automatically set YAKS runtime options such as Camel JBang version or local Kamelets directory
  • Use the Makefile in GitHub actions CI to run all YAKS tests

@christophd
Copy link
Contributor Author

D'oh why is that aws-s3-to-http.feature starting to fail!? Locally it works - I keep investigating ...

@christophd
Copy link
Contributor Author

got it! we need to use YAKS_JBANG_KAMELETS_VERSION=4.1.0-SNAPSHOT because this is the new snapshot version that gets built with the GitHub actions CI job. PR #1626 fixes it so this PR needs to wait and rebase once the other one got merged.

@christophd
Add Makefile to run YAKS tests
1ca2302 
- Adding Makefile as a wrapper for the YAKS binary
- Makes local YAKS test execution more comfortable
- Automatically set YAKS runtime options such as Camel JBang version or local Kamelets directory
- Use the Makefile in GitHub actions CI to run all YAKS tests
@christophd
Improve aws-s3-to-http.feature
7b5a312 
Properly waiting for Camel routes to startup before triggering S3 file on bucket
@christophd
Copy link
Contributor Author

Rebase done based on the version fix - let's see how it goes ...

@oscerd oscerd merged commit b5ba90c into apache:main Sep 7, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oscerd oscerd oscerd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@christophd @oscerd