diff options
author | Chirag Shah <chirag@cumulusnetworks.com> | 2017-08-04 01:34:17 +0200 |
---|---|---|
committer | Chirag Shah <chirag@cumulusnetworks.com> | 2017-08-04 16:47:45 +0200 |
commit | a3d826f0ea4e773ef2e359390e143d13f22a55e5 (patch) | |
tree | c5440aa04e7e445f114e466469c62370c4e327d4 /ospfd/ospf_opaque.c | |
parent | Merge pull request #908 from donaldsharp/safi (diff) | |
download | frr-a3d826f0ea4e773ef2e359390e143d13f22a55e5.tar.xz frr-a3d826f0ea4e773ef2e359390e143d13f22a55e5.zip |
ospfd: Fix MI-OSPF configuraton clis
Multi-Instance OSPF configuration CLI would fail because
first client return error upon seeing qobj_index being 0.
With new marco generate new error code to return from each
instance (vtysh client) and if the command is intended for given
instance, its qobj_index would be nonzero and process the command
and push correct ospf context. Other instance would return the error.
On vtysh end, check all instance return an error log a message to a
file.
Testing Done:
Verfied various MI-OSPF configuration CLI with multi instances.
Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>
Diffstat (limited to 'ospfd/ospf_opaque.c')
-rw-r--r-- | ospfd/ospf_opaque.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ospfd/ospf_opaque.c b/ospfd/ospf_opaque.c index a2c40923b..db523bd2a 100644 --- a/ospfd/ospf_opaque.c +++ b/ospfd/ospf_opaque.c @@ -750,7 +750,7 @@ DEFUN (capability_opaque, "Enable specific OSPF feature\n" "Opaque LSA\n") { - VTY_DECLVAR_CONTEXT(ospf, ospf); + VTY_DECLVAR_INSTANCE_CONTEXT(ospf, ospf); /* Turn on the "master switch" of opaque-lsa capability. */ if (!CHECK_FLAG(ospf->config, OSPF_OPAQUE_CAPABLE)) { @@ -779,7 +779,7 @@ DEFUN (no_capability_opaque, "Enable specific OSPF feature\n" "Opaque LSA\n") { - VTY_DECLVAR_CONTEXT(ospf, ospf); + VTY_DECLVAR_INSTANCE_CONTEXT(ospf, ospf); /* Turn off the "master switch" of opaque-lsa capability. */ if (CHECK_FLAG(ospf->config, OSPF_OPAQUE_CAPABLE)) { |