Skip to content

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

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

azmy98 wants to merge 1 commit into
sonic-net:masterfrom
azmy98:dev-docker-manifest-network-support

Conversation

@azmy98

@azmy98 azmy98 commented Feb 7, 2024 •

Copy link
Copy Markdown
Contributor

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

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

@azmy98

azmy98 commented Aug 26, 2024

Copy link
Copy Markdown
Contributor Author

@qiluo-msft, it does not impact ConfigDB schema

@fastiuk

fastiuk commented Oct 8, 2024

Copy link
Copy Markdown
Contributor

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

@fastiuk

fastiuk commented Oct 14, 2024

Copy link
Copy Markdown
Contributor

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

@fastiuk

fastiuk commented Oct 21, 2024

Copy link
Copy Markdown
Contributor

@qiluo-msft gentle reminder to merge this PR

1 similar comment
@fastiuk

fastiuk commented Dec 2, 2024

Copy link
Copy Markdown
Contributor

@qiluo-msft gentle reminder to merge this PR

@zhangyanzhao

Copy link
Copy Markdown
Collaborator

PRs are not merged, move to backlog

@azmy98

azmy98 commented Dec 10, 2024

Copy link
Copy Markdown
Contributor Author

@zhangyanzhao , why move to backlog?

@fastiuk

fastiuk commented Jan 6, 2025

Copy link
Copy Markdown
Contributor

@zhangyanzhao HLD approved, please merge it

This branch has not been deployed

No deployments
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