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

feat(core.cloud): publish BIRTH on EventAdmin install/uninstall events #4835

Conversation

marcellorinaldo
Copy link
Contributor

This PR modifies the standard CloudServices to publish a BIRTH certificate message on OSGi's EventAdmin install/uninstall events.

Related Issue: N/A.

Description of the solution adopted: N/A.

Screenshots: N/A.

Manual Tests: N/A.

Any side note on the changes made: N/A.

@marcellorinaldo marcellorinaldo marked this pull request as ready for review September 4, 2023 13:33
@marcellorinaldo
Copy link
Contributor Author

@GregoryIvo can you please review?

@marcellorinaldo
Copy link
Contributor Author

Duplication and low coverage are due to the porting of changes to the cloudconnection.eclipseiot.mqtt.cloud component. Such component has no tests and adding them in this PR is out of scope. Duplication is necessary because it needs to "mirror" the behavior of CloudServiceImpl.

@marcellorinaldo marcellorinaldo merged commit 07582f6 into eclipse-kura:develop Sep 8, 2023
@marcellorinaldo marcellorinaldo deleted the add-cloud-service-install-event-handler branch September 8, 2023 12:10
pierantoniomerlino pushed a commit that referenced this pull request Sep 26, 2023
#4835)

* feat(core.cloud): publish BIRTH on EventAdmin install/uninstall events

Signed-off-by: Marcello Martina <[email protected]>

* feat(cloudconnection.eclipseiot.mqtt.cloud): mirrored changes of d17e501

Signed-off-by: Marcello Martina <[email protected]>

* style: corrected log messages

Signed-off-by: Marcello Martina <[email protected]>

* refactor: added return to avoid useless if evaluation

Signed-off-by: Marcello Martina <[email protected]>

* refactor: removed unecessary return

Signed-off-by: Marcello Martina <[email protected]>

---------

Signed-off-by: Marcello Martina <[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.

2 participants