Skip to content

Added container for deploying a basic version of OpenM++ web service.#117

Open
jacek-dudek wants to merge 1 commit intoStatCan:masterfrom
jacek-dudek:issue-1798
Open

Added container for deploying a basic version of OpenM++ web service.#117
jacek-dudek wants to merge 1 commit intoStatCan:masterfrom
jacek-dudek:issue-1798

Conversation

@jacek-dudek
Copy link
Copy Markdown
Contributor

I think those are all the uploads I need to do in this repository. Let me know otherwise.

@jacek-dudek jacek-dudek requested a review from a team as a code owner August 22, 2023 17:18
@Jose-Matsuda
Copy link
Copy Markdown
Contributor

so @jacek-dudek this is changing the old ompp image? there is no more need for it / this will replace it

@vexingly
Copy link
Copy Markdown
Contributor

so @jacek-dudek this is changing the old ompp image? there is no more need for it / this will replace it

The ompp image that I added is only for executing workloads using MPI, this image seems to be for hosting the web interface (which we currently include in the remote desktop image).

@jacek-dudek
Copy link
Copy Markdown
Contributor Author

Which of the images is not building, Pat's old image for running the shell session or the one I added?

@Jose-Matsuda
Copy link
Copy Markdown
Contributor

I think technically both, I think if you wanted to make a new image the directory structure should be different.
As in for a new image you need to have a distinct folder at the root as in
at this level
image

and not
image

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.

3 participants