Skip to content
This repository has been archived by the owner on Apr 7, 2020. It is now read-only.

Add controlplane for azure provider #95

Merged
merged 1 commit into from
May 29, 2019

Conversation

mkoynov
Copy link
Contributor

@mkoynov mkoynov commented May 27, 2019

What this PR does / why we need it:
Adds controlplane controller for provider-azure.

Which issue(s) this PR fixes:
Fixes #54

Special notes for your reviewer:
The code follows the same logic as the other providers that has it implemented already.

Release note:

Add controlplane controller for Azure.

@mkoynov mkoynov requested a review from a team as a code owner May 27, 2019 15:24
@CLAassistant
Copy link

CLAassistant commented May 27, 2019

CLA assistant check
All committers have signed the CLA.

@rfranzke rfranzke 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 May 27, 2019
@mkoynov mkoynov force-pushed the add-azure-controlplane branch from 8915655 to a2d11d7 Compare May 28, 2019 08:33
@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 May 28, 2019
stoyanr
stoyanr previously approved these changes May 29, 2019
@zanetworker zanetworker self-assigned this May 29, 2019
Copy link
Contributor

@zanetworker zanetworker left a comment

Choose a reason for hiding this comment

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

Some minor changes, otherwise looks good :)

@zanetworker
Copy link
Contributor

Also please rebase, otherwise /lgtm

@mkoynov mkoynov force-pushed the add-azure-controlplane branch from 086a1e8 to ee295f2 Compare May 29, 2019 15:57
@mkoynov
Copy link
Contributor Author

mkoynov commented May 29, 2019

Also please rebase, otherwise /lgtm

Done :)

@rfranzke rfranzke 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 May 29, 2019
@gardener-robot-ci-1 gardener-robot-ci-1 removed 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 May 29, 2019
@zanetworker zanetworker merged commit 447d926 into gardener-attic:master May 29, 2019
@mkoynov mkoynov deleted the add-azure-controlplane branch June 13, 2019 13:27
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.

Implement ControlPlane controller for Azure provider
6 participants