thin-edge / thin-edge.io

The open edge framework for lightweight IoT devices
https://thin-edge.io
Apache License 2.0
222 stars 55 forks source link

Permission issue when updating / removing an Apama project #1052

Closed abelikt closed 1 year ago

abelikt commented 2 years ago

Describe the bug

A sequence of Apama project installations leads to a permission issue upon project removal.

Removing existing project at /etc/tedge/apama/project
ERROR: Directory not empty (os error 39)

To Reproduce

... see also screenshot below:

/usr/bin/sudo /etc/tedge/sm-plugins/apama install LimitedBandwidth::project --file /home/pi/actions-runner/_work/thin-edge.io/thin-edge.io/tests/PySys/plugin_apama/Input/limitedbandwidth.zip

/usr/bin/sudo /etc/tedge/sm-plugins/apama install LimitedBandwidth::project --file /home/pi/actions-runner/_work/thin-edge.io/thin-edge.io/tests/PySys/plugin_apama/Input/quickstart.zip

/usr/bin/sudo /etc/tedge/sm-plugins/apama install LimitedBandwidth::project --file /home/pi/actions-runner/_work/thin-edge.io/thin-edge.io/tests/PySys/plugin_apama/Input/quickstart.zip

or run the system test:

pysys.py run -v DEBUG "apama_plugin_update" -XmyPlatform=container

Expected behavior

works :)

Screenshots

image

Environment (please complete the following information):

Additional context

context

rina23q commented 2 years ago

I feel the test steps are wrong.

rina23q commented 1 year ago

@reubenmiller Shall we close this issue as tedge_apama_plugin is now supported by Apama Team? Regarding https://github.com/thin-edge/thin-edge.io/issues/1584

reubenmiller commented 1 year ago

Yes I think we can park this issue for the moment, and close it once we get confirmation about Apama plugin ownership transition.

reubenmiller commented 1 year ago

Closing as the APAMA team has taken ownership of the plugin.