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.

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

Skip surefire when building the IntegrationKit #5472

Closed
lburgazzoli opened this issue May 7, 2024 · 1 comment · Fixed by #5567
Closed

Skip surefire when building the IntegrationKit #5472

lburgazzoli opened this issue May 7, 2024 · 1 comment · Fixed by #5567
Assignees
Labels
area/builder good first issue Does not require full understanding of the codebase kind/bug Something isn't working
Milestone

Comments

@lburgazzoli
Copy link
Contributor

lburgazzoli commented May 7, 2024

What happened?

When building an IntegrationKit, the maven-surefire-plugin:test, maven-resources-plugin:testResources mojos are useless and can be skipped to save some time and CPU cycles.

Steps to reproduce

  1. run an integration
  2. watch the maven build log

Relevant log output

{"level":"info","ts":"2024-05-07T21:55:54Z","logger":"camel-k.maven.build","msg":"--- maven-surefire-plugin:2.12.4:test (default-test) @ camel-k-integration ---"}
{"level":"info","ts":"2024-05-07T21:55:57Z","logger":"camel-k.maven.build","msg":"No tests to run."}
{"level":"info","ts":"2024-05-07T21:55:57Z","logger":"camel-k.maven.build","msg":""}
{"level":"info","ts":"2024-05-07T22:01:01Z","logger":"camel-k.maven.build","msg":"--- maven-resources-plugin:2.6:testResources (default-testResources) @ camel-k-integration ---"}
{"level":"info","ts":"2024-05-07T22:01:01Z","logger":"camel-k.maven.build","msg":"Using 'UTF-8' encoding to copy filtered resources."}
{"level":"info","ts":"2024-05-07T22:01:01Z","logger":"camel-k.maven.build","msg":"skip non existing resourceDirectory /tmp/kit-cota5gcjp6es73cbksm0-170601170/maven/src/test/resources"}

Camel K version

No response

@lburgazzoli lburgazzoli added kind/bug Something isn't working area/build-operator Related to the internal image build operator labels May 7, 2024
@squakez squakez added area/builder and removed area/build-operator Related to the internal image build operator labels May 8, 2024
@squakez
Copy link
Contributor

squakez commented May 8, 2024

@lburgazzoli fyi, we're using area/build-system for those issues related to the operator build image (ie, make images). We're instead using area/builder for the other one. It can be misleading and I'm moving the first to area/build-operator.

@squakez squakez added the good first issue Does not require full understanding of the codebase label May 9, 2024
@squakez squakez self-assigned this May 31, 2024
@squakez squakez added this to the 2.4.0 milestone May 31, 2024
squakez added a commit to squakez/camel-k that referenced this issue May 31, 2024
We have no reason to try executing any test as we don't expect the presence of any test at all

Closes apache#5472
squakez added a commit to squakez/camel-k that referenced this issue May 31, 2024
squakez added a commit to squakez/camel-k that referenced this issue Jun 3, 2024
We have no reason to try executing any test as we don't expect the presence of any test at all

Closes apache#5472
squakez added a commit to squakez/camel-k that referenced this issue Jun 3, 2024
We have no reason to try executing any test as we don't expect the presence of any test at all

Closes apache#5472
squakez added a commit to squakez/camel-k that referenced this issue Jun 3, 2024
We have no reason to try executing any test as we don't expect the presence of any test at all

Closes apache#5472
squakez added a commit to squakez/camel-k that referenced this issue Jun 3, 2024
We have no reason to try executing any test as we don't expect the presence of any test at all

Closes apache#5472
squakez added a commit that referenced this issue Jun 3, 2024
We have no reason to try executing any test as we don't expect the presence of any test at all

Closes #5472
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/builder good first issue Does not require full understanding of the codebase kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants