Skip to content

libn/d/overlay: drop obsolete writeToStore comment#50364

Merged
thaJeztah merged 1 commit intomoby:masterfrom
corhere:libn/ov-writetostore-comment
Jul 10, 2025
Merged

libn/d/overlay: drop obsolete writeToStore comment#50364
thaJeztah merged 1 commit intomoby:masterfrom
corhere:libn/ov-writetostore-comment

Conversation

@corhere
Copy link
Contributor

@corhere corhere commented Jul 9, 2025

- What I did
As discussed in #50260 (comment), the writeToStore() call was removed from CreateNetwork in commit 0fa873c but a comment about undoing the write remained. I removed the obsolete comment.

- How I did it

- How to verify it

- Human readable description for the release notes

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

The writeToStore() call was removed from CreateNetwork in
commit 0fa873c. The comment about
undoing the write is no longer applicable.

Signed-off-by: Cory Snider <csnider@mirantis.com>
@corhere corhere added this to the 29.0.0 milestone Jul 9, 2025
@corhere corhere added kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. area/networking/d/overlay Networking labels Jul 9, 2025
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@thaJeztah thaJeztah added status/4-merge kind/refactor PR's that refactor, or clean-up code and removed kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. labels Jul 9, 2025
@thaJeztah
Copy link
Member

CI failures are unrelated

@thaJeztah thaJeztah merged commit d4aa1cf into moby:master Jul 10, 2025
271 of 275 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking/d/overlay Networking kind/refactor PR's that refactor, or clean-up code status/4-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants