Skip to content

[sonic-utilities] Update submodule#5856

Closed
madhanmellanox wants to merge 1 commit intosonic-net:masterfrom
madhanmellanox:skucreatortoolSUmadhan
Closed

[sonic-utilities] Update submodule#5856
madhanmellanox wants to merge 1 commit intosonic-net:masterfrom
madhanmellanox:skucreatortoolSUmadhan

Conversation

@madhanmellanox
Copy link
Contributor

- Why I did it
Updating submodule for sonic-utilities to get the changes to Azure master. The following were the commits that were part of this submodule.

[show/fgnhg.py] Fix whitespace issues (#1211) 061f428

SONiC installer - fix string formatting during image type check (#1197) 42efc03

[Mellanox] SKU creator Tool (#1163) 99de167

[consutil][show] Remove root need from show line command (#1218) 97dec12

[connect][clear] Support --devicename option for connect/clear line commands (#1214) 927bf9b

Copy missing values from INIT_CFG to config_db as part of db_migration task (#1209) ee6bc49

Replace swsssdk.SonicV2Connector with swsscommon.SonicV2Connector (SWIG wrapper of C++ implementation) in production code (#1217) 0d49a85

[sonic_sku_create] Move tests from sonic-utilities-tests/ folder to tests/ folder (#1222) 673f98d

- How I did it
Merged sonic-utilities changes for SKU creator tool to Azure master. Now, updating the submodule changes.

- How to verify it
Use SKU creator tool to create port_config.ini file from xml file or Minigraph file which contains SKU definition.

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

  • 201811
  • 201911
  • 202006

- Description for the changelog

This changeset contains relevant python script files for SKU creator tool.

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

@lguohan
Copy link
Collaborator

lguohan commented Nov 8, 2020

build failure. @qiluo-msft , looks like swsscommon issue.

@jleveque
Copy link
Contributor

jleveque commented Nov 8, 2020

Closing, because sonic-utilities submodule is being updated by #5849, and I am fixing the swsscommon dependency issues in that PR.

@jleveque jleveque closed this Nov 8, 2020
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.

3 participants