Skip to content

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

Closed
stephenxs wants to merge 2 commits intosonic-net:masterfrom
stephenxs:test-hup-rsyslogd
Closed

[Azure pipeline] Refresh configuration of rsyslogd using SIGHUP#1502
stephenxs wants to merge 2 commits intosonic-net:masterfrom
stephenxs:test-hup-rsyslogd

Conversation

@stephenxs
Copy link
Copy Markdown
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
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

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
Copy Markdown
Contributor

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
Signed-off-by: Stephen Sun <stephens@nvidia.com>
Signed-off-by: Stephen Sun <stephens@nvidia.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
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
@stephenxs
Copy link
Copy Markdown
Contributor Author

@kcudnik
Copy link
Copy Markdown
Collaborator

kcudnik commented Jan 20, 2025

seems like this will succeed

@stephenxs
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stephenxs
Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@jianyuewu
Copy link
Copy Markdown
Contributor

/azpw run

@stephenxs
Copy link
Copy Markdown
Contributor Author

No longer required. The issue was resolved in other ways.

@stephenxs stephenxs closed this Feb 3, 2025
@stephenxs stephenxs deleted the test-hup-rsyslogd branch February 20, 2025 02:06
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