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.

Sign up for GitHub

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

Jump to bottom

service frr reload not working #3194

Open
nikos-github opened this issue Jul 22, 2019 · 4 comments
Open

service frr reload not working #3194

nikos-github opened this issue Jul 22, 2019 · 4 comments
Assignees
@pavel-shirshov

Comments

@nikos-github
Copy link
Collaborator

nikos-github commented Jul 22, 2019 •
edited by lguohan
Loading 

root@sonic:/home/admin# docker exec -it bgp service frr reload                                                                                                                                                    [warn] Reading deprecated /etc/frr/daemons.conf. Please move its settings to /etc/frr/daemons and remove it. ... (warning).
[warn] watchfrr_options contains a bash array value. The configured value is intentionally ignored since it is likely wrong. Please remove or fix the setting. ... (warning).
[FAIL] The frr-pythontools package is required for reload functionality. ... failed! (edited)
@nikos-github
Copy link
Collaborator Author

@xinliu-seattle @zhenggen-xu @MichelMoriniaux @mslocrian

@mslocrian
Copy link
Contributor

I thought that we had included frr-pythontools in a previous PR.

Going to build against current master and check it out.

@nikos-github
Copy link
Collaborator Author

nikos-github commented Jul 24, 2019 •
edited
Loading

@xinliu-seattle @mslocrian We did. Things are now broken at multiple levels unfortunately. Without proper review, frr got moved from systemd to supervisord, watchfrr is not running anymore, split configuration is not working and service frr reload is also not working.

@nikos-github nikos-github mentioned this issue Sep 19, 2019
Closed
@scottlaird scottlaird mentioned this issue Jun 17, 2020
Open
@ITJamie
Copy link

ITJamie commented Sep 22, 2022

any update on this?

mssonicbld added a commit that referenced this issue Mar 28, 2024
@mssonicbld
[submodule] Update submodule sonic-utilities to the latest HEAD autom…
3ed1995 
…atically (#18240)

#### Why I did it
src/sonic-utilities
```
* bdc57206 - (HEAD -> master, origin/master, origin/HEAD) Revert "Fix for Switch Port Modes and VLAN CLI Enhancement (#3108)" (#3246) (89 minutes ago) [jingwenxie]
* e35452b7 - Modify "show interface transceiver status" CLI to show SW cmis state (#3238) (2 days ago) [mihirpat1]
* 04a33e1f - Add "state" field in CONFIG_DB a toggle of the fabric port monitor feature (#2932) (2 days ago) [jfeng-arista]
* 3c489ba5 - Enhance route-check for multi-asic platforms (#3216) (5 days ago) [Deepak Singhal]
* c149e48b - [chassis] Add chassis support for CLI "config qos reload" (#3233) (6 days ago) [wenyiz2021]
* d8541add - Update port2alias (#3217) (8 days ago) [abdosi]
* d4688a8f - [graceful reboot] Add the pre_reboot_hook script execution, add the watchdog arm before the reboot (#3203) (8 days ago) [Vadym Hlushko]
* 125f36f3 - [ipintutil]Handle exception in show ip interfaces command (#3182) (10 days ago) [Sudharsan Dhamal Gopalarathnam]
* 9d532017 - [chassis][show-runningconfig] Fix the show runningconfiguration all issue on the Supervisor (#3194) (2 weeks ago) [Marty Y. Lok]
* 1a9261ce - [Techsupport]Handle SAI kv pair if present in sai common profile (#3196) (2 weeks ago) [Sudharsan Dhamal Gopalarathnam]
* 7466dc4a - Skip the validation of action in acl-loader if capability table in STATE_DB is empty (#3199) (2 weeks ago) [bingwang-ms]
* b879b658 - [Bug] Fix fw_setenv illegel character issue (#3201) (3 weeks ago) [xumia]
* 0b41a560 - [config] Add YANG alerting for override (#3188) (3 weeks ago) [jingwenxie]
* 24683b0c - [show] multi-asic show running test residue (#3198) (3 weeks ago) [jingwenxie]
* 995a797a - CLI to skip polling for periodic information for a port in DomInfoUpdateTask thread (#3187) (3 weeks ago) [mihirpat1]
* 9aa9eaa5 - [config] Add Table hard dependency check (#3159) (3 weeks ago) [jingwenxie]
* 5f0ffcca - [fast/warm-reboot] Put ERR message in syslog when a failure is seen (#3186) (4 weeks ago) [Vaibhav Hemant Dixit]
* 92220dcf - Fix for Switch Port Modes and VLAN CLI Enhancement (#3108) (4 weeks ago) [Saba Akram]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pavel-shirshov pavel-shirshov

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ITJamie @mslocrian @pavel-shirshov @nikos-github