Xiao Pan panxiao81
0 Followers · 0 Following
  • Joined on 2026-04-20
Repositories
9
 Projects Packages Public Activity Starred Repositories
panxiao81 created branch main in panxiao81/workload-sts 2026-09-11 15:44:35 +00:00
panxiao81 pushed to main at panxiao81/workload-sts 2026-09-11 15:44:35 +00:00
cc30a7cb6a 初始化项目文档与设计规范
panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:37:39 +00:00
Instance 阶段性基线：Ready 初版与领域设计

还是面向过程的面条

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:36:03 +00:00
Instance 阶段性基线：Ready 初版与领域设计

连接数据库本身为什么需要配置openbao？这个数据库凭据可能放在openbao，但是会有eso来同步secret，应用应当还是传统方式从env或配置文件来拿这个连接信息

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:32:18 +00:00
Instance 阶段性基线：Ready 初版与领域设计

按说initializer初始化比较后期，这时候校验openbao配置？这个配置校验不应该注入配置的时候就做好吗？说的比较复杂一点就是当配置无效这个配置类就不该被创建，然后fatal直接崩溃掉

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:29:16 +00:00
Instance 阶段性基线：Ready 初版与领域设计

我不理解为什么是initializer，这不是就是postgresqlinstance的domain对象吗？ddd呢？初始化一看就很面向过程啊

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:26:34 +00:00
Instance 阶段性基线：Ready 初版与领域设计

这不能说是坏习惯，但我想知道他什么时候会为null

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:23:47 +00:00
Instance 阶段性基线：Ready 初版与领域设计

虽然嘛返回result也很正常，但是这里要注意的是上面的代码做完了读取状态-》决定动作这一步，那么每一步内要做的事情剩下处理状态-〉进行状态转移，这是标准的状态机实现。

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 15:19:15 +00:00
Instance 阶段性基线：Ready 初版与领域设计

这里可读性差了点，稍微补一下注释

89d9e2316b refactor: consolidate Instance phase handlers
panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:20:37 +00:00
Instance 阶段性基线：Ready 初版与领域设计

同理

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:20:28 +00:00
Instance 阶段性基线：Ready 初版与领域设计

你告诉我加一条校验有效http url有什么意义

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:19:51 +00:00
Instance 阶段性基线：Ready 初版与领域设计

woc从这里往上是一串超长的面向过程面条代码，很多validate的逻辑应该都不要

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:17:28 +00:00
Instance 阶段性基线：Ready 初版与领域设计

你看你重新实现了一遍上面的Validate，你到底想要验证什么？

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:14:50 +00:00
Instance 阶段性基线：Ready 初版与领域设计

所以你这到底是instance的初始化代码放在这里还是把整个控制器的初始化代码扔这里了？

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:09:48 +00:00
Instance 阶段性基线：Ready 初版与领域设计

同上这个函数应该也是状态机吧

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:08:57 +00:00
Instance 阶段性基线：Ready 初版与领域设计

这是？这东西需要一个参数吗？

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:08:12 +00:00
Instance 阶段性基线：Ready 初版与领域设计

死的配置文件？绑定环境了？环境信息应该在ci注入吧

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:07:32 +00:00
Instance 阶段性基线：Ready 初版与领域设计

写一行这可读性有点差了

panxiao81 commented on pull request panxiao81/postgresql-tenant-operator#6 2026-09-11 14:07:07 +00:00
Instance 阶段性基线：Ready 初版与领域设计

话说log库是？