Skip to content

perf: optimize recommender manager#785

Merged
qmhu merged 1 commit intogocrane:mainfrom
lbbniu:perf/lbbniu/recommender
May 16, 2023
Merged

perf: optimize recommender manager#785
qmhu merged 1 commit intogocrane:mainfrom
lbbniu:perf/lbbniu/recommender

Conversation

@lbbniu
Copy link
Copy Markdown
Member

@lbbniu lbbniu commented May 10, 2023

What type of PR is this?

optimize recommender manager

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@lbbniu lbbniu force-pushed the perf/lbbniu/recommender branch 2 times, most recently from b1ddff6 to 2690101 Compare May 10, 2023 03:02
@lbbniu lbbniu changed the title perf: recommender manager perf: optimize recommender manager May 10, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 10, 2023

🎉 Successfully Build Images.
Now Support ARM Platforms.
Comment Post Time: 2023-05-15 15:43
Git Version: 5b59c71

Docker Registry

Overview: https://hub.docker.com/u/gocrane

Image Pull Command
crane-agent:pr-785-5b59c71 docker pull gocrane/crane-agent:pr-785-5b59c71
dashboard:pr-785-5b59c71 docker pull gocrane/dashboard:pr-785-5b59c71
metric-adapter:pr-785-5b59c71 docker pull gocrane/metric-adapter:pr-785-5b59c71
craned:pr-785-5b59c71 docker pull gocrane/craned:pr-785-5b59c71

Quick Deploy - Helm

helm repo add crane https://finops-helm.pkg.coding.net/gocrane/gocrane
helm install crane -n crane-system --create-namespace \
                   --set craned.image.repository=gocrane/craned \
                   --set craned.image.tag=pr-785-5b59c71 \
                   --set metricAdapter.image.repository=gocrane/metric-adapter \
                   --set metricAdapter.image.tag=pr-785-5b59c71 \
                   --set craneAgent.image.repository=gocrane/crane-agent \
                   --set craneAgent.image.tag=pr-785-5b59c71 \
                   --set cranedDashboard.image.repository=gocrane/dashboard \
                   --set cranedDashboard.image.tag=pr-785-5b59c71 crane/crane

Coding Registry

Overview: https://finops.coding.net/public-artifacts/gocrane/crane/packages

Image Pull Command
crane-agent:pr-785-5b59c71 docker pull finops-docker.pkg.coding.net/gocrane/crane/crane-agent:pr-785-5b59c71
dashboard:pr-785-5b59c71 docker pull finops-docker.pkg.coding.net/gocrane/crane/dashboard:pr-785-5b59c71
metric-adapter:pr-785-5b59c71 docker pull finops-docker.pkg.coding.net/gocrane/crane/metric-adapter:pr-785-5b59c71
craned:pr-785-5b59c71 docker pull finops-docker.pkg.coding.net/gocrane/crane/craned:pr-785-5b59c71

Quick Deploy - Helm

helm repo add crane https://finops-helm.pkg.coding.net/gocrane/gocrane
helm install crane -n crane-system --create-namespace \
                   --set craned.image.repository=finops-docker.pkg.coding.net/gocrane/crane/craned \
                   --set craned.image.tag=pr-785-5b59c71 \
                   --set metricAdapter.image.repository=finops-docker.pkg.coding.net/gocrane/crane/metric-adapter \
                   --set metricAdapter.image.tag=pr-785-5b59c71 \
                   --set craneAgent.image.repository=finops-docker.pkg.coding.net/gocrane/crane/crane-agent \
                   --set craneAgent.image.tag=pr-785-5b59c71 \
                   --set cranedDashboard.image.repository=finops-docker.pkg.coding.net/gocrane/crane/dashboard \
                   --set cranedDashboard.image.tag=pr-785-5b59c71 crane/crane

Ghcr Registry

Overview: https://github.com/orgs/gocrane/packages?repo_name=crane

Image Pull Command
crane-agent:pr-785-5b59c71 docker pull ghcr.io/gocrane/crane/crane-agent:pr-785-5b59c71
dashboard:pr-785-5b59c71 docker pull ghcr.io/gocrane/crane/dashboard:pr-785-5b59c71
metric-adapter:pr-785-5b59c71 docker pull ghcr.io/gocrane/crane/metric-adapter:pr-785-5b59c71
craned:pr-785-5b59c71 docker pull ghcr.io/gocrane/crane/craned:pr-785-5b59c71

Quick Deploy - Helm

helm repo add crane https://finops-helm.pkg.coding.net/gocrane/gocrane
helm install crane -n crane-system --create-namespace \
                   --set craned.image.repository=ghcr.io/gocrane/crane/craned \
                   --set craned.image.tag=pr-785-5b59c71 \
                   --set metricAdapter.image.repository=ghcr.io/gocrane/crane/metric-adapter \
                   --set metricAdapter.image.tag=pr-785-5b59c71 \
                   --set craneAgent.image.repository=ghcr.io/gocrane/crane/crane-agent \
                   --set craneAgent.image.tag=pr-785-5b59c71 \
                   --set cranedDashboard.image.repository=ghcr.io/gocrane/crane/dashboard \
                   --set cranedDashboard.image.tag=pr-785-5b59c71 crane/crane

@qmhu qmhu added the bootcamp label May 11, 2023
@lbbniu lbbniu force-pushed the perf/lbbniu/recommender branch 4 times, most recently from f9ec896 to c3dd2af Compare May 12, 2023 03:37
@lbbniu lbbniu force-pushed the perf/lbbniu/recommender branch from c3dd2af to 5b59c71 Compare May 15, 2023 07:09

return &HPARecommender{
*replicasRecommender,
*(replicasRecommender.(*replicas.ReplicasRecommender)),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's this line means?

Copy link
Copy Markdown
Member Author

@lbbniu lbbniu May 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replicas.NewReplicasRecommender return recommender.Recommender interface

@qmhu qmhu merged commit e5bcb70 into gocrane:main May 16, 2023
@lbbniu lbbniu deleted the perf/lbbniu/recommender branch May 16, 2023 06:39
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.

2 participants