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

[venus-project] repo migration planning / repo 迁移计划 #5760

Closed
1 of 3 tasks
Tracked by #5758 ...
Fatman13 opened this issue Feb 24, 2023 · 20 comments
Closed
1 of 3 tasks
Tracked by #5758 ...

[venus-project] repo migration planning / repo 迁移计划 #5760

Fatman13 opened this issue Feb 24, 2023 · 20 comments
Labels
C-dev-productivity Category: Developer productivity C-ux-change Category: ux change that need attentions (CLI, config, database, folder structure) Epic

Comments

@Fatman13
Copy link
Contributor

Fatman13 commented Feb 24, 2023

背景

借着 venus 品牌重塑,以及 nv18 与 nv19 发版中间的机会,把所有 repo 彻底重新梳理/规划。

时间线

  • sprint 59 讨论决定迁移计划
  • 在这个 nv19 milestone 之前,完成迁移

规划

具体规划参见下方。 欢迎参与讨论。

其他关联

@Fatman13 Fatman13 mentioned this issue Feb 24, 2023
38 tasks
@Fatman13 Fatman13 added Epic C-dev-productivity Category: Developer productivity labels Feb 24, 2023
@Fatman13
Copy link
Contributor Author

cc @hunjixin review req

@hunjixin
Copy link
Contributor

hunjixin commented Feb 24, 2023

@Fatman13 麻烦按如下格式列出:

repo repo name data home repo name
venus-market venus-market .venus-market -> . droplet
venus-wallet .venus_wallet -> .?-wallet

原来的链服务名称不用变吗。

@Fatman13
Copy link
Contributor Author

Fatman13 commented Feb 27, 2023

Damocles, Droplet 迁移计划

repo before repo after CLI data home repo name
filecoin-project/venus-market /ipfs-force-community/droplet droplet, droplet-client .droplet .droplet-client
/ipfs-force-community/venus-cluster /ipfs-force-community/damocles damocles-manager, damocles-worker .damocles .damocles-worker

Sophon 迁移计划

另外,sophon考虑如下...

repo before repo after CLI data home repo name 备注
filecoin-project/venus-messager ipfs-force-community/sophon-messager sophon-messager .sophon-messager
filecoin-project/venus-miner ipfs-force-community/sophon-miner sophon-miner .sophon-miner
filecoin-project/venus-auth ipfs-force-community/sophon-auth sophon-auth .sophon-auth
ipfs-force-community/venus-gateway ipfs-force-community/sophon-gateway sophon-gateway .sophon-gateway
ipfs-force-community/chain-co ipfs-force-community/sophon-co sophon-co .sophon-co
filecoin-project/venus 不变 不变 不变
filecoin-project/venus-wallet 不变 不变 不变

这样的话,彻底和节点名称分开。sophon 产品可以通过 lotus 节点同步,也可以通过 venus 节点同步。sophon 服务包含 lotus + venus 节点,然后作为一个大的服务。另外把高可用 chain-co 也囊获进 sophon。

架构

image

考虑

🤔这样venus就一点体现不到了

venus 和 venus-wallet 就作为 venus 节点相关的。和 venus 其他产品完全拆分开。对 grant 申请,产品认知等等,有一定的好处。

@simlecode
Copy link
Collaborator

simlecode commented Feb 27, 2023

统一 repo :#5528

@hunjixin
Copy link
Contributor

本来也是要改的了

@hunjixin
Copy link
Contributor

🤔这样venus就一点体现不到了

@hunjixin
Copy link
Contributor

#5529

@Fatman13 Fatman13 added the C-ux-change Category: ux change that need attentions (CLI, config, database, folder structure) label Feb 27, 2023
@LinZexiao
Copy link
Collaborator

LinZexiao commented Mar 6, 2023

miner, messager, auth 作为名称主体, 挺简单直观的 ?

@Fatman13
Copy link
Contributor Author

Fatman13 commented Mar 10, 2023

sophon-messager, sophon-auth, sophon-miner, sophon-gateway

@Fatman13
Copy link
Contributor Author

规划结束。

@hunjixin
Copy link
Contributor

编译输出位置统一。 dist目录?

@Fatman13
Copy link
Contributor Author

编译输出位置统一。 dist目录?

可以,学习下 venus-cluster 的做法。

@0x5459
Copy link
Collaborator

0x5459 commented Mar 27, 2023

分支名称统一?
master -> main

See: https://github.com/github/renaming#why-main

@LinZexiao
Copy link
Collaborator

分支名称统一? master -> main

See: https://github.com/github/renaming#why-main

支持, 纯粹因为名字更短了一些, 另外 master/slave 这种命名习惯确实不讨喜

@elvin-du
Copy link
Collaborator

amgr,bmgr,mmgr,三个缩写的含义什么?和哪些repo对应?

@Fatman13
Copy link
Contributor Author

amgr,bmgr,mmgr,三个缩写的含义什么?和哪些repo对应?

架构图里面那个词忘记更新了。名称还是以表格里面的为准。那个图片不太好更新。主要体现下架构。

@elvin-du
Copy link
Collaborator

ipfs-force-community/sophon-bmgr sophon-miner .sophon-miner

这里的b是什么意思?

@Fatman13
Copy link
Contributor Author

这里的b是什么意思?

我的我的,没改。😂

@hunjixin
Copy link
Contributor

hunjixin commented Jun 5, 2023

repo迁移过程 数据目录原则上兼容两个目录一个大版本, 下个大版本已出(或者持续多个)。 最后需要给出一个迁移的注意事项和流程

@hunjixin
Copy link
Contributor

hunjixin commented Jun 5, 2023

#5808

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-dev-productivity Category: Developer productivity C-ux-change Category: ux change that need attentions (CLI, config, database, folder structure) Epic
Projects
Archived in project
Development

No branches or pull requests

6 participants