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

fix:add sidecar #191

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

Conversation

lizhipeng629
Copy link
Contributor

增加sidecar解决serverless场景下问题

  • 引入一个sidecar,其中包含多个plugin
  • 可扩展的plugin机制
  • 现有服务质量探测和热更新两个plugin

<img src="../images/probe-struct.png" width="90%"/>
</p>

- 运行过程:
Copy link
Contributor

Choose a reason for hiding this comment

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

这里复制的热更新的运行过程,需改成服务质量探测的过程

resources:
CPU: 100m
Memory: 128Mi
sidecarstartorder: Before ## Sidecar 的启动顺序,是在主容器之后还是之前
Copy link
Contributor

Choose a reason for hiding this comment

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

驼峰变量:sidecarstartorder --> sidecarStartOrder

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

resources:
CPU: 100m
Memory: 128Mi
sidecarstartorder: Before ## Sidecar 的启动顺序,是在主容器之后还是之前
Copy link
Contributor

Choose a reason for hiding this comment

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

驼峰变量:sidecarstartorder --> sidecarStartOrder

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

- plugin运行过程
- 根据设置的延迟启动时间,延迟启动
- 根据设置的探测地址进行循环探测,循环探测的间隔从config中获取;
- 读取探测地址中的http methor/url/header
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: methor --> method

- 读取探测地址中的http methor/url/header
- 对指定地址探测
- 获取探测结果
- 根据探测结果,修改gameserver的spec.opsState的状态,具体规则在configmap中定义;
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的探测结果具体是指什么?body内容?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

是的

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.

2 participants