Skip linux-bridge filtering if no "bridge" section

pkg/state/filter.go

@@ -30,7 +30,16 @@ func FilterOut(currentState shared.State) (shared.State, error) {
 }
 
 func filterOutRoutes(kind string, state map[string]interface{}, interfacesFilterGlob glob.Glob) {
-	routes := state["routes"].(map[string]interface{})
+	routesRaw, hasRoutes := state["routes"]
+	if !hasRoutes {
+		return
+	}
+
+	routes, ok := routesRaw.(map[string]interface{})
+	if !ok {
+		return
+	}
+
 	routesByKind := routes[kind].([]interface{})
 
 	if routesByKind == nil {
@@ -58,9 +67,24 @@ func filterOutDynamicAttributes(iface map[string]interface{}) {
 		return
 	}
 
-	bridge := iface["bridge"].(map[string]interface{})
+	bridgeRaw, hasBridge := iface["bridge"]
+	if !hasBridge {
+		return
+	}
+	bridge, ok := bridgeRaw.(map[string]interface{})
+	if !ok {
+		return
+	}
+
+	optionsRaw, hasOptions := bridge["options"]
+	if !hasOptions {
+		return
+	}
+	options, ok := optionsRaw.(map[string]interface{})
+	if !ok {
+		return
+	}
 
-	options := bridge["options"].(map[string]interface{})
 	delete(options, "gc-timer")
 	delete(options, "hello-timer")
 }

pkg/state/filter_test.go

@@ -307,4 +307,28 @@ routes:
 			Expect(returnedState).To(MatchYAML(filteredState))
 		})
 	})
+	Context("when there is a linux bridge without 'bridge' options because is down", func() {
+		BeforeEach(func() {
+			state = nmstate.NewState(`
+interfaces:
+- name: br1
+  type: linux-bridge
+  state: down
+`)
+
+			filteredState = nmstate.NewState(`
+interfaces:
+- name: br1
+  type: linux-bridge
+  state: down
+`)
+			interfacesFilterGlob = glob.MustCompile("")
+		})
+		It("should keep the bridge as it is", func() {
+			returnedState, err := filterOut(state, interfacesFilterGlob)
+			Expect(err).ToNot(HaveOccurred())
+			Expect(returnedState).To(MatchYAML(filteredState))
+		})
+	})
+
 })