-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Need to change Agent REST API and update UI in "Agents" section #2654
Comments
@gazarenkov @tolusha Please take a look |
I believe Ilya plans a pull request. |
Cosmetic changes, np to me. |
Looks good to me too. |
@garagatyi @gazarenkov @tolusha hi, guys. I have started to look into the 'wsmaster' component in order to change the REST API. Basically, I'm trying to figure out what is the best workflow / environment setup when working with 'wsmaster' component. For now I have imported the wsmaster module to the workspace and can browse through the code & run unit tests. What I would like to setup is possibility to debug wsmaster services. Could you please clarify how can I achieve this (maybe there is some doc) ? Thanks in advance |
Adding @sleshchenko - it may be a good idea to do a quick session with him and @slemeur. @sleshchenko has been developing a lot of our server-side APIs recently and he would have the best workflow in mind. And @slemeur has been documenting these various points for our docs. |
@TylerJewell thank you! I have just had a chat with @l0rd who recommended to use jpda approach:
AFAIK, there is no documentation covering jpda technic. @slemeur we can contribute the info about jpda setup to the che documentation (as far as I know most, if not all, from red hat use this approach for development) |
@ibuziuk |
I think the step 3 - build the che-server docker image is unnecessary. You can use the che-launcher with CHE_LOCAL_BINARY environment variable pointing to your local assembly. This would have the same affect, I believe. |
as @TylerJewell wrote, I'm using flag |
I confirm that @TylerJewell and @benoitf way to debug Che is more efficient than mine. That's because I love typing |
This issue is the second iteration of the #1774
It was decided to change change the Agents REST API the following way:
1 Add id field. The value of it would be the current value of the name field i.e. org.eclipse.che.ls.json
GET /agent/name/{name}
toGET /agent/id/{id}
After that need to update Dashboard UI, so that new API would be consumed.
Current behavior:
https://cloud.githubusercontent.com/assets/1461122/18795730/3551dc76-81c6-11e6-9e47-e47930e12092.gif
As you can see the "description" property is missing and the "name" property does not look viable from the user perspective
Che version: 5.0.0-M3
The text was updated successfully, but these errors were encountered: