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: remove duplicated reset triggered map #642

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

shanyuyulaifengmanlou
Copy link
Contributor

What type of PR is this?

For QoS-Ensurance bug fixing.

What this PR does / why we need it:

Removing the duplicated action of reseting the triggered map would fix the bug of RESTORE IF-CONDITION(pkg/ensurance/analyzer/analyzer.go:295) never work.

Which issue(s) this PR fixes:

Fixes #637

Special notes for your reviewer:

Sincerely

@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2022

🎉 Successfully Build Images.
Now Support ARM Platforms.
Comment Post Time: 2022-12-07 03:21
Git Version: bf02a7d

Docker Registry

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

Image Pull Command
crane-agent:pr-642-bf02a7d docker pull gocrane/crane-agent:pr-642-bf02a7d
dashboard:pr-642-bf02a7d docker pull gocrane/dashboard:pr-642-bf02a7d
metric-adapter:pr-642-bf02a7d docker pull gocrane/metric-adapter:pr-642-bf02a7d
craned:pr-642-bf02a7d docker pull gocrane/craned:pr-642-bf02a7d

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-642-bf02a7d \
                   --set metricAdapter.image.repository=gocrane/metric-adapter \
                   --set metricAdapter.image.tag=pr-642-bf02a7d \
                   --set craneAgent.image.repository=gocrane/crane-agent \
                   --set craneAgent.image.tag=pr-642-bf02a7d \
                   --set cranedDashboard.image.repository=gocrane/dashboard \
                   --set cranedDashboard.image.tag=pr-642-bf02a7d crane/crane

Coding Registry

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

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

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-642-bf02a7d \
                   --set metricAdapter.image.repository=finops-docker.pkg.coding.net/gocrane/crane/metric-adapter \
                   --set metricAdapter.image.tag=pr-642-bf02a7d \
                   --set craneAgent.image.repository=finops-docker.pkg.coding.net/gocrane/crane/crane-agent \
                   --set craneAgent.image.tag=pr-642-bf02a7d \
                   --set cranedDashboard.image.repository=finops-docker.pkg.coding.net/gocrane/crane/dashboard \
                   --set cranedDashboard.image.tag=pr-642-bf02a7d crane/crane

Ghcr Registry

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

Image Pull Command
crane-agent:pr-642-bf02a7d docker pull ghcr.io/gocrane/crane/crane-agent:pr-642-bf02a7d
dashboard:pr-642-bf02a7d docker pull ghcr.io/gocrane/crane/dashboard:pr-642-bf02a7d
metric-adapter:pr-642-bf02a7d docker pull ghcr.io/gocrane/crane/metric-adapter:pr-642-bf02a7d
craned:pr-642-bf02a7d docker pull ghcr.io/gocrane/crane/craned:pr-642-bf02a7d

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-642-bf02a7d \
                   --set metricAdapter.image.repository=ghcr.io/gocrane/crane/metric-adapter \
                   --set metricAdapter.image.tag=pr-642-bf02a7d \
                   --set craneAgent.image.repository=ghcr.io/gocrane/crane/crane-agent \
                   --set craneAgent.image.tag=pr-642-bf02a7d \
                   --set cranedDashboard.image.repository=ghcr.io/gocrane/crane/dashboard \
                   --set cranedDashboard.image.tag=pr-642-bf02a7d crane/crane

Copy link
Contributor

@chenkaiyue chenkaiyue left a comment

Choose a reason for hiding this comment

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

lgtm

@chenkaiyue chenkaiyue merged commit 14a60a3 into gocrane:main Dec 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

QOS: ensurance analyzer: Reset the element of triggered map duplicately when compute the action context
2 participants