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

feat: 沙盒开发测试环境 #1657

Merged
merged 36 commits into from
Nov 1, 2024

Conversation

SheepSheepChen
Copy link
Collaborator

@SheepSheepChen SheepSheepChen commented Oct 12, 2024

整体来说就是将沙盒开发环境抽成了一个大的对象,并且对这个对象 crud 的过程
比较需要关注的点:

  1. 各种硬编码的配置(start_dir 等),那些需要放在 settings 里面,或者以其他方式放置
  2. 原来的 ingress 有问题,saas 的 path_prefix 为 "/", devserver 的为 "/devserver/",并且 saas 的路由不是由我们可以控制的,因此当 saas 应用带有路由 "/devserver/" 时就会有问题,现在进行了修改,但是可能会导致已存在的应用出问题?
  3. 现在 dev_sandbox 和 code_editor 公用一个 service 和 ingress,暂时来说端口是没有重合的,仔细想了下是不是不太合理,还是单独分开比较好?
  4. 开发框架读取增强服务等变量,还是依赖 BKPAAS_ENVIRONMENT 变量,现在注入的还是"dev", 注入"stag" 让用户会更方便,但是是否合理
    大致架构图如下:
image

@SheepSheepChen SheepSheepChen requested review from jamesgetx and narasux and removed request for jamesgetx and narasux October 12, 2024 08:52
@jamesgetx jamesgetx requested a review from piglei October 14, 2024 09:55
@SheepSheepChen SheepSheepChen merged commit 412f2c0 into TencentBlueKing:main Nov 1, 2024
7 checks passed
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.

4 participants