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

Test DaemonReload #818

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

ArtiomDivak
Copy link
Contributor

This integration test tests the Daemond Reload by adding new service that won't start first. After changing the file and by using daemon reload the service will start and will be 'active'

@ArtiomDivak ArtiomDivak force-pushed the issue-668-daemond-reload branch 4 times, most recently from e458dae to 950a978 Compare March 10, 2024 09:53
@coveralls
Copy link

coveralls commented Mar 10, 2024

Coverage Status

coverage: 80.393% (+0.04%) from 80.356%
when pulling 8938e33 on ArtiomDivak:issue-668-daemond-reload
into 9abb3ca on eclipse-bluechi:main.

@ArtiomDivak ArtiomDivak force-pushed the issue-668-daemond-reload branch 4 times, most recently from 2b90b0f to 6e69ed6 Compare March 11, 2024 11:55
@ArtiomDivak ArtiomDivak force-pushed the issue-668-daemond-reload branch 2 times, most recently from 90fc93a to 522b6d6 Compare March 11, 2024 13:08
@ArtiomDivak ArtiomDivak force-pushed the issue-668-daemond-reload branch 2 times, most recently from be7a974 to 8938e33 Compare March 12, 2024 07:43
This integration test tests the Daemond Reload by adding
new service that won't start first. After changing the file and
by using daemon reload the service will start and will be 'active'

Signed-off-by: Artiom Divak <adivak@redhat.com>
Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@engelmi engelmi left a comment

Choose a reason for hiding this comment

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

LGTM

@engelmi engelmi merged commit e604a21 into eclipse-bluechi:main Mar 12, 2024
20 checks passed
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