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
kubevirt: Add edit VM flavor integration test #2517
kubevirt: Add edit VM flavor integration test #2517
Conversation
Rebased as master-4.3 is created now. |
@rhrazdil , can you please have a look? Just last 3 commits are relevant. When rebasing, I noticed a bunch of changes in the integration tests since I wrote this PR originally and waited for the master-4.3 to be created. Aren't my changes in conflict with the intention of other PRs, please? Tests are still passing for me locally. |
/retest |
Rebased as #2392 is merged now. |
).toBeTruthy(); | ||
}); | ||
}, | ||
VM_BOOTUP_TIMEOUT_SECS, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should come up with some more specific timeout for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done: VM_CREATE_AND_EDIT_TIMEOUT_SECS
, wdyt?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
export const CONFIG_NAME_PXE = 'PXE'; | ||
export const CONFIG_NAME_CLONED_DISK = 'ClonedDisk'; | ||
|
||
export const getProvisionConfigs = (testName: string) => |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, nice
I have split this PR to simplify review. |
/lgtm |
rebased as #2812 is merged now |
/hold |
/retest Please review the full test history for this PR and help us cut down flakes. |
18 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
linter issue fixed |
/retest |
In addition, VM Wizard integration tests are slightly refactored for better reuse of the code.
Rebased |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mareklibra, rhrazdil, vojtechszocs The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Follow-up for #2392.
Depends on:
Please note, just last commit is relevant.
Screen-cast of the integration test execution