Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the demo_part_size not initialized issue when creating partition #14296

Merged
merged 1 commit into from
Mar 23, 2023

Conversation

xumia
Copy link
Collaborator

@xumia xumia commented Mar 17, 2023

Why I did it

The demo_part_size should be initialized before creating partition.

How I did it

Move the initializing setting to the line before using it.

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@xumia
Copy link
Collaborator Author

xumia commented Mar 17, 2023

@sheiun-xu, could you please help review it, thanks.

@xumia xumia requested a review from StormLiangMS March 17, 2023 10:12
Copy link
Contributor

@StormLiangMS StormLiangMS left a comment

Choose a reason for hiding this comment

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

LGTM

@StormLiangMS StormLiangMS merged commit 67a57cc into sonic-net:master Mar 23, 2023
mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Mar 23, 2023
…onic-net#14296)

Why I did it
The demo_part_size should be initialized before creating partition.

How I did it
Move the initializing setting to the line before using it.

How to verify it
@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202211: #14394

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

Successfully merging this pull request may close these issues.

3 participants