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

Theia plugins extension doesn't work properly #14553

Closed
4 of 23 tasks
olexii4 opened this issue Sep 14, 2019 · 1 comment
Closed
4 of 23 tasks

Theia plugins extension doesn't work properly #14553

olexii4 opened this issue Sep 14, 2019 · 1 comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Milestone

Comments

@olexii4
Copy link
Contributor

olexii4 commented Sep 14, 2019

Describe the bug

Theia plugins extension doesn't work by default.

Che version

  • latest
  • nightly
  • other: please specify

Steps to reproduce

  1. Create a new workspace with the devfile https://raw.githubusercontent.com/eclipse/che-docs/master/devfile.yaml.
http://<your-che-host>/f?url=https://raw.githubusercontent.com/eclipse/che-docs/master/devfile.yaml
  1. Press 'Ctrl' + 'Shft' + 'j' in the Theia editor.

Plugins widget will be open with an error: 'Your registry is invalid'

Expected behavior

A list of Theia plugins(the same which is in UD).

Runtime

  • kubernetes (include output of kubectl version)
  • Openshift (include output of oc version)
  • minikube (include output of minikube version and kubectl version)
  • minishift (include output of minishift version and oc version)
  • docker-desktop + K8S (include output of docker version and kubectl version)
  • other: (please specify)

Screenshots

Screenshot from 2019-09-14 17-57-45

Installation method

  • chectl
  • che-operator
  • minishift-addon
  • I don't know

Environment

  • my computer
    • Windows
    • Linux
    • macOS
  • Cloud
    • Amazon
    • Azure
    • GCE
    • other (please specify)
  • other: please specify

Additional context

Tested with cheEditor 'che-theia' (latest and next).

@olexii4 olexii4 added kind/bug Outline of a bug - must adhere to the bug report template. team/ide2 labels Sep 14, 2019
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Sep 14, 2019
@ibuziuk ibuziuk added severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Sep 15, 2019
@olexii4
Copy link
Contributor Author

olexii4 commented Sep 16, 2019

To get plugins from a repo, we need two requests:

  1. <your-che-host>/api/workspace/settings
    and get a responce like this
{
...
cheWorkspacePluginRegistryUrl: "http://plugin-registry-che.192.168.99.100.nip.io/v3",
}
  1. <cheWorkspacePluginRegistryUrl>/plugins/
[
 {...},
 {...},
  ...
]

The problem was there:

https://github.com/eclipse/che-theia/blob/852eb4a16c91d93ef7dd087e8dea195c031612eb/extensions/eclipse-che-theia-plugin-ext/src/node/che-plugin-service.ts#L87

        } catch (error) {
            // console.log('ERROR', error);
            // return Promise.reject('Unable to get default plugin registry URI. ' + error.message);

            // A temporary solution. Should throw an error instead.
            this.defaultRegistry = {
                name: 'Eclipse Che plugin registry',
                uri: 'https://che-plugin-registry.openshift.io/v3/plugins/'
            };

Because cheApiService.getWorkspaceSettings() was not implemented at all
https://github.com/eclipse/che-theia/blob/f79cb66c9a57ea9df488b8c62929aecf322f5642/extensions/eclipse-che-theia-plugin-ext/src/plugin/che-workspace.ts#L25

    getSettings(): Promise<che.KeyValue> {
        throw new Error('Method not implemented.');
    }

@sunix sunix added this to the 7.3.0 milestone Sep 18, 2019
evidolob added a commit to eclipse-che/che-theia that referenced this issue Oct 7, 2019
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
evidolob added a commit to eclipse-che/che-theia that referenced this issue Oct 8, 2019
Signed-off-by: Yevhen Vydolob <yvydolob@redhat.com>
@evidolob evidolob closed this as completed Oct 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system.
Projects
None yet
Development

No branches or pull requests

5 participants