Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

fix starting issue in alicloud image: coreos_1745_7_0_64_30G_alibase_… #32

Merged
merged 1 commit into from
Aug 28, 2018
Merged

fix starting issue in alicloud image: coreos_1745_7_0_64_30G_alibase_… #32

merged 1 commit into from
Aug 28, 2018

Conversation

jia-jerry
Copy link
Contributor

@jia-jerry jia-jerry commented Aug 28, 2018 •
edited by rfranzke
Loading

…20180705.vhd

What this PR does / why we need it:
Solve the issue that VPN shoot can't start in Alicloud CoreOS
Which issue(s) this PR fixes:
Fixes #31

Special notes for your reviewer:
The commit is verified in Alicloud.
Release note:

An issue causing vpn-shoot to fail starting on CoreOS machines provisioned by Alicloud has been resolved.

@rfranzke rfranzke added kind/bug Bug reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) topology/shoot Affects Shoot clusters area/networking Networking related status/in-progress Issue is in progress/work platform/ali size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) labels Aug 28, 2018
@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 28, 2018
rfranzke
rfranzke approved these changes Aug 28, 2018
View reviewed changes
Copy link
Contributor

@rfranzke rfranzke left a comment

Choose a reason for hiding this comment

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

LGTM

@rfranzke rfranzke added status/accepted Issue was accepted as something we need to work on and removed needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) status/in-progress Issue is in progress/work labels Aug 28, 2018
@rfranzke rfranzke merged commit 3f926be into gardener-attic:master Aug 28, 2018
@ghost ghost added the platform/alicloud Alicloud platform/infrastructure label Mar 7, 2020
@gardener-robot gardener-robot added priority/3 Priority (lower number equals higher priority) and removed priority/3 Priority (lower number equals higher priority) labels Mar 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rfranzke rfranzke rfranzke approved these changes

@dkistner dkistner Awaiting requested review from dkistner

Assignees
No one assigned
Labels
area/networking Networking related kind/bug Bug platform/alicloud Alicloud platform/infrastructure size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) status/accepted Issue was accepted as something we need to work on topology/shoot Affects Shoot clusters
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

VPN-Shoot failed to start in Alicloud CoreOS
4 participants
@jia-jerry @rfranzke @gardener-robot-ci-1 @gardener-robot