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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(snap): ensure snapd restarts service if the agent crashes #2215

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

jnsgruk
Copy link
Contributor

@jnsgruk jnsgruk commented Nov 8, 2023

Why?

If a user starts Parca Agent with snap start parca-agent and the agent crashes, currently it stays dormant.

What?

A simple change to the systemd configuration for the snap daemon

How?

馃 Generated by Copilot at dc2bd8e

  • Change the restart condition of the parca-agent service to always (link). This improves the reliability and availability of the agent, which collects and sends profiling data to the Parca server. The change was discussed and agreed upon in issue Bump k8s.io/apimachinery from 0.20.6 to 0.22.2聽#62.

Test Plan

@jnsgruk jnsgruk requested a review from a team as a code owner November 8, 2023 16:24
Copy link
Member

@kakkoyun kakkoyun left a comment

Choose a reason for hiding this comment

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

LGTM
馃殌 馃殌 馃殌

@kakkoyun kakkoyun merged commit ae92988 into parca-dev:main Nov 8, 2023
23 checks passed
@jnsgruk jnsgruk deleted the patch-1 branch November 8, 2023 20:41
@javierhonduco
Copy link
Contributor

Thanks for the change!! Please do let us know if you have experienced any crashes in your environment. Feel free to open issues at assign them to me

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.

None yet

3 participants