← Back to issue list

fix(snap): update config dir in post-refresh hook

View original Github issue

Metadata

Project
fetch-service
Number
#171
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged a fix updating the ACL configuration file path in the snap post-refresh hook. Approved by three reviewers and passed CI checks.

Suggested action: —

No scores available.

Issue body

Point the acl configuration file to the correct location in the snap post-refresh hook. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged a fix updating the ACL configuration file path in the snap post-refresh hook. Approved by three reviewers and passed CI checks.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#112 snap: ensure smooth upgrade to acl-enabled version fetch-service merged Merged after two approvals and passing CI. Implements a post-refresh hook to generate missing ACL configuration files during snap upgrades, ensuring services start correctly when transitioning from non-ACL versions.
78%
#497 fix(config): warn on snapctl errors craft-application merged Merged a configuration fix that warns on snapctl errors. Approved by two reviewers, the change adds 32 lines across two files and addresses pending review feedback.
71%
#3974 snap config: catch more errors when config cannot be retrieved snapcraft merged Merged improvements to snap configuration error handling. Follows up on PR #3971 to catch additional retrieval failures reported by users. Approved by one reviewer, passed CI checks, and updated four files (+30/-10).
70%
#499 fix(config): wrap snap config handler in snap check craft-application merged Merged a fix wrapping the snap config handler in a snap environment check to prevent execution errors outside snaps. Approved by two reviewers, passed CI, and introduced 12 lines across two files.
70%