1
82%
master: 82%

Ran 09 May 2018 12:16PM UTC

Files 2

Run time 1s

Badge

Embed ▾

Committed 09 May 2018 12:09PM UTC coverage: 81.633%. Remained the same

Job # 66.1

Build Type

Pull #24

travis-ci

Committed by

web-flow

Commit Message

Rajiv Asati's Update to applyExternalConfig()

Signed-off-by: Rajiv Asati <rajiva@cisco.com>

Ligato BGP code fails if used with an external .yaml file, because line 70 defined externalCfg of pointer type and passed it as-is in PluginConfig.GetValue() in line 71, as shown below:
//
	func (plugin *Plugin) applyExternalConfig() {
	var externalCfg *config.Bgp
	found, err := plugin.PluginConfig.GetValue(externalCfg)
//
The above meant passing a different memory address from where the external file really was (remember, every time a variable is passed as parameter, a new copy of the variable is created), resulting in a garbage being passed to ParseConfigFromYamlFile() function inside p.GetConfigName(), which then returned an error.

Fix is to define config.Bgp variable as a pointer to the address where the external .yaml file is, and pass the same pointer in PluginConfig.GetValue() in line 71 and 80. 

https://gobyexample.com/pointers
http://goinbigdata.com/golang-pass-by-pointer-vs-pass-by-value/

Pull Request Pull Request #24: Rajiv Asati's Update to plugin_impl_gobgp.go

Run Details

80 of 98 relevant lines covered (81.63%)

1.13 hits per line

ligato / bgp-agent / 66 / 1
82%
master: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 66.1

ligato / bgp-agent / 66 / 1 82% master: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 66.1

ligato / bgp-agent / 66 / 1
82%
master: 82%

README BADGES
x