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: helm代理仓库支持页面可配 #2922 #2953

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lannoy0523
Copy link
Collaborator

No description provided.

@PathVariable projectId: String,
@PathVariable repoName: String
): Response<Void> {
helmClient.refreshIndexYamlAndPackage(projectId, repoName)
Copy link
Collaborator

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(
Copy link
Collaborator

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)
Copy link
Collaborator

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}")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个接口使用场景是? op数据量过大,可能会导致超时

@lannoy0523 lannoy0523 marked this pull request as draft February 24, 2025 08:13
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.

2 participants