Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Fixing Replay Feeder #117

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Conversation

erikbosch
Copy link
Contributor

I do not know how much effort we want to put into this component, but it was definitely broken.

What I did:

  • Created a log file with KUKSA.val Server (now included as example)
  • Making sure that it can be used to inject data back to KUKSA.val Server.

If we want to support this component in the long run we should better create some tests, but that is not the intention of this PR

@@ -3,12 +3,13 @@
ip=127.0.0.1
port=8090
insecure=false
token=../../kuksa_certificates/jwt/all-read-write.json.token
token=../../kuksa.val/kuksa_certificates/jwt/all-read-write.json.token
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I changed the example/default config so that it works if you have kuksa.val.feeders and kuksa.val cloned in parallel.

@SebastianSchildt
Copy link
Contributor

I am not sure we want to keep this component in the long run. For playing -sans the "re" we have https://github.com/eclipse/kuksa.val.feeders/tree/main/csv_provider now, and I wonder it won't be better to have component that could create a compatible CSV log, basically "KUKSA recorder"m using Py API only, to effectively get the same behavior

I think @eriksven, you looked into it, but not 100% sure.Where there fundamental hurdles except "only 24 hours in a day"?

This tool originally was really a val-server only thing, depending on a "hacked" version of an internal class, which means for one, currently there is no way to create such logs for databroker, and second not sure this "deeply integrated" way was the best architecture pattern to begin with...

Other opinions?

@SebastianSchildt
Copy link
Contributor

Oops just noticed #105 Should look at it :)

I sort of feel, if that works, this whole thing can be removed.

@erikbosch
Copy link
Contributor Author

The other tools ( csv_provider and #105) (AFAIK) only supports KUKSA.val Databroker, while this one only supports KUKSA.val Server.

My view is:

  • In the long run we should not have different tools for Databroker/Server, i.e. the Replay Feeder shall in the long run be removed and replaced by the others.
  • But then it comes to the question on how we look at the future of KUKSA.val server. If we think that we will deprecate KUKSA.val Server "soon" it is not worth the effort to add websocket-functionality to the other tools, but then maybe keep this tool alive until KUKSA.val Server is obsolete
  • If we decide to use the recorder/provider concept for KUKSA.val Server as well (should be doable), do we then still want to keep the logging concept in KUKSA.val Server (i.e. the format consumed by this tool)? This question is however only relevant if we intend to keep KUKSA.val Server for a reasonably long time.

For the near future we could do like this:

  1. Merge this PR, it does not hurt regardless of how we will continue
  2. Create an issue to handle the decision if we should keep this feeder or do the effort of making the others support websocket as well.

@erikbosch erikbosch changed the title Fixing Reply Feeder Fixing Replay Feeder Jul 6, 2023
@lukasmittag
Copy link
Contributor

I agree, will not hurt anyone, if we merge this. We should remove once we deprecate KUKSA.val server.

@erikbosch erikbosch merged commit 53e1be1 into eclipse-kuksa:main Jul 12, 2023
2 checks passed
@erikbosch erikbosch deleted the erik_replay branch July 12, 2023 15:20
This was referenced Sep 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants