Skip to content

Conversation

@atiratree
Copy link
Member

@atiratree atiratree commented Oct 7, 2019

  • cloud init data is persisted inside the volume
  • fix disks editing
  • create InlineBooleanRadio component

depends on:

@openshift-ci-robot openshift-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. component/kubevirt Related to kubevirt-plugin approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Oct 7, 2019
@atiratree
Copy link
Member Author

/retest

@atiratree
Copy link
Member Author

screenshots

ci

cia

@atiratree atiratree force-pushed the kubevirt.createVMWizard.cloudInit branch 6 times, most recently from 8e6c951 to 6f50819 Compare October 8, 2019 16:42
- add/update/remove of network interfaces according to new design
- add PXE network selector
@atiratree atiratree force-pushed the kubevirt.createVMWizard.cloudInit branch from 6f50819 to 4a579d9 Compare October 9, 2019 15:14
suomiy added 5 commits October 9, 2019 18:08
- add edit nic functionality
- add sources (Container, URL, Attach PVC, Attach Cloned PVC))
- remove unnecesary resource loading for the whole wizard
- fix bootOrder changes
- create ReviewList component
- create CombinedDisk and ProvisionSource wrappers
- cloud init data is persisted inside the volume
- fix disks editing
- create InlineBooleanRadio component
@atiratree atiratree force-pushed the kubevirt.createVMWizard.cloudInit branch from 4a579d9 to c1908fd Compare October 9, 2019 16:10
@atiratree
Copy link
Member Author

/retest

@mareklibra
Copy link
Contributor

/lgtm

Considering size of this PR, it was reviewed f2f off-line.
Additional comments which (can) arise can be addressed in follow-ups.

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 10, 2019
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mareklibra, suomiy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@atiratree
Copy link
Member Author

/retest

1 similar comment
@atiratree
Copy link
Member Author

/retest

@openshift-merge-robot openshift-merge-robot merged commit 527c28d into openshift:master Oct 10, 2019
@spadgett spadgett added this to the v4.3 milestone Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. component/kubevirt Related to kubevirt-plugin lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants