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/super k8s #256

Closed
wants to merge 24 commits into from
Closed

feat/super k8s #256

wants to merge 24 commits into from

Conversation

cubxxw
Copy link
Contributor

@cubxxw cubxxw commented Nov 7, 2023

  • Adapting to k8s environment: modify service discovery and service registration logic,modify config file reading logic
  • Submit initial chart script
  • change chat script
  • fix bug:if env.descovery==k8s,not check zk
  • fix bug:Implement option interface
  • fix bug:change K8sDR.Register
  • change config.yaml
  • docs: add openim chat options
  • docs: add openim flag
  • docs: add openim version
  • docs: add openim version
  • docs: add openim version
  • docs: add openim version
  • update the config code
  • update the config code
  • fix:solve the conflict
  • fix: fix the flag parse error
  • fix:slove the script error
  • fix:alter the config.yaml
  • fix:alter the comfig.yaml
  • feat: add api test make file

lin.huang and others added 24 commits October 26, 2023 03:49
…istration logic,modify config file reading logic
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
Signed-off-by: Xinwei Xiong(cubxxw) <[email protected]>
@cubxxw cubxxw requested review from a team as code owners November 7, 2023 09:54
@pull-request-size pull-request-size bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Nov 7, 2023
Copy link

Whoa! Easy there, Partner!

This PR is too big. Please break it up into smaller PRs.

Copy link

sweep-ai bot commented Nov 7, 2023

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.

@cubxxw cubxxw closed this Nov 8, 2023
@openimsdk openimsdk locked and limited conversation to collaborators Nov 8, 2023
@cubxxw cubxxw deleted the feat/super-k8s branch January 15, 2024 11:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants