Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

feat: support multiple services #10

Merged
merged 15 commits into from
Mar 11, 2024
Merged

Conversation

YkeLit
Copy link
Collaborator

@YkeLit YkeLit commented Mar 7, 2024

  • Support multiple services
  • Warning about migration of a single service port to multiple service ports
  • Change appIngressroute to appingressroutes
  • Update linter, add unit test
  • Support helm status get access address

@YkeLit YkeLit requested a review from fahchen March 7, 2024 16:32
Copy link

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
values.yaml Outdated Show resolved Hide resolved
@YkeLit YkeLit requested a review from fahchen March 8, 2024 11:14
Chart.yaml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
templates/deployment.yaml Show resolved Hide resolved
templates/service.yaml Show resolved Hide resolved
templates/traefik/ingress-route.yaml Show resolved Hide resolved
@YkeLit YkeLit requested a review from fahchen March 10, 2024 12:45
Copy link
Member

@fahchen fahchen left a comment

Choose a reason for hiding this comment

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

全部有兼容处理的地方都做一下 todo,另外版本那个可以先不做大版本升级,我们现在还兼容了之前的,还不算 break change,等我们把兼容代码去除掉之后,就可以做大版本升级了

所以 todo 里面的描述可以写成:1. 升级成什么,2. 在 2.0.0 后会被移除支持

@YkeLit YkeLit requested a review from fahchen March 10, 2024 14:50
@YkeLit YkeLit enabled auto-merge (squash) March 11, 2024 01:47
@YkeLit YkeLit merged commit e6fb782 into main Mar 11, 2024
1 check passed
@YkeLit YkeLit deleted the feat/yangke/support-multiple-services branch March 11, 2024 01:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants