Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Docker] Updated Dockerfile's #1

Merged
merged 23 commits into from
Feb 28, 2024
Merged

Conversation

OfficialR3ido101
Copy link

This PR will allow us to run the up to date code for Overte with the node version provided by VCPKG

Before this is merged I would recommend that @daleglass has a runner with a tag of self-hosted.

@OfficialR3ido101
Copy link
Author

We have tested it and it works

@ksuprynowicz ksuprynowicz added bug Something isn't working needs CR This pull request needs to be code reviewed labels Feb 11, 2024
.github/workflows/deploy.yaml Outdated Show resolved Hide resolved
Dockerfile.runtime Outdated Show resolved Hide resolved
OfficialR3ido101 referenced this pull request in terminalx-resonite/overte-domain-server-docker Feb 25, 2024
* trying to remove root #1

* tesitng

* de-rooting 2 test

* de-root test #2

* de-root #3

* added pyton3 de-root #4

* hardcode path de-root overte-org#5

* forced to only master de-root overte-org#5

* removed env arguments that are not needed.

* temp check on dale's repo de-root overte-org#6

* forgot space

* de-rooting

* de-root

* de-root

* de-root overte-org#8

* removed builder image and get it ready for pr

* removed unneeded stuff

* Will this this to run

* prepping the runtime dockerfile for overte-org
@OfficialR3ido101
Copy link
Author

OfficialR3ido101 commented Feb 27, 2024

I have found a bug regarding ports not binding I will have a look at some point

Update: I did not forward the ports when running docker run

@daleglass daleglass merged commit dd32c4b into overte-org:main Feb 28, 2024
daleglass pushed a commit that referenced this pull request Feb 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working needs CR This pull request needs to be code reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants