Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

feat: introduce devfile v2 for Apache Camel-k #548

Merged
merged 1 commit into from
Dec 10, 2021

Conversation

benoitf
Copy link
Contributor

@benoitf benoitf commented Dec 10, 2021

What does this PR do?

Add camel-k examples

There is an issue for Apache Camel Language Server but it's not related to the devfile itself
Issue found:
camel-tooling/camel-lsp-client-vscode#836
PR to fix:
camel-tooling/camel-lsp-client-vscode#837

Screenshot/screencast of this PR

What issues does this PR fix or reference?

part of eclipse-che/che#19341

How to test this PR?

Click on https://che-server#https://github.com/che-samples/apache-camel-k/tree/devfilev2

Go to the View/Output and check Apache Camel-k output

you should see that camel-k binary has been found on the filesystem.

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Change-Id: I50035d90f917bde31d92cd27ce9f9042103be498
Signed-off-by: Florent Benoit fbenoit@redhat.com

Change-Id: I50035d90f917bde31d92cd27ce9f9042103be498
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@github-actions
Copy link

Click here to review and test in web IDE: Contribute

Copy link
Contributor

@svor svor left a comment

Choose a reason for hiding this comment

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

I see an error in Camel K output window but the plugin was initialized

Found Kubernetes CLI (kubectl) version 1.23... 
No version found. Using default version 1.5.0 of Apache Camel K CLI 
error: the server doesn't have a resource type "integrations"

I'm not familiar with Apache Camel K tools, so can't say if it's a big problem or not.
LGTM

@svor svor merged commit c9df29e into eclipse-che:main Dec 10, 2021
@che-bot che-bot added this to the 7.41 milestone Dec 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants