Skip to content
This repository has been archived by the owner on Aug 16, 2024. It is now read-only.

Update NodeJS-MongoDB devfile to use new sample #99

Merged
merged 2 commits into from
Oct 3, 2019
Merged

Update NodeJS-MongoDB devfile to use new sample #99

merged 2 commits into from
Oct 3, 2019

Conversation

svor
Copy link
Contributor

@svor svor commented Sep 23, 2019

Signed-off-by: svor vsvydenk@redhat.com

What does this PR do?

Updates stack for developing nodeJS+mongoDB web application.
The application is https://github.com/che-samples/nodejs-mongodb-sample

The demo about how it works: https://youtu.be/UYTpVw2uasI

What issues does this PR fix or reference?

eclipse-che/che#14392

Signed-off-by: svor <vsvydenk@redhat.com>
@svor svor self-assigned this Sep 23, 2019
Copy link
Contributor

@amisevsk amisevsk left a comment

Choose a reason for hiding this comment

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

One suggestion below. I also get a notification when running:

A new process is now listening on port 9229 but is listening on interface 127.0.0.1 which is internal. You should change to be remotely available. Would you want to add a redirect for this port so it becomes available ?

but that is probably a separate issue.

Apart from that I've briefly tested and it works well.

devfiles/nodejs-mongo/devfile.yaml Show resolved Hide resolved
Signed-off-by: svor <vsvydenk@redhat.com>
@svor svor requested a review from nickboldt as a code owner September 24, 2019 07:20
Copy link
Contributor

@amisevsk amisevsk left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @svor!

@svor svor requested a review from tolusha October 3, 2019 07:24
@svor svor merged commit 26951be into eclipse-che:master Oct 3, 2019
@svor svor deleted the sv/node_mongo branch October 3, 2019 14:50
monaka referenced this pull request in PizzaFactory/che-devfile-registry Nov 25, 2019
* Update NodeJS-MongoDB devfile to use new sample

Signed-off-by: svor <vsvydenk@redhat.com>

* Fix endpoint port

Signed-off-by: svor <vsvydenk@redhat.com>
monaka referenced this pull request in PizzaFactory/che-devfile-registry Nov 25, 2019
* Update NodeJS-MongoDB devfile to use new sample

Signed-off-by: svor <vsvydenk@redhat.com>

* Fix endpoint port

Signed-off-by: svor <vsvydenk@redhat.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants