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

Kafka bridges+self hosted test pr #125

Merged

Conversation

wagmarcel
Copy link
Member

No description provided.

@wagmarcel wagmarcel force-pushed the kafka-bridges+Self-hosted-test-PR branch from 397dfaf to 4ae7113 Compare March 12, 2022 21:55
This Commit adds the link between Kafka and other services. It provides

* The Alerta bridge which allows to send alerts through a Kafka channel
* The NGSI-LD Updates bridge which allows to send ngsi-ld data updates through Kafka
* The Debezium bridge which maps the debezium entitiy update to respective Kafka channels
* All is in a single container, only startup command must be different
* A helm chart with deployment files and configmaps for the bridges

Closes issue IndustryFusion#126
…idges

This commit adds new end-2-end tests for the Kafka bridges and restructures the existing test.
* Main reason for the restructuring is that now it is possible to install the platform not only at once but
  also in several steps to reduce the startup-load in the test image and reduce time to download relevant containers.
  This needed to label the helm subcharts and deploy different iterations based on labels an dependenies. This also needed changes for the
  startup tests to check the success of the different installation iterations.
* Second main change is the introduction of a self-hosted runner. A self-hosted runner does not allow sudo operations.
  Therfore, the prepare-platform scripts are now split between the parts which need sudo and the parts which can be installed
  by the local runner without sudo.
@wagmarcel wagmarcel force-pushed the kafka-bridges+Self-hosted-test-PR branch from 4ae7113 to e90b260 Compare March 13, 2022 00:32
@wagmarcel
Copy link
Member Author

See related security issue #127

@wagmarcel wagmarcel marked this pull request as ready for review March 21, 2022 14:36
Copy link

@sysarcher sysarcher left a comment

Choose a reason for hiding this comment

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

LGTM

@KonstantinKe KonstantinKe merged commit 9fb6471 into IndustryFusion:main Mar 22, 2022
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.

4 participants