20200221_jeffrey - silenceuncrio/diary GitHub Wiki

review

ariel 表示 m330 看到 init fail

## module <sntp> init
### module <snmp> init
json_object_from_file: error reading file /etc/icos/schedule_reboot/schedule_reboot.json: No such file or directory
### module <ssh> init
### module <upnp> init 

先試著複製出問題

利用下面的 command 持續作個幾次試著複製出問題

rm /etc/icos/schedule_reboot/schedule_reboot.json
rm /home/log/schedule_reboot.log
reboot

目前正需要彌補 自己過去的輕忽所造成的問題

所以千萬不可心急

信任需要重新累積

改利用 reset button 來 reset default

改利用 uboot 作 upgrade - run ldf

比想像的更難複製

先不急著追出 root cause 在這一時

但若下次有發現記得分析 log

每個禮拜的 review 若只是在每一天作節錄的話

不容易看到跨天的進度

最好有一張 task list

再依據 task 來這禮拜作了什麼

換個角度更好的來 review 這個禮拜

M300 - task

• [0%] P1 - DQA - BGP MD5 Authentication

M300 - issue

• [resolved] 698: sometimes lost apply button
• [resolved] 914: online manual opened via ie without the ability to adjust window screen size
• [resolved] 919: [GPS config] Lost TCP Server
• [40%] 922: sometimes when browser issue cgi request, some cgi will response 401 even we already login
  • [100%] M300; [100%] M350-C(SDK6.11)
  • [0%] M330; [0%] M350-C(SDK10.0.0); [0%] M360P
• [resolved] 923: change CID and upgrade firmware, the HW MCSV will change to SW MCSV

M300 - commit

• [bug] Include the leaflet-directive dependency on out Angular module
• [ariel 的需求] 'SIM Status' supports Unblock Fail / Illegal
• [issue 914] use 'target' attribute to specify opening the online manual in a new tab
• [issue 919] sync the 'LTE > GPS' web page from m330
• [issue 919] porting 'LTE > GPS Track' web page from m330
• [issue 922] output information base on different error
• [issue 922] web session bug
• [issue 923] fix issue 923

M330 - task

• [70%] P4 - CTCU - Web UI: GPS Track drawing(Server)
• [0%] P4 - Airtel - BGP Attributers shall be supported
• [0%] P4 - Airtel - GUI Display LTE Signal strength indicator
• [0%] P4 - Airtel - Upon Failure of above comparision, ...
• [0%] P4 - DQA - BGP MD5 Authentication

M330 - issue

• [resolved] 918: after apply ok with 'Work As' field changed, sometimes the value will be kept the old one
  • ariel 直接從 icos module 來解決

M330 - commit

• [ariel 的需求] 'SIM Status' supports Modem Locked / Unblock Fail / Illegal
• [issue 914] use 'target' attribute to specify opening the online manual in a new tab
• [release/v1.00] customize web ui for profile ADVICE/0_ADVICE_WIFI
• [release/v1.00] customize web ui for profile CTCU/2_CTCU-CHT

M330 - bug

• wan ethernet mtu - ui hint 與 html 的 min 不一致
• (sometimes after factory default) error reading file schedule_reboot.json at boot int

M350-C(SDK6.11) - task

• [100%] P1 - Web UI: CA info
• [0%] P4 - DQA - BGP MD5 Authentication

M350-C(SDK6.11) - commit

• [ariel 的需求] 'SIM Status' supports Modem Locked / Unblock Fail / Illegal
• [task] add 'Carrier Aggregation Info' at 'LTE > Serving Cell' page
• [issue 914] use 'target' attribute to specify opening the online manual in a new tab
• [issue 918] [workaround]sleep a while in connmgr.cgi?act=chg_wproto_apply
• [issue 918] remove the workaround since the problem fixed at the right point
• [issue 922] output information base on different error
• [issue 922] web session bug

M350-C(SDK6.11) - 注意

• firmware upgrade 時 web.log 會被 lighttpd 本身這個 process 塞爆

M350-C(SDK10.0.0)

• [50%] P1 - GRE
• [0%] P1 - GRE keep alive

M360P - task

• P1 - BGP with MD5
• P2 - Airtel - BGP status for every neighbors

M360P - commit

• [winston 的建議] adjust the size of key[] from 17 to 20
• [ariel 的需求] 'SIM Status' supports Modem Locked / Unblock Fail / Illegal
• [issue 914] use 'target' attribute to specify opening the online manual in a new tab

M300 - issue

• [40%] 922: sometimes when browser issue cgi request, some cgi will response 401 even we already login
  • [100%] M300; [100%] M350-C(SDK6.11)
  • [0%] M330; [0%] M350-C(SDK10.0.0); [0%] M360P

來套到 M300[develop]

參考 使用 Git 時如何做出跨 repo 的 cherry-pick

➜  m350 git:(develop) ✗ git log

...

commit 59fbc8063ee7ff5c83d76e37065797824f1c603e
Author: jeffrey <[email protected]>
Date:   Wed Feb 19 15:32:11 2020 +0800

    web session bug

    ...

commit 92f219ef9c68c930075be454eb697481324654a7
Author: john <[email protected]>
Date:   Wed Feb 19 15:10:42 2020 +0800

...

commit d97d6ee43cc6edacb7ed1c68eddb84f2908d0eba
Author: jeffrey <[email protected]>
Date:   Wed Feb 19 10:00:59 2020 +0800

    output information base on different error

commit 38fcc45697375ac00ecaced0719e3c088cd8c815
Author: jeffrey <[email protected]>
Date:   Wed Feb 19 09:13:44 2020 +0800

...

我需要先在 M350 的 develop branch repository 之下產生兩個 patch

git format-patch -k --stdout 38fcc45697375ac00ecaced0719e3c088cd8c815..d97d6ee43cc6edacb7ed1c68eddb84f2908d0eba > ~/tmp/issue_922.01.patch
git format-patch -k --stdout 92f219ef9c68c930075be454eb697481324654a7..59fbc8063ee7ff5c83d76e37065797824f1c603e > ~/tmp/issue_922.02.patch

試著在 M330 develop branch 套用這兩個 patch

issue_922.01.patch - 套用成功

➜  m330 git:(develop) ✗ git am -k -3 < ~/tmp/issue_922.01.patch
Applying: output information base on different error

issue_922.02.patch - 套用成功

➜  m330 git:(develop) ✗ git am -k -3 < ~/tmp/issue_922.02.patch
Applying: web session bug

build 來試一下

有效 上 code 吧 - M330

commit a2abaffac3375cd36ffa7cb657b9c89b50a674b2
Refs: [develop], {origin/develop}
Author: jeffrey <[email protected]>
Date:   Wed Feb 19 15:32:11 2020 +0800

    web session bug

    when
    after login, session timeout not yet

    what
    sometimes multiple concurrent CGI requests will cause a issue
    some cgi will get 401 return and then the browder will redirect to login page

    why
    401 return is cause by we found no match session id

    how(workaround)
    but if we sleep(1) and load_session_store then search again
    we will find the match session id

 proscend/prosrc/webcgi/jweb.c | 62 ++++++++++++++++++++++++++++++++++++++++---
 1 file changed, 59 insertions(+), 3 deletions(-)

commit ebfebbab044d7edb89b804c9d34dfcdba3c03813
Author: jeffrey <[email protected]>
Date:   Wed Feb 19 10:00:59 2020 +0800

    output information base on different error

 proscend/prosrc/webcgi/jweb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

M330 - bug

• wan ethernet mtu - ui hint 與 html 的 min 不一致

解決

commit c212181772c44f08ccd6451a765e37b5291da777
Refs: [develop], {origin/develop}
Author: jeffrey <[email protected]>
Date:   Fri Feb 21 16:55:48 2020 +0800

    let the min value of WAN Ethernet MTU match the hint

 proscend/prosrc/www/app/feature/switch.html | 4 ++--
 proscend/prosrc/www/app/locale-en.json      | 1 +
 proscend/prosrc/www/app/locale-fr.json      | 1 +
 proscend/prosrc/www/app/locale-zh-tw.json   | 1 +
 4 files changed, 5 insertions(+), 2 deletions(-)