Support stage yaml #13
Conversation
|
依赖这个: laincloud/lain-sdk#8 |
|
@xtao 构建 meta 镜像的时候，现在应该还是会把 lain.yaml 拷到镜像里面而不是拷贝 lain.STAGE.yaml 吧 |
|
@panli889 改了 SDK 了 |
lain_cli/appversion.py
Outdated
| check_phase(phase) | ||
| yml = lain_yaml(ignore_prepare=True) | ||
| stage = get_phase_stage(phase) | ||
| yml = lain_yaml(ignore_prepare=True, stage) |
There was a problem hiding this comment.
我改一下这个，有很多地方还没改全，这个 stage yaml 的 feature 准备合入主干么?
There was a problem hiding this comment.
我觉得可以合啊，挺好的，正好有些app现在部署还要修改 lain.yaml 才行。。
lain_cli/build.py
Outdated
| @arg('--release', help="build from build image if it exists") | ||
| @arg('--push', help="tag release and meta image with version and push to registry") | ||
| def build(push=False, release=False): | ||
| @arg('phase', help="lain cluster phase id, can be added by lain config save") |
There was a problem hiding this comment.
@panli889 有个问题，就是 --phase 支持 lain build 和 lain build local 么？
还是说需要 lain build --phase local ?
There was a problem hiding this comment.
应该是需要 lain build --phase local，你是想弄成哪种？
There was a problem hiding this comment.
@panli889 期望不改变原来的行为，并且自然的支持。期望是 支持 lain build 和 lain build local，不期望加个 --phase。我刚才试了一下，--phase 需要输入 --phase，我研究一下怎么改..
|
@xtao 使用方式是 |
|
@iwinux |
|
@xtao 这个应该已经 ok 了吧，我觉得可以 merge 了 |
|
@panli889 lain deploy stage 的时候，还有个 bug，我还没有修 |
|
lain deploy stage 的 bug 修复了吗？ @xtao |
5 participants
说明:
在 lain config 里面设置 stage 变量，lain build PHASE 的时候，会读取这个 stage 的值，载入对应的 lain.STAGE.yaml 的配置。同理: lain tag PHASE, lain push PHASE, lain deploy PHASE