markzhang0928 comments

Results 14 comments of


                                            markzhang0928

Problem with relationship in SQLmodel and fastapi

https://github.com/tiangolo/sqlmodel/issues/383#issuecomment-1193373160

Have svn return something from `Branches()` and `Tags()`

Calling svn without an appropriate golang sdk is really frustrated.

Add proposal doc for accelerate replicate speed between Harbor and the target cr

> @wy65701436 @bitsf > > Let's have a look at this proposal if it can improve the perf of replication that is the focus of post 2.0 release. @steven-zou Much...

Add proposal doc for accelerate replicate speed between Harbor and the target cr

> hi @markzhang0928 > 不好意思我没有太看明天你的这个proposal，你这里提到的 ui model 是指的我们 harbor 的portal ui模块，还是 nginx proxy模块，还是一个你提议单独实现的用于拦截upload blob request 的新模块？ > 你说的用于加速实际复制传输数据的操作是发生在什么时候你好，@bitsf 目前修改是在harbor-ui 模块，`src/ui/proxy/proxy.go` 这个文件。以下是一些关键的代码片段。用于加速实际复制传输数据的操作，是在harbor-ui ReverseProxy拿到每次HEAD请求的返回，并且判断为200时。 ```go func (bt *ProxyBlobsTransfer) RoundTrip(req *http.Request)...

Add proposal doc for accelerate replicate speed between Harbor and the target cr

> 你这个用的是哪个版本的harbor？看起来是几年前的代码了。 > 我们现在没有src/ui 这个模块 > 我们现在的架构是 > nginx -> portal > -> core -> harbor api > -> registry api > > 我估计你的代码对应当前master 的代码应该是 > https://github.com/goharbor/harbor/blob/master/src/server/registry/proxy.go#L33 是的。不好意思，我忽略了注明版本。目前的流程对应是在您发的这个`src/server/registry/proxy.go`文件. > 我理解你的这个提议目的是说在用户...

Add proposal doc for accelerate replicate speed between Harbor and the target cr

> 个人感觉这个改动和我们当前的架构设计差异比较大，会破坏设计模式，不适合直接在代码库中增加这样的修改。 > 而且这个改动还需要考虑安全、调度、缓存、日志、阀值、异常处理等方面的需求，开发维护工作量都较大。 > 如果对于内部使用场景确实有比较大的性能提升，我建议找一些其他的方式加载此实现，比如能够以单独的插件配置进来，比如修改nginx再配置一个额外的代理专门做这个 > > @steven-zou @ywk253100 What do you think? 异地镜像复制本身确实需要考虑比较多问题，原本的jobservice模块也做得十分健壮。但是考虑到是用户开关控制的加速功能，针对两个内网可达的harbor实例场景，提前开始镜像层的复制，确实能有比较好的加速效果，缩短3-5倍镜像复制的时间窗口。如果habor 原生的release 可以支持该可选功能，也是一种旁路支持的加速方式，且开箱即用，并不影响原本的镜像复制流程。不需要用户进行额外的开发和配置，对有多region的harbor用户，提供了实在的便捷。

Add proposal doc for accelerate replicate speed between Harbor and the target cr

> 个人感觉这个改动和我们当前的架构设计差异比较大，会破坏设计模式，不适合直接在代码库中增加这样的修改。 > 而且这个改动还需要考虑安全、调度、缓存、日志、阀值、异常处理等方面的需求，开发维护工作量都较大。 > 如果对于内部使用场景确实有比较大的性能提升，我建议找一些其他的方式加载此实现，比如能够以单独的插件配置进来，比如修改nginx再配置一个额外的代理专门做这个 > > @steven-zou @ywk253100 What do you think? 内部测试统计，同步一个大小1.05GB，11层的镜像，加速之后跨域同步的时间从原本的61s缩短到17s. ```shell SH-harbor同步到SZ-harbor SH-node push到SH-harbor，镜像大小1.05GB，十一层 (1)13:36:06，执行docker push reg.harbor154-shanghai.com/tmp1/name2:v2 (2)13:36:22，PUT /v2/tmp1/name2/manifests/v2 HTTP/1.1 docker push用时 16s (3)13:36:24，（SH-harbor）开始下载镜像，192.168.192.1...