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

WIP: ovirt: Bring the oVirt virtual machines provider into the tree #7133

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@stefwalter
Contributor

stefwalter commented Jun 28, 2017

This shares a ton of code with the machines package, and we don't want to be exporting an additional stable API to make this possible.

This is really another way of finding and displaying virtual machines and needs to be covered by CI and testing.

To be finished:

  • #7128
  • Plug the oVirt functionality in - #7139
  • An oVirt image
  • Tests for the ovirt functionality
  • Documentation in doc/guide/feature-ovirt.xml

stefwalter added some commits Jun 28, 2017

machines: Providers for machines javascript use webpack linking
Other providers that want to reuse the machines javascript will
use static webpack or browserify javascript linking to bring
that code into their component.

This means that there will be a completely different package for
each provider, and they may replace each other using the standard
manifest.json mechanisms.

This allows us to keep from committing to a spceific version
of the machines provider API. We instead publish an NPM module
each time the code changes, and other providers can pull versions
of that module that are compatible with their code.
ovirt: Bring the oVirt virtual machines provider into the tree
This shares a *ton* of code with the machines package, and we
don't want to be exporting an additional stable API to make this
possible.

This is really another way of finding and displaying virtual machines
and needs to be covered by CI and testing.

@stefwalter stefwalter added the blocked label Jun 28, 2017

@mareklibra mareklibra referenced this pull request Jun 29, 2017

Closed

oVirt Machines #7139

14 of 14 tasks complete
@stefwalter

This comment has been minimized.

Show comment
Hide comment
@stefwalter

stefwalter Aug 3, 2017

Contributor

Replaced by #7139

Contributor

stefwalter commented Aug 3, 2017

Replaced by #7139

@stefwalter stefwalter closed this Aug 3, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment