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

Script to create the 8GB blob that can be used in lieu of onie installer #806

Closed
wants to merge 1 commit into from
Closed

Script to create the 8GB blob that can be used in lieu of onie installer #806

wants to merge 1 commit into from

Conversation

padmanarayana
Copy link
Contributor

Compared to a ONIE based SONiC install, when the dd image generated from this script is used, the following services failed/have warnings:

[FAILED] Failed to start /etc/rc.local Compatibility.
[FAILED] Failed to start Update rsyslog configuration.
[FAILED] Failed to start Update NTP configuration.
[FAILED] Failed to start System Logging Service.
aufs au_opts_verify:1570:docker[679]: dirperm1 breaks the protection by the permission bits on the lower branch

This is under investigation.

@msftclas
Copy link

@padmanarayana,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

mkdir ${PLATFORM}_mount
mount -t auto -o loop ${PLATFORM}_8GB_dd.img ${PLATFORM}_mount

echo -e "\n# Extracting tarballs into ${PLATFORM}_mount..."
Copy link
Collaborator

Choose a reason for hiding this comment

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

how can this script work? it does not have kernel, squashfs???

Copy link
Collaborator

@lguohan lguohan left a comment

Choose a reason for hiding this comment

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

script won't work.

@lguohan
Copy link
Collaborator

lguohan commented Jul 14, 2017

most of the logic in build_dd_image.sh is in installer/x86_64/install.sh file. therefore, I think the best to do create the raw(dd) image is using following steps:

  1. create a 8G file, format as ext4 and mount to a directory.
  2. run sonic-broadcom.bin onie installer to install it into the directory.

mkdir -p ${PLATFORM}_mount/disk-img
dd if=/dev/zero of=${PLATFORM}_mount/disk-img/var-log.ext4 count=8388608
mkfs.ext4 -q ${PLATFORM}_mount/disk-img/var-log.ext4 -F

Copy link
Collaborator

Choose a reason for hiding this comment

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

line 16 ~ 24 are in installer/x86_64/install.sh. can you try to embed the logic into install.sh https://github.com/Azure/sonic-buildimage/blob/master/installer/x86_64/install.sh#L411

gzip ${PLATFORM}_8GB_dd.img
mv ${PLATFORM}_8GB_dd.img.gz target
echo -e "\n# Done. DD image is under target/${PLATFORM}_8GB_dd.img.gz\n"

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'd like to have a grub directory in this sonic partition, so that you can chain the grub in ftos to grub in sonic.

Copy link
Collaborator

Choose a reason for hiding this comment

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

it is much easier for later sonic-to-sonic upgrade scenario.

@lguohan
Copy link
Collaborator

lguohan commented Jul 14, 2017

the sonic partition should also have a /host/machine.conf file for sonic to work properly.

@lguohan lguohan closed this Jul 14, 2017
@lguohan lguohan reopened this Jul 14, 2017
@msftclas
Copy link

@padmanarayana,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.

It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@msftclas
Copy link

msftclas commented Aug 1, 2017

@padmanarayana, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.

Thanks, Microsoft Pull Request Bot

@padmanarayana
Copy link
Contributor Author

Closing this. Please refer to #846 that addresses review comments.

@padmanarayana padmanarayana deleted the sonic_dd_blob4 branch August 29, 2017 09:43
volodymyrsamotiy pushed a commit to volodymyrsamotiy/sonic-buildimage that referenced this pull request Mar 6, 2019
* 3f14956 2019-03-06 [portsorch] fix bug in speed set (sonic-net#806) [Stepan Blyshchak]
* c903b5d 2019-03-05 [vnetorch]: Add support of nexthop routes for Bitmap VNET implementation (sonic-net#805) [Volodymyr Samotiy]
* 3b34b21 2019-02-26 [vnetorch]: Add support of local routes for Bitmap VNET implementation (sonic-net#801) [Volodymyr Samotiy]
* 62b7ca4 2019-02-22 Fix incorrect ACL CRM increment, resulting orchagent crash (sonic-net#800) [Prince Sunny]
* dd6bdd9 2019-02-21 [vnetorch]: Bitmap VNet implementation (sonic-net#773) [Marian Pritsak]

Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
yxieca added a commit to yxieca/sonic-buildimage that referenced this pull request Mar 7, 2019
Submodule src/sonic-swss 4b489ce..90eb25d:
  > [portsorch] fix bug in speed set (sonic-net#806)

Submodule src/sonic-swss-common 2592b0c..c674e64:
  > Add multiple fields hdel support (sonic-net#267)
  > Update PFC_WD table name in CONFIG_DB (sonic-net#266)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
yxieca added a commit that referenced this pull request Mar 7, 2019
Submodule src/sonic-swss 4b489ce..90eb25d:
  > [portsorch] fix bug in speed set (#806)

Submodule src/sonic-swss-common 2592b0c..c674e64:
  > Add multiple fields hdel support (#267)
  > Update PFC_WD table name in CONFIG_DB (#266)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
lguohan pushed a commit that referenced this pull request Mar 23, 2019
* 3f14956 2019-03-06 [portsorch] fix bug in speed set (#806) [Stepan Blyshchak]
* c903b5d 2019-03-05 [vnetorch]: Add support of nexthop routes for Bitmap VNET implementation (#805) [Volodymyr Samotiy]
* 3b34b21 2019-02-26 [vnetorch]: Add support of local routes for Bitmap VNET implementation (#801) [Volodymyr Samotiy]
* 62b7ca4 2019-02-22 Fix incorrect ACL CRM increment, resulting orchagent crash (#800) [Prince Sunny]
* dd6bdd9 2019-02-21 [vnetorch]: Bitmap VNet implementation (#773) [Marian Pritsak]

Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 10, 2019
)

Submodule src/sonic-swss 4b489ce..90eb25d:
  > [portsorch] fix bug in speed set (sonic-net#806)

Submodule src/sonic-swss-common 2592b0c..c674e64:
  > Add multiple fields hdel support (#267)
  > Update PFC_WD table name in CONFIG_DB (#266)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 10, 2019
)

Submodule src/sonic-swss 4b489ce..90eb25d:
  > [portsorch] fix bug in speed set (sonic-net#806)

Submodule src/sonic-swss-common 2592b0c..c674e64:
  > Add multiple fields hdel support (#267)
  > Update PFC_WD table name in CONFIG_DB (#266)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
tiantianlv pushed a commit to SONIC-DEV/sonic-buildimage that referenced this pull request Apr 10, 2019
)

Submodule src/sonic-swss 4b489ce..90eb25d:
  > [portsorch] fix bug in speed set (sonic-net#806)

Submodule src/sonic-swss-common 2592b0c..c674e64:
  > Add multiple fields hdel support (#267)
  > Update PFC_WD table name in CONFIG_DB (#266)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
madhanmellanox pushed a commit to madhanmellanox/sonic-buildimage that referenced this pull request Mar 23, 2020
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
lguohan pushed a commit that referenced this pull request Apr 2, 2021
1906fdf [pyext] Update Makefile to disable warning Wconversion for older swig (#817)
1dbcaeb Remove gbsyncd_startup.py (#813)
0372615 [vslib] Refresh queue pause status (#814)
00d883c [sairedis] Add sai_dbg_generate_dump api to redis interface (#811)
8585803 [sairedis] Unlock api mutex for communication mode (#812)
c69549a [pyext] Add SWIG sairedis python submodule (#806)
daall pushed a commit that referenced this pull request Apr 2, 2021
1906fdf [pyext] Update Makefile to disable warning Wconversion for older swig (#817)
1dbcaeb Remove gbsyncd_startup.py (#813)
0372615 [vslib] Refresh queue pause status (#814)
00d883c [sairedis] Add sai_dbg_generate_dump api to redis interface (#811)
8585803 [sairedis] Unlock api mutex for communication mode (#812)
c69549a [pyext] Add SWIG sairedis python submodule (#806)
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
1906fdf [pyext] Update Makefile to disable warning Wconversion for older swig (sonic-net#817)
1dbcaeb Remove gbsyncd_startup.py (sonic-net#813)
0372615 [vslib] Refresh queue pause status (sonic-net#814)
00d883c [sairedis] Add sai_dbg_generate_dump api to redis interface (sonic-net#811)
8585803 [sairedis] Unlock api mutex for communication mode (sonic-net#812)
c69549a [pyext] Add SWIG sairedis python submodule (sonic-net#806)
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
1906fdf [pyext] Update Makefile to disable warning Wconversion for older swig (sonic-net#817)
1dbcaeb Remove gbsyncd_startup.py (sonic-net#813)
0372615 [vslib] Refresh queue pause status (sonic-net#814)
00d883c [sairedis] Add sai_dbg_generate_dump api to redis interface (sonic-net#811)
8585803 [sairedis] Unlock api mutex for communication mode (sonic-net#812)
c69549a [pyext] Add SWIG sairedis python submodule (sonic-net#806)
AidanCopeland pushed a commit to Metaswitch/sonic-buildimage that referenced this pull request Apr 14, 2022
Bring sairedis python modules for both python2 and python3 using SWIG and extra helper methods, as well as adding debian packages for those modules.
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