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: expose mongo uri. #3586

Merged
merged 4 commits into from Jul 26, 2023
Merged

feat: expose mongo uri. #3586

merged 4 commits into from Jul 26, 2023

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Jul 26, 2023

πŸ€– Generated by Copilot at 9489259

Summary

πŸŒπŸ“πŸ—£οΈ

This pull request adds support for using an external mongodb database for sealos-cloud deployments. It introduces a new environment variable mongodbUri in the Kubefile and the init.sh script, and updates the README.md file to explain how to use it.

mongodbUri set
External or internal
Spring choice for cloud

Walkthrough

  • Add support for using an external mongodb by setting the mongodbUri environment variable (link, link, link)
  • Modify the Kubefile to accept the mongodbUri value and pass it to the init.sh script (link)
  • Update the README.md file to explain the new option and provide an example command (link)
  • Add a conditional message to the init.sh script to indicate whether the kubeblocks mongodb custom resource will be applied or not (link)

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Jul 26, 2023

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 914
βœ… Successful 347
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 566
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (59e7dd1) 67.92% compared to head (5432210) 67.92%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3586   +/-   ##
=======================================
  Coverage   67.92%   67.92%           
=======================================
  Files           8        8           
  Lines         664      664           
=======================================
  Hits          451      451           
  Misses        171      171           
  Partials       42       42           

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@bxy4543 bxy4543 merged commit f303c39 into labring:main Jul 26, 2023
33 of 34 checks passed
lingdie added a commit to lingdie/sealos that referenced this pull request Jul 26, 2023
* feat: expose mongo uri. (labring#3586)

* expose mongo uri env.

* feat: operation request -> rolebinding (sa <-> role)

* feat: operation request -> rolebinding (sa <-> role)

* feat: operation request

---------

Co-authored-by: yy <56745951+lingdie@users.noreply.github.com>
bxy4543 pushed a commit to bxy4543/sealos that referenced this pull request Jul 27, 2023
lingdie added a commit that referenced this pull request Aug 21, 2023
…ller. (#3595)

* User (#7)

* feat: expose mongo uri. (#3586)

* expose mongo uri env.

* feat: operation request -> rolebinding (sa <-> role)

* feat: operation request -> rolebinding (sa <-> role)

* feat: operation request

---------

Co-authored-by: yy <56745951+lingdie@users.noreply.github.com>

* init user controller for multi namespace feat.

* fix makefile.

* add samples

* init operation request webhook.

* init operationrequest webhook

* fix create rolebinding and log error.

* add samples.

* print status and other info.

* fix lint/makefile, add label for rolebinding

* fix crd.

* fix const.

---------

Co-authored-by: xy <135087223+Yurnero-Xy@users.noreply.github.com>
@lingdie lingdie deleted the cloud branch December 7, 2023 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants