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

Server updates #17

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

rgallor
Copy link
Collaborator

@rgallor rgallor commented Nov 27, 2024

Give the possibility to pause/resume the stream and change the math function, the scale and the interval between 2 samples by handling server-owned datastream data.

Closes #4

@rgallor rgallor force-pushed the feat/server-updates branch from 350eacc to 08a76c4 Compare November 27, 2024 11:34
Copy link

codecov bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 0% with 51 lines in your changes missing coverage. Please review.

Project coverage is 0.0%. Comparing base (7db1595) to head (3630da0).

Files with missing lines Patch % Lines
src/config.rs 0.0% 27 Missing ⚠️
src/math.rs 0.0% 12 Missing ⚠️
src/shutdown.rs 0.0% 8 Missing ⚠️
src/lib.rs 0.0% 4 Missing ⚠️
Additional details and impacted files
Files with missing lines Coverage Δ
src/astarte.rs 0.0% <ø> (-8.8%) ⬇️
src/lib.rs 0.0% <0.0%> (ø)
src/shutdown.rs 0.0% <0.0%> (ø)
src/math.rs 0.0% <0.0%> (ø)
src/config.rs 0.0% <0.0%> (ø)

@rgallor rgallor marked this pull request as draft November 29, 2024 09:27
@rgallor rgallor force-pushed the feat/server-updates branch 2 times, most recently from 396cda2 to 6d4e15c Compare November 29, 2024 15:21
@rgallor rgallor marked this pull request as ready for review November 29, 2024 15:22
@rgallor rgallor requested review from joshuachp, harlem88 and lucaato and removed request for joshuachp November 29, 2024 15:22
@rgallor
Copy link
Collaborator Author

rgallor commented Nov 29, 2024

At the moment the org.astarte-platform.genericsensors.Values interface has a major version 1 and minor 0. Instead, the org.astarte-platform.genericcommands.ServerCommands has major 0 and minor 2. I'm writing it down here to remember changing the version before merging this PR into master

@rgallor rgallor force-pushed the feat/server-updates branch 2 times, most recently from fef4f87 to 0bb15d6 Compare December 2, 2024 11:55
README.md Show resolved Hide resolved
@rgallor rgallor force-pushed the feat/server-updates branch from 0bb15d6 to b6532e5 Compare December 3, 2024 14:13
In this commit:
- Define scripts, env file and Dockerfile to build and run the container
- Get astarte config from env vars
- Add dependabot checks for docker
- Update Readme with the information to build and run a docker container

Signed-off-by: Riccardo Gallo <[email protected]>
Gracefully shut down all the active tasks when SIGINT or SIGTERM
signals are received.

Signed-off-by: Riccardo Gallo <[email protected]>
Update the math function, the scale and the interval between two samples
through a server-owned datastream interface

Signed-off-by: Riccardo Gallo <[email protected]>
Now when the stream is paused/resumed the select! macro will not
continue looping at every sleep timeout, but it simply disables the
branch meant to send data to Astarte

Signed-off-by: Riccardo Gallo <[email protected]>
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.

Support server commands to pause/resume and change streaming parameters
2 participants