Skip to content

[mlnx-ffb.sh] Update issu-version location#14925

Merged
qiluo-msft merged 4 commits intosonic-net:masterfrom
stepanblyschak:ffb-check-new
Jun 15, 2023
Merged

[mlnx-ffb.sh] Update issu-version location#14925
qiluo-msft merged 4 commits intosonic-net:masterfrom
stepanblyschak:ffb-check-new

Conversation

@stepanblyschak
Copy link
Copy Markdown
Collaborator

@stepanblyschak stepanblyschak commented May 3, 2023

Why I did it

ISSU version check fails due to inability to mount squashfs from 202211 on 201911

Work item tracking
  • Microsoft ADO (24417508):

How I did it

Put ISSU version file under platform directory

How to verify it

Warm-upgrade matrix:

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

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

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

@vaibhavhd
Copy link
Copy Markdown
Contributor

The change LGTM. However, the description says that the change has been tested for 201911 to 202211

Since this change is to master, I think the test should be done for all eligible base branches to master as target.

@vaibhavhd
Copy link
Copy Markdown
Contributor

Why is this change not needed on 202205?

@liat-grozovik
Copy link
Copy Markdown
Collaborator

Why is this change not needed on 202205?

it is needed this is why there is an ask to backport it.
there are PRs directly to 202012 and 201911 as they are not cleanly cherry picked

@liat-grozovik
Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stepanblyschak
Copy link
Copy Markdown
Collaborator Author

@vaibhavhd updated the test upgrade matrix in description

@stepanblyschak
Copy link
Copy Markdown
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@stepanblyschak
Copy link
Copy Markdown
Collaborator Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Copy Markdown
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

mkdir -p "${FS_MOUNTPOINT}"
mount -t squashfs "${FS_PATH}" "${FS_MOUNTPOINT}" || {
>&2 echo "Failed to mount next SONiC image"
ISSU_VERSION_FILE_PATH="/etc/mlnx/issu-version"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/etc/mlnx/issu-version

Is this old path deprecated?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@qiluo-msft Right

@qiluo-msft qiluo-msft merged commit e2e5b77 into sonic-net:master Jun 15, 2023
@vaibhavhd
Copy link
Copy Markdown
Contributor

MSFT ADO: 24417508

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jun 29, 2023
#### Why I did it

ISSU version check fails due to inability to mount squashfs from 202211 on 201911

#### How I did it

Put ISSU version file under platform directory

#### How to verify it

Warm-upgrade matrix:
- 201911 (with sonic-net#14928) to master
- 201911 (with sonic-net#14928) to 202211
- 202012 (with sonic-net#14927) to master
- 202205 (with this change cherry-picked) to master
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202205: #15673

mssonicbld pushed a commit to mssonicbld/sonic-buildimage that referenced this pull request Jun 29, 2023
#### Why I did it

ISSU version check fails due to inability to mount squashfs from 202211 on 201911

#### How I did it

Put ISSU version file under platform directory

#### How to verify it

Warm-upgrade matrix:
- 201911 (with sonic-net#14928) to master
- 201911 (with sonic-net#14928) to 202211
- 202012 (with sonic-net#14927) to master
- 202205 (with this change cherry-picked) to master
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202211: #15674

yxieca pushed a commit that referenced this pull request Jun 30, 2023
#### Why I did it

ISSU version check fails due to inability to mount squashfs from 202211 on 201911

#### How I did it

Put ISSU version file under platform directory

#### How to verify it

Warm-upgrade matrix:
- 201911 (with #14928) to master
- 201911 (with #14928) to 202211
- 202012 (with #14927) to master
- 202205 (with this change cherry-picked) to master

Co-authored-by: Stepan Blyshchak <38952541+stepanblyschak@users.noreply.github.com>
qiluo-msft pushed a commit that referenced this pull request Jul 2, 2023
BACKPORT OF #14925

#### Why I did it

ISSU version check fails due to inability to mount squashfs from 202211 on 201911

#### How I did it

Put ISSU version file under platform directory

#### How to verify it

202012 (with [202012][mlnx-ffb.sh] Update issu-version location  #14927) to master
@mssonicbld
Copy link
Copy Markdown
Collaborator

@stepanblyschak PR conflicts with 202305 branch

@StormLiangMS
Copy link
Copy Markdown
Contributor

@stepanblyschak @liat-grozovik cherry-pick failure, need separate PR to 202305.

sonic-otn pushed a commit to sonic-otn/sonic-buildimage that referenced this pull request Sep 20, 2023
#### Why I did it

ISSU version check fails due to inability to mount squashfs from 202211 on 201911

#### How I did it

Put ISSU version file under platform directory

#### How to verify it

Warm-upgrade matrix:
- 201911 (with sonic-net#14928) to master
- 201911 (with sonic-net#14928) to 202211
- 202012 (with sonic-net#14927) to master
- 202205 (with this change cherry-picked) to master
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.

8 participants