Skip to content
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

Allow drivers to run in isolation #380

Merged
merged 2 commits into from
Jan 5, 2022
Merged

Conversation

cloudnull
Copy link
Collaborator

This change will allow each driver to run in isolation and ensure that
the options provided by the driver are only present in CLI interactions
when the driver would be functional. Non-functional drivers will be
omitted in the help output. This is a UX improvement, as well as a
packaging one.

Signed-off-by: Kevin Carter kecarter@redhat.com

@cloudnull cloudnull force-pushed the driver-iso branch 3 times, most recently from 4c80c2f to a1a43d9 Compare January 5, 2022 21:27
This change will allow each driver to run in isolation and ensure that
the options provided by the driver are only present in CLI interactions
when the driver would be functional. Non-functional drivers will be
omitted in the help output. This is a UX improvement, as well as a
packaging one.

Signed-off-by: Kevin Carter <kecarter@redhat.com>
@cloudnull cloudnull force-pushed the driver-iso branch 5 times, most recently from 320606d to 3fd2eea Compare January 5, 2022 22:53
the zmq driver has been renamed to correct an import issue with the zmq
module using the same namespace as the zmq driver. While this issue can
be "fixed" with absolute imports, the driver should never have been
using this same namespace and since zmq is no longer the default driver
now is the time to correct this mistake.

Signed-off-by: Kevin Carter <kecarter@redhat.com>
@cloudnull cloudnull merged commit 97991fd into oshied:main Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant