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: add tedge-write to tedge's sudoers rule #66

Merged
merged 2 commits into from
Jan 31, 2025

Conversation

reubenmiller
Copy link
Contributor

@reubenmiller reubenmiller commented Jan 31, 2025

Add the tedge-write utility to the sudoers.d/tedge definition to allow the tedge-configuration-plugin to set the file ownership and permissions when writing a file to the container.

A system test was also added to cover this use-case in the future.

Copy link

github-actions bot commented Jan 31, 2025

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass % ⏱️ Duration
20 0 1 20 100 8m28.924733s

Passed Tests

Name ⏱️ Duration Suite
Ignore Containers Marked With A Specific Label 28.325 s Container
Grace period to allow container to startup 5.028 s Operations
Service is up 0.190 s Operations
Get Logfile Request 4.692 s Operations
Get Configuration File 5.039 s Operations
Set Configuration File 2.696 s Operations
Execute Shell Command 2.528 s Operations
Install application using docker compose 18.178 s Operations
Get Container Logs 4.691 s Operations
Get Container Logs without explicit container name 4.698 s Operations
Get Container Logs For Non-existent container 2.528 s Operations
Update From Legacy Versions 139.408 s Self-Update-Compat
Trigger self update via local command 25.417 s Self-Update
Self update should only update if there is a new image 20.948 s Self-Update
Self update using software update operation 103.890 s Self-Update
Rollback when trying to install a non-tedge based image 23.769 s Self-Update
Self update using software update operation using Container type 113.553 s Self-Update
Cloud Connection is Online 0.211 s Telemetry
Service status 0.378 s Telemetry
Sends measurements 2.732 s Telemetry

@reubenmiller reubenmiller added this pull request to the merge queue Jan 31, 2025
Merged via the queue into main with commit 9435430 Jan 31, 2025
8 checks passed
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.

1 participant