Skip to content

[202205][Mellanox] Disable MFT bash autocompletion#17362

Merged
yxieca merged 1 commit intosonic-net:202205from
nazariig:202205-mft-bash-fix
Dec 12, 2023
Merged

[202205][Mellanox] Disable MFT bash autocompletion#17362
yxieca merged 1 commit intosonic-net:202205from
nazariig:202205-mft-bash-fix

Conversation

@nazariig
Copy link
Collaborator

@nazariig nazariig commented Nov 30, 2023

A W/A to overcome SN2700 20 sec delay on login due to MFT bash autocompletion bug.
Must be reverted once a new MFT is ready.

Why I did it

To overcome SN2700 20 sec delay on login

Work item tracking

N/A

How I did it

Removed MFT bash autocompletion part

How to verify it

  1. Build a mellanox image
  2. Verify no such links after system boot.

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

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

Tested branch (Please provide the tested image version)

  • 202205

Details if related

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

  • N/A

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

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@bingwang-ms
Copy link
Contributor

@liuh-80 Can you please help review this change? Thanks!

Copy link
Contributor

@liuh-80 liuh-80 left a comment

Choose a reason for hiding this comment

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

Verifyed with Mellanox device and SONiC.202205-17362.422766-3f7a2c946

The MFT scripts been removed, and issue fixed.

@liat-grozovik liat-grozovik changed the title [202205][mellanox]: Disable MFT bash autocompletion [202205][Mellanox]: Disable MFT bash autocompletion Dec 7, 2023
@liat-grozovik
Copy link
Collaborator

@yxieca could you please help to merge?

@yxieca
Copy link
Contributor

yxieca commented Dec 7, 2023

@yxieca could you please help to merge?

@liat-grozovik can you confirm this change has been tested on 202205 branch? Thanks!

@liat-grozovik liat-grozovik changed the title [202205][Mellanox]: Disable MFT bash autocompletion [202205][Mellanox] Disable MFT bash autocompletion Dec 10, 2023
@yxieca yxieca merged commit 4897953 into sonic-net:202205 Dec 12, 2023
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.

6 participants