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

add user env #3469

Merged
merged 2 commits into from
Jul 6, 2023
Merged

add user env #3469

merged 2 commits into from
Jul 6, 2023

Conversation

lingdie
Copy link
Collaborator

@lingdie lingdie commented Jul 6, 2023

🤖 Generated by Copilot at 084c691

Summary

🌐🛂💰

This pull request adds or modifies environment variables in the Kubefiles and manifests of the user controller and the costcenter provider, to make them more configurable and consistent with the sealos cloud API server settings. It also fixes the icon URL of the costcenter appcr.

Kubefile changes
Configuring sealos cloud
Autumn of features

Walkthrough

  • Add and interpolate environment variables for sealos cloud domain and port in user controller Kubefile and deployment manifest (link, link)
  • Pass environment variables from user controller Kubefile to sealos run command in init script (link)
  • Add and interpolate environment variables for transfer and recharge features in costcenter provider Kubefile and deployment manifest (link, link)
  • Replace hard-coded icon URL with relative path in costcenter provider appcr manifest (link)

@sealos-ci-robot
Copy link
Member

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 844
✅ Successful 322
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 521
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Jul 6, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (d73cb20) 72.97% compared to head (084c691) 72.97%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3469   +/-   ##
=======================================
  Coverage   72.97%   72.97%           
=======================================
  Files           8        8           
  Lines         618      618           
=======================================
  Hits          451      451           
  Misses        132      132           
  Partials       35       35           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@lingdie lingdie added this to the v5.0 milestone Jul 6, 2023
@lingdie lingdie merged commit 22b0d6c into labring:main Jul 6, 2023
87 checks passed
@lingdie lingdie deleted the cost-center branch July 13, 2023 09:03
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.

3 participants