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

[Azure pipeline] Refresh configuration of rsyslogd using SIGHUP #1502

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

Conversation

stephenxs
Copy link
Contributor

@stephenxs stephenxs commented Jan 17, 2025

To overcome the rsyslog issue in azure pipeline test by refreshing rsyslogd configuration using SIGHUP instead of killing and restarting it.

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs stephenxs changed the title [TEST PR DO NOT MERGE] Refresh configuration of rsyslogd using SIGHUP Refresh configuration of rsyslogd using SIGHUP Jan 17, 2025
@stephenxs stephenxs changed the title Refresh configuration of rsyslogd using SIGHUP [Azure pipeline] Refresh configuration of rsyslogd using SIGHUP Jan 17, 2025
@jianyuewu
Copy link

All passed, nice👍

@stephenxs stephenxs requested a review from saiarcot895 January 17, 2025 12:32
@stephenxs stephenxs marked this pull request as ready for review January 19, 2025 00:13
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Contributor Author

Hi @kcudnik @saiarcot895
Would you please review this PR? It is to overcome the rsyslog issue in azure pipeline test

@stephenxs stephenxs requested a review from kcudnik January 19, 2025 12:19
@kcudnik
Copy link
Collaborator

kcudnik commented Jan 20, 2025

seems like this will succeed

@stephenxs
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Contributor Author

The latest rerun failed due to the following issue which is also observed previously

[ OK ] VendorSaiTest.bulk_flow_entry (6 ms)
[ RUN ] VendorSaiTest.bulk_meter_bucket_entry
Assertion failed: _owned.empty () (src/own.cpp:197)
/bin/bash: line 6: 25392 Aborted

@stephenxs
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jianyuewu
Copy link

/azpw run

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