Skip to content

actually allow to select custom IDE image #2507

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

Merged
merged 1 commit into from
Dec 14, 2020

Conversation

akosyakov
Copy link
Member

@akosyakov akosyakov commented Dec 11, 2020

What it does

How to test

  • Have custom IDE permission enabled: update d_b_user set rolesOrPermissions = '["admin","devops","ide-settings"]';
  • Go to settings
  • Try to select "Image" option
  • Try to play by typing image and switching radio button. You should not get anything surprising and on reload values should be persisted.

@akosyakov akosyakov marked this pull request as draft December 11, 2020 09:37
@akosyakov akosyakov force-pushed the akosyakov/staging-unable-to-select-2477 branch 3 times, most recently from f2196b0 to 33a1bf5 Compare December 11, 2020 11:18
@akosyakov
Copy link
Member Author

@csweichel I think it would be good to add some placehoder for image, do you have something in mind which points to the pattern which shoudl be used.

@akosyakov akosyakov force-pushed the akosyakov/staging-unable-to-select-2477 branch from 33a1bf5 to 08e403f Compare December 11, 2020 11:30
@akosyakov akosyakov marked this pull request as ready for review December 11, 2020 11:40
@csweichel
Copy link
Contributor

@csweichel I think it would be good to add some placehoder for image, do you have something in mind which points to the pattern which shoudl be used.

We could use something that looks like a docker image reference, e.g. index.docker.io/gitpod-io/theia-ide:latest.
For now we don't have a minimal example custom IDE, let alone documentation of how that contract looks like (my Todo, see #2419)

@akosyakov
Copy link
Member Author

We could use something that looks like a docker image reference, e.g. index.docker.io/gitpod-io/theia-ide:latest.
For now we don't have a minimal example custom IDE, let alone documentation of how that contract looks like (my Todo, see #2419)

We could use our theia image in the placeholder? or it is internal secret

btw @JesterOrNot is already looking in to #2419

@csweichel
Copy link
Contributor

We could use something that looks like a docker image reference, e.g. index.docker.io/gitpod-io/theia-ide:latest.
For now we don't have a minimal example custom IDE, let alone documentation of how that contract looks like (my Todo, see #2419)

We could use our theia image in the placeholder? or it is internal secret

It's not a secret per se, but not something we'd want to surface either. Also, it depends on the deployment, i.e. is not a fixed value.

@akosyakov akosyakov force-pushed the akosyakov/staging-unable-to-select-2477 branch 4 times, most recently from ed2e392 to a0cec34 Compare December 12, 2020 14:05
@akosyakov akosyakov force-pushed the akosyakov/staging-unable-to-select-2477 branch from a0cec34 to 7396b44 Compare December 12, 2020 14:15
@akosyakov
Copy link
Member Author

@csweichel I added the empty value validation and the placeholder.

@csweichel
Copy link
Contributor

csweichel commented Dec 14, 2020

/werft run

👍 started the job as gitpod-build-akosyakov-staging-unable-to-select-2477.11

Copy link
Contributor

@csweichel csweichel left a comment

Choose a reason for hiding this comment

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

LGTM

@akosyakov akosyakov merged commit 227b132 into master Dec 14, 2020
@akosyakov akosyakov deleted the akosyakov/staging-unable-to-select-2477 branch December 14, 2020 10:26
@yyoncho
Copy link

yyoncho commented Jun 29, 2021

Why Select custom IDE image is disabled for the public gitpod? What are the chances to have that enabled for the public gitpod?

@csweichel
Copy link
Contributor

We've not made this selection public for now, because we fret it would lead to more confusion than do good. The IDE image has very particular requirements that are hard to fulfil, not intuitive and can break things in really strange ways. For example, the IDE image must be built FROM scratch and ideally not touch anything outside of /ide.

Which use-case/IDE do you have in mind?

@yyoncho
Copy link

yyoncho commented Jun 30, 2021

@csweichel

Which use-case/IDE do you have in mind?

I am working on integration between Emacs and gitpod see emacs-lsp/lsp-mode#2845 and
https://github.com/emacs-lsp/lsp-gitpod . It is still a work in progress but it works fine. The issue is that I can specify the image in several ways but there is no way to specify the emacs image globally so I can just open any github project using that image. This issue is one of the blockers.

We've not made this selection public for now, because we fret it would lead to more confusion than do good.

It will work fine for me if this is not public but accessible in a certain way. (e. g. uri param, env variable, etc).

@svenefftinge
Copy link
Member

Hey @yyoncho, what you are trying to accomplish sounds pretty cool. I'm not an emacs user myself but if you are interested we could talk through your plans and see how Gitpod can support that and where we would need to change.
https://calendly.com/efftinge/emacs-gitpod

@yyoncho
Copy link

yyoncho commented Jul 2, 2021

@svenefftinge thank you, I scheduled a meeting on Monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[staging] Unable to select custom image
4 participants