Closed Prashanth684 closed 4 years ago
/test images
/test e2e-libvirt
/retest
Looks good to me.
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cfergeau
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
/retest
/retest
Please review the full test history for this PR and help us cut down flakes.
/bugzilla refresh
@praveenkumar: No Bugzilla bug is referenced in the title of this pull request.
To reference a bug, add 'Bug XXX:' to the title of this pull request and request another bug refresh with /bugzilla refresh
.
/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.
/retitle Bug 1840695: ignition: use setCoreOSIgnition function to inject ignition config
@Prashanth684: This pull request references Bugzilla bug 1840695, which is invalid:
Comment /bugzilla refresh
to re-evaluate validity if changes to the Bugzilla bug are made, or edit the title of this pull request to link to a different bug.
/bugzilla refresh
@Prashanth684: This pull request references Bugzilla bug 1840695, which is valid. The bug has been moved to the POST state. The bug has been updated to refer to the pull request using the external bug tracker.
/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.
@Prashanth684: The following test failed, say /retest
to rerun all failed tests:
Test name | Commit | Details | Rerun command |
---|---|---|---|
ci/prow/actuator-pkg-staleness | 2f479a035b202d4acce86dd07613df1ddd09844a | link | /test actuator-pkg-staleness |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
/retest
Please review the full test history for this PR and help us cut down flakes.
@Prashanth684: All pull requests linked via external trackers have merged: openshift/cluster-api-provider-libvirt#194. Bugzilla bug 1840695 has been moved to the MODIFIED state.
A minor refactoring of code which is repeated in domain.go. The setCoreOSIgnition function is handling the injection of ignition. There is no need to repeat this code as this function can just be used.