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.

Sign up for GitHub

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

Jump to bottom

[Features] Incorporate the config-manager sidecar into the lorry #7286

Open
kizuna-lek opened this issue May 9, 2024 · 2 comments
Open

[Features] Incorporate the config-manager sidecar into the lorry #7286

kizuna-lek opened this issue May 9, 2024 · 2 comments
Assignees
@xuriwuyun @kizuna-lek
Labels
area/configuration kind/feature New feature lorry

Comments

@kizuna-lek
Copy link
Collaborator

Incorporate the config-manager sidecar into the lorry.
@kizuna-lek kizuna-lek self-assigned this May 9, 2024
@shanshanying
Copy link
Contributor

KubeBlocks sidecars, such as lorry and config-manager, are injected in different transformation stages.
When HostNetWork is on, all sidecars should use ports allocated dynamically. Config-manager will look-up all containers to find a port named 'config-manger', which requires this port to be explicitly defined in cmpd.spec.hostNetwork.containerPort, and it is quite confusing to users.

@kizuna-lek kizuna-lek linked a pull request May 12, 2024 that will close this issue
support: incorporate config manager into lorry #7313
Closed
@kizuna-lek
Copy link
Collaborator Author

Continue after the lorry split is completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xuriwuyun xuriwuyun

@kizuna-lek kizuna-lek

Labels
area/configuration kind/feature New feature lorry
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

support: incorporate config manager into lorry apecloud/kubeblocks
3 participants
@xuriwuyun @kizuna-lek @shanshanying