[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-16 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1432769232 Sounds good to be reviewed now, the build `native / install-native` only failed during the cleanup phase, the tests pass -- This is an automated message from the Apache Git Service. To

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-09 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1424427236 Depends on https://github.com/container-tools/kind-action/pull/12 (memory allocated to Colima by default is too low) -- This is an automated message from the Apache Git Service. To res

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-08 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1422847031 Depends on https://github.com/container-tools/kind-action/pull/11 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-07 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1421335897 Depends on https://github.com/container-tools/kind-action/pull/10 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-06 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1419395648 FYI, the build is not failing because of these changes since the same errors occur on a different PR -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-06 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1419101180 I added some unit tests -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-06 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1419098767 > Have you rebased it? it seems there was some failure I've fixed in #4022 Yes I rebased it already -- This is an automated message from the Apache Git Service. To respo

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-06 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1418679580 Infortunatelly the native E2E test cannot pass on the GitHub CI because it requires more than 6 Go of RAM, so I need to remove it. -- This is an automated message from the Apache Git S

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-03 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1415777497 @squakez your remarks have been addressed. What remains: * The test `TestOLMAutomaticUpgrade` fails and it is not clear to me why. I see this [test fails in another build](https:/

[GitHub] [camel-k] essobedo commented on pull request #4021: Ref #3023: Add native compilation with sources

2023-02-02 Thread via GitHub
essobedo commented on PR #4021: URL: https://github.com/apache/camel-k/pull/4021#issuecomment-1413913816 Your idea is great however it is scoped to Camel K 2.0 while I would expect a solution for Camel K 1.17, don't you believe that it could be an acceptable approach for 1.17? -- This is