Skip to content

Conversation

joelsmith
Copy link
Contributor

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 31, 2017
@openshift-merge-robot openshift-merge-robot added the vendor-update Touching vendor dir or related files label Oct 31, 2017
@joelsmith joelsmith added the kind/bug Categorizes issue or PR as related to a bug. label Oct 31, 2017
@joelsmith
Copy link
Contributor Author

/assign @sjenning
/assign @derekwaynecarr
/unassign @deads2k

@derekwaynecarr
Copy link
Member

/approve
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 2, 2017
@openshift-merge-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: derekwaynecarr, joelsmith

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 2, 2017
@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 17089, 17120).

@openshift-merge-robot openshift-merge-robot merged commit 3ea2451 into openshift:master Nov 2, 2017
openshift-merge-robot added a commit that referenced this pull request Nov 2, 2017
Automatic merge from submit-queue.

[stage] UPSTREAM: 54828: trigger endpoint update on pod deletion

xref #17120
openshift-merge-robot added a commit that referenced this pull request Nov 3, 2017
Automatic merge from submit-queue.

UPSTREAM: 54828: trigger endpoint update on pod deletion

xref #17120
openshift-merge-robot added a commit that referenced this pull request Nov 29, 2017
Automatic merge from submit-queue (batch tested with PRs 17476, 17143, 15115, 17094, 17500).

fix basename: illegal option -- b issue with upstream filenames starting with -

issue:
```
$ git branch -vv
* devel  3ea2451 Merge pull request #17120 from joelsmith/master
  master 3ae4430 [origin/master] fix basename: illegal option -- b issue with upstream filenames starting with -
xiaods at XiaoTommydeMacBook-Pro in ~/go/src/github.com/openshift/origin on devel*
$ export PATH="${PATH}:$( source hack/lib/init.sh; echo "${OS_OUTPUT_BINPATH}/$( os::build::host_platform )/" )"
basename: illegal option -- b
usage: basename string [suffix]
       basename [-a] [-s suffix] string [...]
```
caused by :

The -- (dash dash) stops basename from processing any options in the argument.
Always quote $0 in case there are spaces in the name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. vendor-update Touching vendor dir or related files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants