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

Update distributed-tracing.md #99

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

zhuhoudong
Copy link
Contributor

增加了一部分内容:在容器环境下如何搭建调用链跟踪。


   假设zipkin在虚拟机192.168.100.101的9411中运行。

    dockerfile:
Copy link
Contributor

Choose a reason for hiding this comment

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

对齐与67行对齐不一致。

EXPOSE 7781
ENTRYPOINT java $JAVA_OPTS -jar bmi-calculator-tracing.jar "-Ptracing"
```
打包image文件:
Copy link
Contributor

Choose a reason for hiding this comment

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

Find code fense here for "打包image文件" when using View option with this file.

bmi-calculator-0.3.0-SNAPSHOT.jar dockerfile
  root@i-wzmhsx68:/home/ubuntu/docker/bmi-calculator-tracing# docker build -t="bmi-calculator-tracing" .
```
运行docker容器:
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as line 67th.

用同样的方法打包镜像和运行*体质指数页面* 微服务。重新编写dokerfile,编译jar文件以及打包和运行docker镜像文件。
(注意将4.1中的bmi-calculator替换为bmi-webapp)。


体质指数应用中已配置好了上述配置项,您只需执行以下几步即可:
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this step 5?

用同样的方法打包镜像和运行*体质指数页面* 微服务。重新编写dokerfile,编译jar文件以及打包和运行docker镜像文件。
(注意将4.1中的bmi-calculator替换为bmi-webapp)。


体质指数应用中已配置好了上述配置项,您只需执行以下几步即可:

1. 使用 Docker 运行 *Zipkin* 分布式追踪服务:
Copy link
Contributor

Choose a reason for hiding this comment

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

Shoud we change these order format as the same as step 4.1?


    dockerfile:

```bash
Copy link
Contributor

Choose a reason for hiding this comment

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

代码缩进能否和下面的第五步中的保持一致?

@@ -48,6 +48,42 @@ last_modified_at: 2017-09-03T10:01:43-04:00
</dependency>
```

4. 在docker容器环境下运行 *体质指数计算器* 微服务和*体质指数界面* 微服务:
Copy link
Contributor

Choose a reason for hiding this comment

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

建议在"在docker容器"前面加上"(可选)"来提示用户这是可选步骤。

@zenlint
Copy link
Contributor

zenlint commented Sep 27, 2017

@zhuhoudong , we can discuss relative topic in Add guide for deploying BMI sample within docker(using docker swarm and kubernetes, etc). #104

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