Bug 1695645

Summary: [Kubevirt-Foreman] Image based provisioning should have bootorder: 1 for containerDisk
Product: Red Hat Satellite Reporter: Vatsal Parekh <vparekh>
Component: Compute Resources - CNVAssignee: Moti Asayag <masayag>
Status: CLOSED ERRATA QA Contact: Lukáš Hellebrandt <lhellebr>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.6.0CC: egolov, masayag
Target Milestone: 6.7.0Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: tfm-rubygem-foreman_kubevirt-0.1.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-04-14 13:24:10 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1746434    
Bug Blocks:    

Description Vatsal Parekh 2019-04-03 13:57:58 UTC
Description of problem:
Image based provisioning should have bootorder: 1 for the containerDisk
a example yaml from a VM created from KubeVirt console
```
        devices:
          disks:
            - bootOrder: 1
              disk:
                bus: virtio
              name: rootdisk
            - disk:
                bus: virtio
              name: test-noboot
          interfaces:
            - bridge: {}
              name: nic0
          rng: {}
        machine:
          type: ''
        resources:
          requests:
            memory: 4G
      networks:
        - name: nic0
          pod: {}
      terminationGracePeriodSeconds: 0
      volumes:
        - containerDisk:
            image: kubevirt/fedora-cloud-registry-disk-demo
          name: rootdisk
        - dataVolume:
            name: test-noboot-test-fedora
          name: test-noboot
```

Version-Release number of selected component (if applicable):
master of the plugin

How reproducible:


Steps to Reproduce:
1.Create a Image based host
2.
3.

Actual results:
The provisioning works, but should have the bootorder defined 

Expected results:


Additional info:

Comment 3 Vatsal Parekh 2019-04-12 07:00:18 UTC
This seems working on latest nightly Foreman + plugin master

Comment 6 Lukáš Hellebrandt 2020-03-23 11:27:30 UTC
Verified with Sat 6.7 snap 16.

A disk tied to contanerDisk volume has indeed bootorder 1 specified in yaml.

Comment 9 errata-xmlrpc 2020-04-14 13:24:10 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:1454