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

Resource Region Mgr Simulator Containerization #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

q131172019
Copy link
Collaborator

This PR is to give two files to be used to build container image for resource region manager simulator.

  • docker build file : Dockerfile
  • executable file: resourceRegionMgrSimulator

The following steps to build image, run image have been tested on development EC2 running on AWS Ubuntu 1804
$ ~/go/src/global-resource-service/resource-management/test/resourceRegionMgrSimulator
$ docker build --tag docker-resource-region-mgr-simulator .
$ docker image ls
$ docker image tag docker-resource-region-mgr-simulator:latest docker-resource-region-mgr-simulator:v0.1
$ docker image ls
$ docker run --publish 9119:9119 docker-resource-region-mgr-simulator

OR Run in detached mode
$ docker run -d -p 9119:9119 docker-resource-region-mgr-simulator
$ docker ps

Also, using above docker image of simulator, plus server-api and redis to run integration test is successful on on development EC2 running on AWS Ubuntu 1804 as well.

@q131172019 q131172019 force-pushed the Carl_ResourceRegionMgrSimulator_Containerization branch from 285096f to 5f19b08 Compare June 23, 2022 01:11
@yb01 yb01 requested a review from sonyafenge June 28, 2022 17:57
@yb01
Copy link
Collaborator

yb01 commented Jun 28, 2022

@sonyafenge, please review this docker file for the region simulator.

FROM ubuntu:18.04

# LABEL about the custom image
LABEL maintainer="carl Xie <[email protected]>"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we really need add personal information here?

@sonyafenge
Copy link
Collaborator

It is uncommon to checkin bin file to source code.

@yb01
Copy link
Collaborator

yb01 commented Sep 20, 2022

put on hold for more dev resource on this effort.

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