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

Multiple issues with nodejs devfile #13806

Closed
4 of 5 tasks
JPinkney opened this issue Jul 9, 2019 · 10 comments
Closed
4 of 5 tasks

Multiple issues with nodejs devfile #13806

JPinkney opened this issue Jul 9, 2019 · 10 comments
Labels
kind/bug Outline of a bug - must adhere to the bug report template. status/analyzing An issue has been proposed and it is currently being analyzed for effort and implementation approach

Comments

@JPinkney
Copy link
Contributor

JPinkney commented Jul 9, 2019

Description

While trying to dogfood Che I found multiple issues with the node devfile that makes working with it almost impossible right off the bat.

Just from the short video below I've found a few things:

  • Git clone location prompt shows you by default somewhere that is different than /projects
    This dialog is being removed.
  • Git clone gets permission denied
    Looks like you're trying to clone to /. User error?
  • When opening up the node terminal it sets you to a location other than /projects
    Which container?
  • Permission denied when running npm install
  • yarn does not exist in the image (you can easily install it but a lot of projects I see use yarn)

https://www.youtube.com/watch?v=ep5nOJiL99w

Diagnostics:

7.0.0-rc-4.0-SNAPSHOT deployed via chectl onto minikube

@l0rd l0rd added kind/bug Outline of a bug - must adhere to the bug report template. status/analyzing An issue has been proposed and it is currently being analyzed for effort and implementation approach labels Jul 9, 2019
@slemeur slemeur modified the milestone: 7.0.0 Jul 10, 2019
@slemeur
Copy link
Contributor

slemeur commented Jul 10, 2019

Please @tsmaeder and @tolusha give a look to this issue.
Probably related to eclipse-che/che-devfile-registry#29

@JPinkney
Copy link
Contributor Author

If i'm not mistaken when I was testing it was using registry.access.redhat.com/ubi8/nodejs-10 as the image so my devfile must not have been updated. I'll try with the new one and report back.

@l0rd
Copy link
Contributor

l0rd commented Jul 10, 2019

cc @skabashnyuk @sparkoo @ibuziuk

@ibuziuk
Copy link
Member

ibuziuk commented Jul 11, 2019

I suspect that most of the issues would be fixed with https://github.com/eclipse/che-devfile-registry/tree/master/arbitrary-users-patch images - will try to check this assumption after CI setup

@tsmaeder
Copy link
Contributor

@ibuziuk yes, looks like it. However, the fact that yarn is IMO one of the cases where the user will have to bring their own images. Not saying we can't offer a devfile with yarn installed, but there will always be that "one more tool" some user needs that is not installed on any of the packaged devfiles.

@JPinkney
Copy link
Contributor Author

Looks like you're trying to clone to /. User error?

I would have expected that if projects is selected in the dialog that it would clone to /projects maybe that's an user error, maybe its not, that's just what I expected. I tried double clicking on /projects and it brought me to
2019-07-11-092219_763x733_scrot and then when I selected 'Select Repository Location' the error that came up was

Git: fatal: could not create work tree dir '/schemastore': Permission denied

(I was test cloning https://github.com/schemaStore/schemastore)

Whats interesting is I was able to successfully clone it in other locations like theia/node_modules/@eclipse-che/api folder.

Which container?

Its the container with alias nodejs
From the devfile:

type: dockerimage
image: registry.access.redhat.com/ubi8/nodejs-10
alias: nodejs

However, the fact that yarn is IMO one of the cases where the user will have to bring their own images.

True, I'm just sad I have to do that myself 😉

@slemeur slemeur added team/languages status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Jul 26, 2019
@benoitf benoitf removed the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Aug 5, 2019
@benoitf benoitf added this to the 7.2.0 milestone Aug 5, 2019
@tsmaeder tsmaeder modified the milestones: 7.2.0, Backlog - Languages Sep 23, 2019
@tsmaeder
Copy link
Contributor

It all works. The fact that yarn is not installed is not a bug.

@slemeur
Copy link
Contributor

slemeur commented Sep 25, 2019

@tsmaeder : Could you create an enhancement issue to add "yarn" by default in the default nodejs stack? Because I agree with the initial requirement, that gets pretty "standard" and a lot of projects are using it. Having it, would be beneficial to end-users.

@slemeur
Copy link
Contributor

slemeur commented Oct 1, 2019

ping @tsmaeder ???

@slemeur
Copy link
Contributor

slemeur commented Oct 1, 2019

#14720

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. status/analyzing An issue has been proposed and it is currently being analyzed for effort and implementation approach
Projects
None yet
Development

No branches or pull requests

7 participants