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

/host/machine.conf is incorrent #1519

Closed
fillmorechen0928 opened this issue Mar 21, 2018 · 19 comments
Closed

/host/machine.conf is incorrent #1519

fillmorechen0928 opened this issue Mar 21, 2018 · 19 comments

Comments

@fillmorechen0928
Copy link

Please refer
#1404

Original /host/machine.conf:

onie_version=master-201802221653.0.1-dirty
onie_vendor_id=31874
onie_build_machine=alphanetworks_snh60b0_640f
onie_machine_rev=0
onie_arch=x86_64
onie_build_platform=x86_64-alphanetworks_snh60b0_640f-r0
onie_config_version=1
onie_build_date="2018-02-22T16:53+0800"
onie_partition_type=gpt
onie_kernel_version=3.14.27
onie_firmware=bios
onie_switch_asic=bcm
onie_skip_ethmgmt_macs=no
onie_grub_image_name=grubx64.efi
onie_machine=alphanetworks_snh60b0_640f
onie_platform=x86_64-alphanetworks_snh60b0_640f-r0

Current /host/machine.conf:
onie_arch='x86_64'
onie_bin=''
onie_boot_reason='rescue'
onie_build_date='2018-01-23T19:05+0800'
onie_build_machine='alphanetworks_snh60b0_640f'
onie_cli_static_parms=''
onie_cli_static_url='sonic-broadcom.bin'
onie_config_version='1'
onie_dev='/dev/sda2'
onie_exec_url='sonic-broadcom.bin'
onie_firmware='bios'
onie_grub_image_name='grubx64.efi'
onie_initrd_tmp='/'
onie_installer='/var/tmp/installer'
onie_kernel_version='3.14.27'
onie_machine='alphanetworks_snh60b0_640f'
onie_machine_rev='0'
onie_partition_type='gpt'
onie_platform='x86_64-alphanetworks_snh60b0_640f-r0'
onie_root_dir='/mnt/onie-boot/onie'
onie_skip_ethmgmt_macs='no'
onie_switch_asic='bcm'
onie_vendor_id='31874'
onie_version='master-201801231905.0.1-dirty'

Please remove quote.

It cause path with more quotes for other APP in SONIC.

@lguohan
Copy link
Collaborator

lguohan commented Mar 23, 2018

the original /host/machine.conf and current /host/machine.conf seems to have different onie_version

original:
onie_version=master-201802221653.0.1-dirty

current:
onie_version='master-201801231905.0.1-dirty'

also in #1404, you said it works, so something has changed. It seems your current onie version is older than the original version. I suspect it is an onie issue. Can you check which onie you are going to use?

@fillmorechen0928
Copy link
Author

The difference is Original /host/machine.conf and Current /host/machine.conf.

ONIE's machine.conf is the same as onie_machine.zip (#1404)

@fillmorechen0928
Copy link
Author

The problem is how you generating "/host/machine.conf".

@fillmorechen0928
Copy link
Author

ONIE's machine.conf is the same as onie_machine.zip (#1404).

Before (#1404) /host/machine.conf
and after (#1404) /host/machine.conf
should be the same format.

@fillmorechen0928
Copy link
Author

The result of "set | grep ^onie_ > $demo_mnt/machine.conf" command is different from "cp /etc/machine.conf $demo_mnt" .

@lguohan
Copy link
Collaborator

lguohan commented Mar 26, 2018

#1404 is based on your request, and you have verified the PR and said it works. If it is not working now, then something has changed. In the example you given, you are using different onie versions. I verified the 'set | grep ^onie_ > $demo_mnt/machine.conf', it does not generate quote on my machine.

@fillmorechen0928
Copy link
Author

PR #1468 is working for #1404 issue. sonic_installer problem is fixed.

@fillmorechen0928
Copy link
Author

But PR #1468 generate #1519 found by other team member.

I will verifiy the 'set | grep ^onie_ > $demo_mnt/machine.conf' on my device.

@fillmorechen0928
Copy link
Author

I verified the 'set | grep ^onie_ > $demo_mnt/machine.conf', it generates quote on my machine.

@fillmorechen0928
Copy link
Author

  1. Boot up with ONIE and show machine.conf.

machine_conf

  1. show machine-build.conf

machine_build_conf

  1. Execute machine.conf

exec_machine_conf

  1. Show the result of 'set | grep ^onie_ > /tmp/machine.conf', it generates quote on my machine.

quote_machine_conf

@lguohan
Copy link
Collaborator

lguohan commented Mar 29, 2018

can you check your onie version. master-201802221653.0.1-dirty, according to you log above, that one does not have quote.

@fillmorechen0928
Copy link
Author

The file machine-build.conf does not have quote.

But the "set" command has quote.

@fillmorechen0928
Copy link
Author

#1404 has the attachment files onie_machine.zip

@lguohan
Copy link
Collaborator

lguohan commented Mar 29, 2018

Original /host/machine.conf:

onie_version=master-201802221653.0.1-dirty
onie_vendor_id=31874
onie_build_machine=alphanetworks_snh60b0_640f
onie_machine_rev=0
onie_arch=x86_64
onie_build_platform=x86_64-alphanetworks_snh60b0_640f-r0
onie_config_version=1
onie_build_date="2018-02-22T16:53+0800"
onie_partition_type=gpt
onie_kernel_version=3.14.27
onie_firmware=bios
onie_switch_asic=bcm
onie_skip_ethmgmt_macs=no
onie_grub_image_name=grubx64.efi
onie_machine=alphanetworks_snh60b0_640f
onie_platform=x86_64-alphanetworks_snh60b0_640f-r0

@lguohan
Copy link
Collaborator

lguohan commented Mar 29, 2018

this one does not have quote.

@fillmorechen0928
Copy link
Author

Don't care about master-201802221653.0.1-dirty version.

This is workaround version for SONIC issue. It's /host/machine.conf does not have quote due to generating from "cp /etc/machine.conf $demo_mnt" .

@fillmorechen0928
Copy link
Author

The command 'set | grep ^onie_ > $demo_mnt/machine.conf', it generates quote.

@lguohan
Copy link
Collaborator

lguohan commented Mar 29, 2018

can you check if this works for you?

set | grep ^onie |  sed -e "s/='/=/" -e "s/'$//"

@fillmorechen0928
Copy link
Author

Yes, it is work.

The test result:

  1. Execute machine.conf
    exec_machine_conf

  2. Show the result of 'set | grep ^onie | sed -e "s/='/=/" -e "s/'$//"', it does not generate quote on my machine.

no_quote_machine_conf

@lguohan lguohan closed this as completed Apr 1, 2018
stephenxs added a commit to stephenxs/sonic-buildimage that referenced this issue Dec 3, 2020
swss:
- [port_rates][rif_rates] Fix port_rates.lua (sonic-net#1520)
- Improve the way of handling BUFFER_PG during PFC storm (sonic-net#1480)
- [Orchagent] Vxlanorch and Portsorch changes for EVPN VXLAN (sonic-net#1264)
- [sub intf] ecmp hardware convergence acceleration at parent port oper status changes (sonic-net#1492)
- [dvs/copp] Adjust policer checks in VS tests for default COPP policers (sonic-net#1519)
- Flush ARP/neighbor entry on FDB flush when port L2-L3 (sonic-net#1506)
- [intfmgr]: Add GARP support (sonic-net#1503)

swss-common:
- [Dynamic buffer calc] Support dynamic buffer calculation (sonic-net#361)
- README.md: Update build requirements (sonic-net#414)

Signed-off-by: Stephen Sun <stephens@nvidia.com>
daall pushed a commit that referenced this issue Dec 4, 2020
swss:
- [port_rates][rif_rates] Fix port_rates.lua (#1520)
- Improve the way of handling BUFFER_PG during PFC storm (#1480)
- [Orchagent] Vxlanorch and Portsorch changes for EVPN VXLAN (#1264)
- [sub intf] ecmp hardware convergence acceleration at parent port oper status changes (#1492)
- [dvs/copp] Adjust policer checks in VS tests for default COPP policers (#1519)
- Flush ARP/neighbor entry on FDB flush when port L2-L3 (#1506)
- [intfmgr]: Add GARP support (#1503)

swss-common:
- [Dynamic buffer calc] Support dynamic buffer calculation (#361)
- README.md: Update build requirements (#414)

Signed-off-by: Stephen Sun <stephens@nvidia.com>
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this issue Feb 25, 2021
…t#6111)

swss:
- [port_rates][rif_rates] Fix port_rates.lua (sonic-net#1520)
- Improve the way of handling BUFFER_PG during PFC storm (sonic-net#1480)
- [Orchagent] Vxlanorch and Portsorch changes for EVPN VXLAN (sonic-net#1264)
- [sub intf] ecmp hardware convergence acceleration at parent port oper status changes (sonic-net#1492)
- [dvs/copp] Adjust policer checks in VS tests for default COPP policers (sonic-net#1519)
- Flush ARP/neighbor entry on FDB flush when port L2-L3 (sonic-net#1506)
- [intfmgr]: Add GARP support (sonic-net#1503)

swss-common:
- [Dynamic buffer calc] Support dynamic buffer calculation (sonic-net#361)
- README.md: Update build requirements (sonic-net#414)

Signed-off-by: Stephen Sun <stephens@nvidia.com>
renukamanavalan added a commit to renukamanavalan/sonic-buildimage that referenced this issue Apr 26, 2021
* 9dba93f disk_check: Check & mount RO as RW using tmpfs (sonic-net#1569)
* c3963c5 Fix remove ip rif (sonic-net#1535)
* 41d8ddc [config][generic-update] Adding apply-patch, rollback, checkpoints commands (sonic-net#1536)
* a3d37f1 [console] Display success message after line cleared (sonic-net#1579)
* b10c157 RADIUS Management User Authentication Feature (sonic-net#1521)
* 59ed6f3 platform pre-check for reboot in master branch (sonic-net#1556)
* f5efe89 [acl] Use a list instead of a comma-separated string for ACL port list (sonic-net#1519)
* e296a69 No more IP validation as it is more likely a URL (sonic-net#1555)
* d5f5382 [CLI][queue counters] add JSON output option for queue counters (sonic-net#1505)
* 176cc4a 1) Loopback interfaces with valid nexthop IP are not ignored/treated as loopback. (sonic-net#1565)
* 149ccbd [techsupport] Update show ip interface command (sonic-net#1562)
* 0e84418 Stop PMON docker before cold and soft reboots (sonic-net#1514)
* eba5c04 Fix Multi-ASIC show specific resursive route by using common parsing function (sonic-net#1560)
* e57e7f7 cache the bvid to vlan translations (sonic-net#1523)
* 38f9f60 sonic-installer: fix py3 issues in bootloader.aboot (sonic-net#1553)
* 02b263a [voq/inbandif] Voq inbandif port (sonic-net#1363)
* 0539789 [load_minigraph]: Avoid starting PFCWD for EPMS devicetype (sonic-net#1552)
* 030293c Use 'importlib' module in lieu of deprecated 'imp' module (sonic-net#1450)
* 50e5c61 Fixed the possibility of using uninitialized variable in route_check.py (sonic-net#1551)
lguohan pushed a commit that referenced this issue Apr 27, 2021
* 9dba93f disk_check: Check & mount RO as RW using tmpfs (#1569)
* c3963c5 Fix remove ip rif (#1535)
* 41d8ddc [config][generic-update] Adding apply-patch, rollback, checkpoints commands (#1536)
* a3d37f1 [console] Display success message after line cleared (#1579)
* b10c157 RADIUS Management User Authentication Feature (#1521)
* 59ed6f3 platform pre-check for reboot in master branch (#1556)
* f5efe89 [acl] Use a list instead of a comma-separated string for ACL port list (#1519)
* e296a69 No more IP validation as it is more likely a URL (#1555)
* d5f5382 [CLI][queue counters] add JSON output option for queue counters (#1505)
* 176cc4a 1) Loopback interfaces with valid nexthop IP are not ignored/treated as loopback. (#1565)
* 149ccbd [techsupport] Update show ip interface command (#1562)
* 0e84418 Stop PMON docker before cold and soft reboots (#1514)
* eba5c04 Fix Multi-ASIC show specific resursive route by using common parsing function (#1560)
* e57e7f7 cache the bvid to vlan translations (#1523)
* 38f9f60 sonic-installer: fix py3 issues in bootloader.aboot (#1553)
* 02b263a [voq/inbandif] Voq inbandif port (#1363)
* 0539789 [load_minigraph]: Avoid starting PFCWD for EPMS devicetype (#1552)
* 030293c Use 'importlib' module in lieu of deprecated 'imp' module (#1450)
* 50e5c61 Fixed the possibility of using uninitialized variable in route_check.py (#1551)
daall added a commit to daall/sonic-buildimage that referenced this issue Apr 29, 2021
sonic-swss:
- [Monitor Vlan] Fix a typo in hostif (sonic-net#1722)
- Update pool sizes during initialization from timer only (sonic-net#1708)
- [SflowMgr] SamplingRate Update by Speed Change Added (sonic-net#1721)

sonic-swss-common:
- [swss-common] Add MUX Metrics Table (sonic-net#482)
- [azp] Purge swss before installing the newly built deb package (sonic-net#472)

sonic-utilities:
- disk_check: Check & mount RO as RW using tmpfs (sonic-net#1569)
- No more IP validation as it is more likely a URL (sonic-net#1555)
- Stop PMON docker before cold and soft reboots (sonic-net#1514)
- Add soft-reboot reboot type (sonic-net#1453)
- [acl] Use a list instead of a comma-separated string for ACL port list (sonic-net#1519)
- sonic-installer: fix py3 issues in bootloader.aboot (sonic-net#1553)
- Fix unsupported fs.squashfs extraction in sonic-installer (sonic-net#1366)
- [show][config] cli support for firmware upgrade on Y-Cable (sonic-net#1528) (sonic-net#1558)

sonic-linux-kernel:
- [Mellanox] backport kernel patches for hw-management 7.0100.2303 (sonic-net#211)

Signed-off-by: Danny Allen <daall@microsoft.com>
lguohan pushed a commit that referenced this issue May 4, 2021
* [202012][swss/swss-common/utilities/kernel] Update submodule

sonic-swss:
- [Monitor Vlan] Fix a typo in hostif (#1722)
- Update pool sizes during initialization from timer only (#1708)
- [SflowMgr] SamplingRate Update by Speed Change Added (#1721)

sonic-swss-common:
- [swss-common] Add MUX Metrics Table (#482)
- [azp] Purge swss before installing the newly built deb package (#472)

sonic-utilities:
- disk_check: Check & mount RO as RW using tmpfs (#1569)
- No more IP validation as it is more likely a URL (#1555)
- Stop PMON docker before cold and soft reboots (#1514)
- Add soft-reboot reboot type (#1453)
- [acl] Use a list instead of a comma-separated string for ACL port list (#1519)
- sonic-installer: fix py3 issues in bootloader.aboot (#1553)
- Fix unsupported fs.squashfs extraction in sonic-installer (#1366)
- [show][config] cli support for firmware upgrade on Y-Cable (#1528) (#1558)

sonic-linux-kernel:
- [Mellanox] backport kernel patches for hw-management 7.0100.2303 (#211)

Signed-off-by: Danny Allen <daall@microsoft.com>

* Update utilities w/ build fix
stepanblyschak pushed a commit to stepanblyschak/sonic-buildimage that referenced this issue May 10, 2021
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this issue May 23, 2021
* 9dba93f disk_check: Check & mount RO as RW using tmpfs (sonic-net#1569)
* c3963c5 Fix remove ip rif (sonic-net#1535)
* 41d8ddc [config][generic-update] Adding apply-patch, rollback, checkpoints commands (sonic-net#1536)
* a3d37f1 [console] Display success message after line cleared (sonic-net#1579)
* b10c157 RADIUS Management User Authentication Feature (sonic-net#1521)
* 59ed6f3 platform pre-check for reboot in master branch (sonic-net#1556)
* f5efe89 [acl] Use a list instead of a comma-separated string for ACL port list (sonic-net#1519)
* e296a69 No more IP validation as it is more likely a URL (sonic-net#1555)
* d5f5382 [CLI][queue counters] add JSON output option for queue counters (sonic-net#1505)
* 176cc4a 1) Loopback interfaces with valid nexthop IP are not ignored/treated as loopback. (sonic-net#1565)
* 149ccbd [techsupport] Update show ip interface command (sonic-net#1562)
* 0e84418 Stop PMON docker before cold and soft reboots (sonic-net#1514)
* eba5c04 Fix Multi-ASIC show specific resursive route by using common parsing function (sonic-net#1560)
* e57e7f7 cache the bvid to vlan translations (sonic-net#1523)
* 38f9f60 sonic-installer: fix py3 issues in bootloader.aboot (sonic-net#1553)
* 02b263a [voq/inbandif] Voq inbandif port (sonic-net#1363)
* 0539789 [load_minigraph]: Avoid starting PFCWD for EPMS devicetype (sonic-net#1552)
* 030293c Use 'importlib' module in lieu of deprecated 'imp' module (sonic-net#1450)
* 50e5c61 Fixed the possibility of using uninitialized variable in route_check.py (sonic-net#1551)
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this issue Aug 7, 2021
* 9dba93f disk_check: Check & mount RO as RW using tmpfs (sonic-net#1569)
* c3963c5 Fix remove ip rif (sonic-net#1535)
* 41d8ddc [config][generic-update] Adding apply-patch, rollback, checkpoints commands (sonic-net#1536)
* a3d37f1 [console] Display success message after line cleared (sonic-net#1579)
* b10c157 RADIUS Management User Authentication Feature (sonic-net#1521)
* 59ed6f3 platform pre-check for reboot in master branch (sonic-net#1556)
* f5efe89 [acl] Use a list instead of a comma-separated string for ACL port list (sonic-net#1519)
* e296a69 No more IP validation as it is more likely a URL (sonic-net#1555)
* d5f5382 [CLI][queue counters] add JSON output option for queue counters (sonic-net#1505)
* 176cc4a 1) Loopback interfaces with valid nexthop IP are not ignored/treated as loopback. (sonic-net#1565)
* 149ccbd [techsupport] Update show ip interface command (sonic-net#1562)
* 0e84418 Stop PMON docker before cold and soft reboots (sonic-net#1514)
* eba5c04 Fix Multi-ASIC show specific resursive route by using common parsing function (sonic-net#1560)
* e57e7f7 cache the bvid to vlan translations (sonic-net#1523)
* 38f9f60 sonic-installer: fix py3 issues in bootloader.aboot (sonic-net#1553)
* 02b263a [voq/inbandif] Voq inbandif port (sonic-net#1363)
* 0539789 [load_minigraph]: Avoid starting PFCWD for EPMS devicetype (sonic-net#1552)
* 030293c Use 'importlib' module in lieu of deprecated 'imp' module (sonic-net#1450)
* 50e5c61 Fixed the possibility of using uninitialized variable in route_check.py (sonic-net#1551)
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this issue Feb 5, 2022
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

No branches or pull requests

2 participants