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

fix: mogodb url connent #1232

Merged
merged 5 commits into from
Oct 19, 2023
Merged

fix: mogodb url connent #1232

merged 5 commits into from
Oct 19, 2023

Conversation

originnets
Copy link
Contributor

@originnets originnets commented Oct 16, 2023


🔍 What type of PR is this?

/kind bug

👀 What this PR does / why we need it:

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

🅰 Which issue(s) this PR fixes:

Fixes #1233

@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 16, 2023
@kubbot
Copy link
Contributor

kubbot commented Oct 16, 2023

CLA Assistant Lite bot 🤖 All Contributors have signed the open-im-server CLA.
The signed information is recorded 🤖here

@sweep-ai
Copy link
Contributor

sweep-ai bot commented Oct 16, 2023

Apply Sweep Rules to your PR?

  • Apply: Any clearly inefficient or repeated code should be optimized or refactored.

@originnets
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@cubxxw
Copy link
Contributor

cubxxw commented Oct 16, 2023

Please use issue to drive the PR. You need to raise an issue first.

@originnets
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

#1233

@cubxxw
Copy link
Contributor

cubxxw commented Oct 16, 2023

Hello friend, it seems that the CI test has failed. May I kindly ask if you have any insight on the possible reasons for this? Thank you in advance for your help.

auto-merge was automatically disabled October 16, 2023 05:46

Head branch was pushed to by a user without write access

@pull-request-size pull-request-size bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 16, 2023
@kubbot
Copy link
Contributor

kubbot commented Oct 16, 2023

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


Hello friend, it looks like the CI test failed. Do you know the possible reasons for this? Thanks in advance for your help.

fixed

@originnets
Copy link
Contributor Author

Hello friend, it seems that the CI test has failed. May I kindly ask if you have any insight on the possible reasons for this? Thank you in advance for your help.

fixed

cubxxw
cubxxw previously approved these changes Oct 17, 2023
@originnets originnets temporarily deployed to openim October 18, 2023 13:20 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 18, 2023 13:21 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 18, 2023 13:21 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 18, 2023 13:21 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 18, 2023 13:21 — with GitHub Actions Inactive
@cubxxw
Copy link
Contributor

cubxxw commented Oct 18, 2023

Hi, thank you for your contribution and dedication. I have noticed that the CI for testing the local startup is not passing, indicating that the local usage of make build; make start has failed. I have also tested it using your remote repository and encountered the same issue. Perhaps you could try testing it locally.

# command-line-arguments
tools/component/component.go:38:2: "go.mongodb.org/mongo-driver/mongo/readpref" imported and not used
chmod: cannot access '/root/workspace/open-im-server/_output/bin/tools/linux/amd64/component': No such file or directory
make[3]: *** [scripts/make-rules/golang.mk:149: go.build.linux_amd64.component] Error 1
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [Makefile:84: build] Error 2
!!! Error in /root/workspace/open-im-server/scripts/build-all-service.sh:65 
  Error in /root/workspace/open-im-server/scripts/build-all-service.sh:65. 'make --no-print-directory -C "${OPENIM_ROOT}" -j$compile_count build' exited with status 2
Call stack:
  1: /root/workspace/open-im-server/scripts/build-all-service.sh:65 main(...)
Exiting with status 1
You need to start the following scripts in order: openim-api.sh openim-rpc.sh openim-push.sh openim-msgtransfer.sh openim-msggateway.sh openim-crontask.sh openim-tools.sh

## Pre Starting OpenIM services
Preparing to start OpenIM Tools...
Starting ncpu...
Starting PATH: /root/workspace/open-im-server/_output/bin/tools/linux/amd64/ncpu...
Specifying config: /root/workspace/open-im-server/config
Starting ncpu...
4Starting component...
Starting PATH: /root/workspace/open-im-server/_output/bin/tools/linux/amd64/component...
Specifying config: /root/workspace/open-im-server/config
Starting component...
/root/workspace/open-im-server/scripts/install//openim-tools.sh: line 106: /root/workspace/open-im-server/_output/bin/tools/linux/amd64/component: No such file or directory
!!! [1018 21:28:50] Call tree:
!!! [1018 21:28:50]  1: /root/workspace/open-im-server/scripts/install//openim-tools.sh:123 openim::tools::start_service(...)
!!! [1018 21:28:50]  2: /root/workspace/open-im-server/scripts/install//openim-tools.sh:166 openim::tools::pre-start(...)
!!! Error in /root/workspace/open-im-server/scripts/install//openim-tools.sh:106 
  Error in /root/workspace/open-im-server/scripts/install//openim-tools.sh:106. '((i<4-1))' exited with status 127
Call stack:
  1: /root/workspace/open-im-server/scripts/install//openim-tools.sh:106 openim::tools::start_service(...)
  2: /root/workspace/open-im-server/scripts/install//openim-tools.sh:123 openim::tools::pre-start(...)
  3: /root/workspace/open-im-server/scripts/install//openim-tools.sh:166 main(...)
Exiting with status 1
!!! Error in /root/workspace/open-im-server/scripts/start-all.sh:75 
  Error in /root/workspace/open-im-server/scripts/start-all.sh:75. '${TOOLS_START_SCRIPTS_PATH} openim::tools::pre-start' exited with status 1
Call stack:
  1: /root/workspace/open-im-server/scripts/start-all.sh:75 main(...)
Exiting with status 1
make[1]: *** [scripts/make-rules/golang.mk:108: go.start] Error 1
make: *** [Makefile:89: start] Error 2

@originnets

@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 03:35 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@originnets originnets temporarily deployed to openim October 19, 2023 04:02 — with GitHub Actions Inactive
@cubxxw cubxxw enabled auto-merge October 19, 2023 11:16
@cubxxw cubxxw added this pull request to the merge queue Oct 19, 2023
Merged via the queue into openimsdk:main with commit 4b23171 Oct 19, 2023
17 checks passed
@github-actions github-actions bot added this to the v3.1 milestone Oct 19, 2023
@openimsdk openimsdk locked and limited conversation to collaborators Oct 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Mongodb connection config.yaml URL error in component.go
4 participants