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.
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
Fix 'ovsdb-client needs-conversion' to access remote server #347
Conversation
Thanks! The change looks fine. Also, please, add the problem description to the commit message itself. |
caee2af
to
ea96d8a
Compare
Hi Ilya, thanks for the review. |
ea96d8a
to
2e4a499
Compare
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Signed-off-by: Alexey Roytman <roytman@il.ibm.com>
2e4a499
to
79981e6
Compare
hi @igsilya, should I do something else here? |
@roytman, the change looks good to me, I'll apply it later today. |
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
When you try to specify `SERVER` to the 'ovsdb-client needs-conversion' command, it interprets the `SERVER` parameter as the path to the schema and returns an error. This PR fixes it. Fixes: 1b1d2e6 ("ovsdb: Introduce experimental support for clustered databases.") Submitted-at: openvswitch#347 Signed-off-by: Alexey Roytman <roytman@il.ibm.com> Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Thanks! |
When you try to specify
SERVER
it interprets the parameter as the path to the schema.`ovsdb-client needs-conversion unix:/var/run/openvswitch/db.sock ./_server.ovsschema
ovsdb-client: failed to read schema: "unix:/var/run/openvswitch/db.sock" could not be read as JSON (error opening "unix:/var/run/openvswitch/db.sock": No such file or directory)`
This PR fixes it.
Signed-off-by: Alexey Roytman roytman@il.ibm.com