Skip to content

libnetwork/osl: Namespace.setSysctls: use stdlib errors#49224

Merged
robmry merged 1 commit intomoby:masterfrom
thaJeztah:std_errs
Jan 7, 2025
Merged

libnetwork/osl: Namespace.setSysctls: use stdlib errors#49224
robmry merged 1 commit intomoby:masterfrom
thaJeztah:std_errs

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

This was the only location in this file where pkg/errors was used, and this function already mixed stdlib errors with pkg/errors.

- A picture of a cute animal (not mandatory but encouraged)

This was the only location in this file where pkg/errors was used,
and this function already mixed stdlib errors with pkg/errors.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review area/networking Networking kind/refactor PR's that refactor, or clean-up code labels Jan 7, 2025
@thaJeztah thaJeztah added this to the 28.0.0 milestone Jan 7, 2025
@thaJeztah thaJeztah self-assigned this Jan 7, 2025
@thaJeztah thaJeztah requested a review from robmry January 7, 2025 10:05
Copy link
Copy Markdown
Contributor

@robmry robmry left a comment

Choose a reason for hiding this comment

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

LGTM - thank you!

@thaJeztah
Copy link
Copy Markdown
Member Author

Saw your other PR still had a WIP commit in it, so probably would need a rebase at some point, so thought; we might as well get it out of the way already 😃

Copy link
Copy Markdown
Member

@laurazard laurazard left a comment

Choose a reason for hiding this comment

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

LGTM

@robmry
Copy link
Copy Markdown
Contributor

robmry commented Jan 7, 2025

I'll merge this, then I can rebase the other PR ...

@robmry robmry merged commit 3c628f7 into moby:master Jan 7, 2025
@thaJeztah thaJeztah deleted the std_errs branch January 7, 2025 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking Networking kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants