Skip to content
This repository has been archived by the owner on Dec 14, 2023. It is now read-only.

adding ssh key support #8

Merged

Conversation

moadqassem
Copy link
Contributor

What this PR does / why we need it:
Add ssh keys support to the kubevirt cloud provider.

Which issue(s) this PR fixes:
Fixes #
gardener-attic/gardener-extension-provider-kubevirt#10

Special notes for your reviewer:
Next step is the integrate it in the kubevirt extension.

Release note:

ssh keys support for kubevirt cloud provider

@moadqassem moadqassem requested a review from a team as a code owner August 17, 2020 18:33
@gardener-robot-ci-3 gardener-robot-ci-3 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Aug 17, 2020
@gardener-robot-ci-1 gardener-robot-ci-1 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Aug 17, 2020
Signed-off-by: Moath Qasim <[email protected]>
@moadqassem moadqassem force-pushed the support-user-ssh-keys-in-vms branch from 3b273ef to 6fea3e2 Compare August 17, 2020 18:41
@gardener-robot-ci-2 gardener-robot-ci-2 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Aug 17, 2020
@moadqassem
Copy link
Contributor Author

/cc @stoyanr @mfranczy

Copy link
Contributor

@stoyanr stoyanr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm, thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants