Skip to content

[doc] Moving Configuration.md from swss to sonic-yang-models folder#10078

Merged
qiluo-msft merged 3 commits intosonic-net:masterfrom
dgsudharsan:config_md
Mar 15, 2022
Merged

[doc] Moving Configuration.md from swss to sonic-yang-models folder#10078
qiluo-msft merged 3 commits intosonic-net:masterfrom
dgsudharsan:config_md

Conversation

@dgsudharsan
Copy link
Collaborator

Signed-off-by: Sudharsan Dhamal Gopalarathnam [email protected]

Why I did it

Moved Configuration.md from swss to yang folder. The configuration schema should be present along with yang models and when users add yang model they could update configuration document as well.

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

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

@dgsudharsan
Copy link
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@qiluo-msft qiluo-msft changed the title [doc]Moving Configuration.md from swss to sonic-yang-models folder [doc] Moving Configuration.md from swss to sonic-yang-models folder Mar 10, 2022
@qiluo-msft
Copy link
Collaborator

qiluo-msft commented Mar 10, 2022

Could you add the link to the PR on sonic-swss which removes the old md file?

Copy link
Member

@praveen-li praveen-li left a comment

Choose a reason for hiding this comment

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

-- The diff is full file, due to move in Repos I guess?
-- There will be another PR which deletes the file in sonic-swss repo ?

@dgsudharsan
Copy link
Collaborator Author

-- The diff is full file, due to move in Repos I guess? -- There will be another PR which deletes the file in sonic-swss repo ?

Correct.

@dgsudharsan dgsudharsan requested a review from praveen-li March 14, 2022 04:43
@dgsudharsan
Copy link
Collaborator Author

Could you add the link to the PR on sonic-swss which removes the old md file?

sonic-net/sonic-swss#2177

@qiluo-msft
Copy link
Collaborator

This PR only update document, will override checkers.

@qiluo-msft qiluo-msft merged commit 28f6a51 into sonic-net:master Mar 15, 2022
qiluo-msft pushed a commit to sonic-net/sonic-swss that referenced this pull request Mar 15, 2022
**What I did**
Moved Configuration.md from swss to yang folder. The configuration schema should be present along with yang models and when users add yang model they could update configuration document as well.

The PR to move to yang folder is sonic-net/sonic-buildimage#10078
dprital pushed a commit to dprital/sonic-swss that referenced this pull request May 8, 2022
…#2177)

**What I did**
Moved Configuration.md from swss to yang folder. The configuration schema should be present along with yang models and when users add yang model they could update configuration document as well.

The PR to move to yang folder is sonic-net/sonic-buildimage#10078
preetham-singh pushed a commit to preetham-singh/sonic-swss that referenced this pull request Aug 6, 2022
…#2177)

**What I did**
Moved Configuration.md from swss to yang folder. The configuration schema should be present along with yang models and when users add yang model they could update configuration document as well.

The PR to move to yang folder is sonic-net/sonic-buildimage#10078
@dgsudharsan dgsudharsan deleted the config_md branch March 9, 2023 01:55
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
…#2177)

**What I did**
Moved Configuration.md from swss to yang folder. The configuration schema should be present along with yang models and when users add yang model they could update configuration document as well.

The PR to move to yang folder is sonic-net/sonic-buildimage#10078
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants