-
Notifications
You must be signed in to change notification settings - Fork 39
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: helm代理仓库支持页面可配 #2922 #2953
base: master
Are you sure you want to change the base?
Conversation
5372230
to
6f7d428
Compare
876323f
to
79a0d5b
Compare
@PathVariable projectId: String, | ||
@PathVariable repoName: String | ||
): Response<Void> { | ||
helmClient.refreshIndexYamlAndPackage(projectId, repoName) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里为什么还是feign调用
logger.info("Helm Remote event ${getFullResourceKey()} completed.") | ||
} | ||
} catch (exception: Exception) { | ||
if (event.type == EventType.REPO_REFRESHED) { | ||
val operateLog = OperateLog( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
代码重复了,优化下
val httpEntity = HttpEntity<Any>(headers) | ||
// 暂时只添加了helm类型的校验 | ||
val response = restTemplate.exchange( | ||
url + "/index.yaml", HttpMethod.GET, httpEntity, String::class.java) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
判断连通性只需head请求即可
} | ||
|
||
@ApiOperation("获取最近一次的同步时间和状态") | ||
@GetMapping("/getLatestSyncStatus/{projectId}/{repoName}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个接口使用场景是? op数据量过大,可能会导致超时
No description provided.