Closed hh closed 12 months ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: hh Once this PR has been reviewed and has the lgtm label, please assign agradouski for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
pkg/kubevirt/machine.go | 2 | 4 | 50.0% | ||
pkg/kubevirt/utils.go | 36 | 52 | 69.23% | ||
<!-- | Total: | 38 | 56 | 67.86% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
pkg/kubevirt/utils.go | 2 | 73.94% | ||
<!-- | Total: | 2 | --> |
Totals | |
---|---|
Change from base Build 4069808194: | -0.2% |
Covered Lines: | 964 |
Relevant Lines: | 1838 |
Ensuring we loop in @davidvossel
someone could theoretically set a CloudInitConfigDrive volume too. I don't like that this logic is assuming that only a CloudInitNoCloud volume get's the special behavior.
making the default cloudInitConfigDrive is good though
I'm comfortable merging this once the mergo logic is replaced.
/ok-to-test
this comment [1] from the previous pr was the primary remaining thing. We don't have to remove mergo from go.mod since i know it's an indirect dependency somewhere in the dependency chain. I'd just like to see that we dont' introduce that dependency directly in the capk code for this PR simply because it isn't needed.
The Kubernetes project currently lacks enough contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the PR is closedYou can:
/remove-lifecycle stale
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale
The Kubernetes project currently lacks enough active contributors to adequately respond to all PRs.
This bot triages PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the PR is closedYou can:
/remove-lifecycle rotten
/close
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle rotten
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.
This bot triages PRs according to the following rules:
lifecycle/stale
is appliedlifecycle/stale
was applied, lifecycle/rotten
is appliedlifecycle/rotten
was applied, the PR is closedYou can:
/reopen
/remove-lifecycle rotten
Please send feedback to sig-contributor-experience at kubernetes/community.
/close
@k8s-triage-robot: Closed this PR.
We are picking this up from @BobyMCbobs (thanks for the work on this) Context: https://github.com/kubernetes-sigs/cluster-api-provider-kubevirt/pull/212