Skip to content

Log warning message upon receiver resource creation failure (backport #3924, backport #3954)#3938

Merged
MiguelCompany merged 3 commits into2.6.xfrom
mergify/bp/2.6.x/pr-3924
Oct 26, 2023
Merged

Log warning message upon receiver resource creation failure (backport #3924, backport #3954)#3938
MiguelCompany merged 3 commits into2.6.xfrom
mergify/bp/2.6.x/pr-3924

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Oct 17, 2023

This was an automatic backport of pull request #3924 done by Mergify.

We have also added the backport of #3954

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
(cherry picked from commit 77a85dd)
MiguelCompany
MiguelCompany previously approved these changes Oct 17, 2023
@MiguelCompany MiguelCompany added this to the v2.6.7 milestone Oct 17, 2023
@MiguelCompany MiguelCompany added the ci-pending PR which CI is running label Oct 17, 2023
@MiguelCompany
Copy link
Copy Markdown
Member

@richiprosima Please test this

@MiguelCompany
Copy link
Copy Markdown
Member

@richiprosima Please test mac

@JesusPoderoso
Copy link
Copy Markdown
Contributor

I am closing this PR because in a future PR, this log message is going to be managed as a warning instead of as an error.

* Refs #19773. Turn log error into a log warning.

Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>

* Refs #19773. Add boolean argument to `createReceiverResources`.

Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>

---------

Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
@MiguelCompany MiguelCompany reopened this Oct 25, 2023
@MiguelCompany MiguelCompany changed the title [19638] Log error message upon receiver resource creation failure (backport #3924) Log warning message upon receiver resource creation failure (backport #3924, backport #3954) Oct 25, 2023
@MiguelCompany MiguelCompany merged commit be85b5e into 2.6.x Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-pending PR which CI is running

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants