Conversation
|
Hi @dirkmueller. Thanks for your PR. PRs from untrusted users cannot be marked as trusted with I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
0xFelix
left a comment
There was a problem hiding this comment.
Thanks for the PR! Looking great except for a few comments.
| common.DefaultInstancetypeEnv: "u1.medium", | ||
| common.DefaultPreferenceEnv: "opensuse.leap", |
There was a problem hiding this comment.
I wonder if these could work on aarch64 too?
There was a problem hiding this comment.
The preference set a minimum of 1 cpu and 2GiB. I'm not able to find the minimum requirements for aarch64. @dirkmueller, could you please check if arm version meets this minimum hardware requirements too?
There was a problem hiding this comment.
I don't think it is officially documented anywhere, but I think that would work. I've started very minimal VMs with that configuration.
|
/release-note-none |
|
@dirkmueller ping |
thanks for the reminder, this fell off the list |
Thanks to you for your contribution! |
|
/retest |
Signed-off-by: Dirk Müller <dmueller@suse.com>
|
@dirkmueller: The following tests failed, say
DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
New changes are detected. LGTM label has been removed. |
|
ok, so ssh login to leap 16.0 fails, I need to take a look at that.. |
|
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with /lifecycle stale |
|
/remove-lifecycle stale I'll work on this, just had other priorities so far. |
|
@dirkmueller I could successfully build a containerdisk manually using the Leap 16.0 image: https://quay.io/repository/kastl-atvantage/kubevirt-containerdisk-opensuse-leap-16.0 This worked fine and I could log in using the VNC console and SSH (via Anything I can help out with? |
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
What this PR does / why we need it:
openSUSE Leap has recently released it's newest stable release 16.0, which has a slightly different url layout.