Skip to content

Commit 816851c

Browse files
authored
Merge pull request #351 from myakove/nncp-set-self.res-default-to-dict
NNCP: call to_dict() if not self.res
2 parents 60e6147 + 7e072bf commit 816851c

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

ocp_resources/node_network_configuration_policy.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ def __init__(
5151
ipv6_dhcp=False,
5252
ipv6_auto_dns=True,
5353
ipv6_addresses=None,
54-
node_active_nics=None,
5554
dns_resolver=None,
5655
routes=None,
5756
yaml_file=None,
@@ -115,6 +114,8 @@ def __init__(
115114
self.nodes = list(Node.get(dyn_client=self.client))
116115

117116
def set_interface(self, interface):
117+
if not self.res:
118+
self.res = super().to_dict()
118119
# First drop the interface if it's already in the list
119120
interfaces = [
120121
iface

ocp_resources/node_network_state.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,7 @@ def __init__(
3232
delete_timeout=delete_timeout,
3333
)
3434
status = self.instance.to_dict()["status"]
35-
if "desiredState" in status:
36-
self.desired_state = status["desiredState"]
37-
else:
38-
self.desired_state = {"interfaces": []}
35+
self.desired_state = status.get("desiredState", {"interfaces": []})
3936

4037
def set_interface(self, interface):
4138

0 commit comments

Comments
 (0)