Skip to content

Expand docker manifest capabilities to include network and ports configuration#1601

Open
azmy98 wants to merge 1 commit intosonic-net:masterfrom
azmy98:dev-docker-manifest-network-support
Open

Expand docker manifest capabilities to include network and ports configuration#1601
azmy98 wants to merge 1 commit intosonic-net:masterfrom
azmy98:dev-docker-manifest-network-support

Conversation

@azmy98
Copy link
Contributor

@azmy98 azmy98 commented Feb 7, 2024

Adding documentation regarding the network and ports configurations for docker manifest

Submodule PR Title Status
sonic-buildimage Adjusting docker creation to include ports forwarding and docker network type GitHub issue/pull request detail
sonic-utilities Expanding the manifest capabilities GitHub issue/pull request detail

@azmy98 azmy98 force-pushed the dev-docker-manifest-network-support branch from 22ab927 to c8b70fb Compare February 7, 2024 12:20
@qiluo-msft
Copy link
Contributor

Does this design change impact ConfigDB schema? If yes, please also include ConfigDB schema in HLD, and add yang model PR.

@azmy98
Copy link
Contributor Author

azmy98 commented Aug 26, 2024

@qiluo-msft, it does not impact ConfigDB schema

@fastiuk
Copy link
Contributor

fastiuk commented Oct 8, 2024

@qiluo-msft , please merge this PR, since it was approved

@fastiuk
Copy link
Contributor

fastiuk commented Oct 14, 2024

@qiluo-msft could you please merge this PR? It was already approved

@fastiuk
Copy link
Contributor

fastiuk commented Oct 21, 2024

@qiluo-msft gentle reminder to merge this PR

1 similar comment
@fastiuk
Copy link
Contributor

fastiuk commented Dec 2, 2024

@qiluo-msft gentle reminder to merge this PR

@zhangyanzhao
Copy link
Collaborator

PRs are not merged, move to backlog

@azmy98
Copy link
Contributor Author

azmy98 commented Dec 10, 2024

@zhangyanzhao , why move to backlog?

@fastiuk
Copy link
Contributor

fastiuk commented Jan 6, 2025

@zhangyanzhao HLD approved, please merge it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: MovedToBacklog

Development

Successfully merging this pull request may close these issues.

5 participants