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

Plugin installation fails from che-theia workspace #15006

Closed
2 of 16 tasks
JonasHelming opened this issue Oct 28, 2019 · 19 comments
Closed
2 of 16 tasks

Plugin installation fails from che-theia workspace #15006

JonasHelming opened this issue Oct 28, 2019 · 19 comments
Labels
area/editor/theia Issues related to the che-theia IDE of Che 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.

Comments

@JonasHelming
Copy link

I have a trial instance of openshift.io and a default workspace (Java maven). When I open the plugin view and click on "install" on any plugin, I get:
"Unable to install plugin 'redhat/php-debugger/latest'. Request addPlugin failed unexpectedly without providing any details."

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)

Environment

  • my computer
    • Windows
    • Linux
    • macOS
  • Cloud
    • Amazon
    • Azure
    • GCE
    • openshift.io
  • other: please specify
@JonasHelming JonasHelming added the kind/bug Outline of a bug - must adhere to the bug report template. label Oct 28, 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 Oct 28, 2019
@ibuziuk ibuziuk added team/osio 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 Oct 28, 2019
@ibuziuk ibuziuk added this to the Backlog - Hosted Che milestone Oct 30, 2019
@ibuziuk
Copy link
Member

ibuziuk commented Nov 4, 2019

Also fails for me but with a bit different flow on 7.3.1:

Peek 2019-11-04 20-25

@ibuziuk
Copy link
Member

ibuziuk commented Nov 4, 2019

@evidolob any comments on that ^ ?

@ibuziuk ibuziuk changed the title Plugin installation fails on openshift.io Plugin installation from che-theia workspace fails on Hosted Che Nov 4, 2019
@ibuziuk ibuziuk added the area/editor/theia Issues related to the che-theia IDE of Che label Nov 4, 2019
@skabashnyuk
Copy link
Contributor

Reproduced
Знімок екрана  о 16 32 24

@ibuziuk ibuziuk changed the title Plugin installation from che-theia workspace fails on Hosted Che Plugin installation fails from che-theia workspace Nov 5, 2019
@ibuziuk ibuziuk mentioned this issue Nov 6, 2019
23 tasks
@evidolob
Copy link
Contributor

evidolob commented Nov 6, 2019

che-theia use DELETE http request with /api/workspace/{workspaceId}/runtime URL.
When I do curl request on that URL, I got

Error: Failed to run the workspace: "Failure executing: POST at: https://f8osoproxy-test-dsaas-production.09b5.dsaas.openshiftapps.com/api/v1/namespaces/yvydolob-che/configmaps. 
Message: configmaps "workspacegyj2ynixetn1vkm2-gitconfig" already exists. Received status: 
Status(apiVersion=v1, code=409, details=StatusDetails(causes=[], group=null, kind=configmaps, name=workspacegyj2ynixetn1vkm2-gitconfig, retryAfterSeconds=null, uid=null, 
additionalProperties={}), kind=Status, message=configmaps "workspacegyj2ynixetn1vkm2-
gitconfig" already exists, metadata=ListMeta(_continue=null, resourceVersion=null, selfLink=null,
 additionalProperties={}), reason=AlreadyExists, status=Failure, additionalProperties={})."

error on ws-loader,
I think problem in implementation of /api/workspace/{workspaceId}/runtime REST service

@ibuziuk
Copy link
Member

ibuziuk commented Nov 6, 2019

@evidolob you execute delete and get Failure executing: POST ?
Also, could you please test locally (I see that you are still testing against Hosted Che)

@vitaliy-guliy
Copy link
Contributor

To solve the problem we need:

  • fix an error when updating the devfile of running workspace (Request addPlugin failed unexpectedly)
  • check workpsace status before refreshing the page (should wait until workspace status has been changed)

@vitaliy-guliy
Copy link
Contributor

@JonasHelming could you provide more information about your workspace? I need at least your devfile to reproduce the case and check the error when installing a plugin (addPlugin request).

I created a workspace from the following devfile but didn't get any errors.

metadata:
  name: che-theia-all
projects:
  - name: che-theia
    source:
      location: 'https://github.com/eclipse/che-theia.git'
      type: git
      branch: master
  - name: theia
    source:
      location: 'https://github.com/theia-ide/theia.git'
      type: git
      branch: master
components:
  - mountSources: true
    endpoints:
      - name: theia-dev-flow
        attributes:
          protocol: http
          public: 'true'
        port: 3010
    memoryLimit: 3000Mi
    type: dockerimage
    alias: che-dev
    image: 'eclipse/che-theia-dev:next'
apiVersion: 1.0.0
commands:
  - name: init ... DEV che-theia
    actions:
      - workdir: /projects/theia
        type: exec
        command: |
          che:theia init
        component: che-dev

@ibuziuk
Copy link
Member

ibuziuk commented Nov 7, 2019

@vitaliy-guliy hmm.. why do you need a devfile? steps to repro are basically:

  • start any workspace
  • open plugin view
  • install any plugin

#15006 (comment)

This need to be tested against multiuser with tls / celf-signed-certs and workspace creation in the unique namespace though

@vitaliy-guliy
Copy link
Contributor

@ibuziuk I cannot get an error with updating the devfile (adding a plugin component) on che.openshift.io as @JonasHelming noted

I did exactly as you wrote:

  • start any workspace
  • open plugin view
  • install any plugin

Result: new plugin component in the devfile is appeared.

But yes, there is still a problem with restarting the workspace.

@ibuziuk
Copy link
Member

ibuziuk commented Nov 7, 2019

But yes, there is still a problem with restarting the workspace.

This is the biggest issue, workspace is not stopped properly and all the workspace resources are still there (routes, deployments etc) see the error message #15006 (comment)

@ibuziuk
Copy link
Member

ibuziuk commented Nov 7, 2019

@evidolob if the plugin installation issue / workspace stop + restart is not Hosted Che specific we would probably need to disable plugin installation from che-theia UI

@skabashnyuk
Copy link
Contributor

@ibuziuk I think at this moment we can't guarantee for sure that there is no Hosted Che specific. I believe that we need to fix this redhat-developer/rh-che#1689 and clarify all exceptions that happened during workspace removal on Hosted Che.

@skabashnyuk
Copy link
Contributor

Tested upstream multi-user workspace stop with machine token on crc version: 1.1.0+95966a9 OpenShift version: 4.2.2 (embedded in binary). Everything works as expected.
Знімок екрана  о 13 41 00 No issues on che server logs

@vitaliy-guliy
Copy link
Contributor

Cannot stop the workspace with machine token on local minishift and openshift.io.

Che server responds OK on STOP request but then, when listening workspace status via websocket, client receives the message that an error occurred when stopping the workspace.
After that when getting the workspace info, it's status is STOPPED but on the infrastructure everything is still running.

@skabashnyuk
Copy link
Contributor

openshift.io is another story. I was not able to reproduce in on minishift. What was your configuration on minishift? chectl, operator, OAuth?

@Katka92
Copy link
Contributor

Katka92 commented Nov 12, 2019

Hi guys, I've tried to reproduce on Hosted Che. When I install a plugin and click the button to refresh the workspace, I got an error [1]. I've refreshed the browser window again and got a different error [2] saying that Config maps already exist.. I came to the console to check if the resources are really here - and there were, but in a few seconds they were deleted correctly. I refreshed again and the workspace was loaded correctly.

[1]
Screenshot from 2019-11-12 08-09-24

[2]
Screenshot from 2019-11-12 08-09-56

@vitaliy-guliy
Copy link
Contributor

Everything's merged. Must work now.

@ibuziuk
Copy link
Member

ibuziuk commented Nov 14, 2019

@vitaliy-guliy was it cherry-picked to 7.3.x?

@ibuziuk
Copy link
Member

ibuziuk commented Nov 19, 2019

verified on prod-preview running against 7.3.2 and the functionality works just fine \o/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/editor/theia Issues related to the che-theia IDE of Che 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

7 participants