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

[device/alibaba]: Update BMC watchdog handler #89

Merged
merged 5 commits into from May 28, 2019
Merged

[device/alibaba]: Update BMC watchdog handler #89

merged 5 commits into from May 28, 2019

Conversation

mudsut4ke
Copy link

@mudsut4ke mudsut4ke commented May 27, 2019 •
edited
Loading

- What I did

  • Update BMC watchdog service to avoid trash Syslog
  • Remove BMC watchdog handler on fwmgrutil
  • Add missing BMC watchdog service on Phalanx platform

- How I did it

  • Remove bmc_wdt.timer
  • Update bmc_wdt.service to send request to BMC every 20 sec
  • Update bmc_wdt.sh to hide the log
  • Remove BMC watchdog handler on fwmgrutil (BMC Restful already supported concurrent request)
  • Install BMC watchdog service files to Phalanx platform

- How to verify it

  • Check BMC watchdog service status on Phalanx via this following command
    systemctl status bmc_wdt.service

@mudsut4ke mudsut4ke added the invalid This doesn't seem right label May 27, 2019
@mudsut4ke mudsut4ke self-assigned this May 27, 2019
@mudsut4ke mudsut4ke changed the title [device/alibaba]: Update watchdog handler [device/alibaba]: Update BMC watchdog handler May 27, 2019
@mudsut4ke mudsut4ke requested a review from tiantianlv May 27, 2019 09:30
@tiantianlv tiantianlv merged commit a5600a1 into SONIC-DEV:201811_cel May 28, 2019
@mudsut4ke mudsut4ke deleted the 201811_cel-update-firmware-upgrade-api branch May 28, 2019 04:36
jerseyang pushed a commit that referenced this pull request Oct 12, 2021
@abdosi
[Submodule Update] sonic-restapi
44d2ec6 
e466ac226c48cf87f83aa5564efd7155ac97a0c1 (HEAD -> master, origin/master, origin/HEAD) Reject incorrect CIDR addresses while configuring routes (#91)
07cf59d866914ef7cb6901d51f289f0bee721e44 [CI] Set up Azure pipeline (#89)
47476acdfaf668ab0cd9ae2443205a3bdc2e9ef9 Upgrade Go version to 1.14 (#90)
ee887c785c10d8b4f70db90ca50cc398327c5d73 Added flag to make go/pkg/ folder writable (removable) (#87)
6cfc8fe110c46a1ee5a8430e02a440b5a7b4006d (origin/shilongliu/azp) Default Vrf static route support (#85)
d3e8b8e4aee278fa11d7d86acabdab3fffc70d5a Add BFD config to subinterface and neighbor (#68)
8d4fd4a56b637d3b7403c321aef36b8dff58cd62 Fix WS-2018-0594 potential non-random UUIDs security issue (#78)
fad87fa20f6b15e28c90e02a131c44d236d6a751 Update API description for default vrf and ecmp routes (#82)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tiantianlv tiantianlv tiantianlv approved these changes

Assignees

@mudsut4ke mudsut4ke

Labels
invalid This doesn't seem right
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mudsut4ke @tiantianlv