Skip to content

Commit

Permalink
tests, dry-run: Move test to pending
Browse files Browse the repository at this point in the history
in the current Kubevirt client the dry-run for VM create is not yet
implemented.
Moving this test temporarily to pending.

Signed-off-by: Ram Lavi <ralavi@redhat.com>
  • Loading branch information
RamLavi committed Dec 7, 2022
1 parent e42c773 commit 206885d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/virtual_machines_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,8 @@ var _ = Describe("[rfe_id:3503][crit:medium][vendor:cnv-qe@redhat.com][level:com
macAddress = vm.Spec.Template.Spec.Domain.Devices.Interfaces[0].MacAddress
})

It("should not allocate the Mac assigned in the dryRun request into the pool", func() {
//TODO Add dry-run functionality in kubevirt/client-go's VirtualMachine().Create()
PIt("should not allocate the Mac assigned in the dryRun request into the pool", func() {
By("creating a vm with the same mac to make sure the dryRun assigned mac is not occupied on the macpool")
var err error
vmDryRunOverlap := CreateVmObject(TestNamespace, false, []kubevirtv1.Interface{newInterface("brDryRunOverlap", macAddress)}, []kubevirtv1.Network{newNetwork("brDryRunOverlap")})
Expand Down

0 comments on commit 206885d

Please sign in to comment.