Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

fix issue#526 #527

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

shaoyongfeng
Copy link

What this PR does / why we need it:
In order to run aliyun cmd via gardenctl, we need arguments after aliyun key word and pass to aliyun cli. but aliyun key word is duplicated in this case which causes error when running any aliyun cmd.

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

Special notes for your reviewer:

Release note:


@shaoyongfeng shaoyongfeng requested a review from a team as a code owner October 19, 2021 13:36
@CLAassistant
Copy link

CLAassistant commented Oct 19, 2021

CLA assistant check
All committers have signed the CLA.

@gardener-robot-ci-2
Copy link
Contributor

Thank you @shaoyongfeng for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below.

@gardener-robot
Copy link

@shaoyongfeng Thank you for your contribution.

@gardener-robot gardener-robot added needs/review Needs review size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) labels Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs/review Needs review size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

can't run aliyun cli command via gardenctl
4 participants