Skip to content

Bug 1206109 - Default empty tag slice to 'latest' #1465

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 26, 2015
Merged

Bug 1206109 - Default empty tag slice to 'latest' #1465

merged 1 commit into from
Mar 26, 2015

Conversation

0xmichalis
Copy link
Contributor

@mfojtik
Copy link
Contributor

mfojtik commented Mar 26, 2015

@Kargakis the proper format is to make PR title 'Bug XYZ - Description' (or commit message). Without description one must go to BZ ;-) people are lazy nowadays ;-)

@mfojtik
Copy link
Contributor

mfojtik commented Mar 26, 2015

LGTM (just change the commit message)

@0xmichalis 0xmichalis changed the title Bug 1206109 Bug 1206109 - Default empty tag slice to 'latest' Mar 26, 2015
@0xmichalis
Copy link
Contributor Author

@Kargakis the proper format is to make PR title 'Bug XYZ - Description' (or commit message). Without description one must go to BZ ;-) people are lazy nowadays ;-)

🆗

@mfojtik
Copy link
Contributor

mfojtik commented Mar 26, 2015

[merge] thanks!

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/1303/) (Image: devenv-fedora_1137)

@openshift-bot
Copy link
Contributor

Evaluated for origin up to f4d8bf0

openshift-bot pushed a commit that referenced this pull request Mar 26, 2015
@openshift-bot openshift-bot merged commit e0c2288 into openshift:master Mar 26, 2015
@0xmichalis 0xmichalis deleted the bug-fix branch March 26, 2015 11:49
jboyd01 pushed a commit to jboyd01/origin that referenced this pull request Nov 3, 2017
…service-catalog/' changes from 892b0368f0..3064247d05

3064247d05 origin build: add origin tooling
48ecff1 Chart changes for 0.1.2 (openshift#1527)
8727247 Fix change validator to look up serviceclass properly (openshift#1518)
b0b138b Broker Reconciliation occurs too frequently (openshift#1514)
cc816eb When a SI is unbindable, mark the binding request failed. (openshift#1522)
41290e9 Clarify semantics around RelistDuration (openshift#1516)
6bcb593 Send Context with UpdateInstanceRequest (openshift#1517)
c3b72d8 Enforce stricly increasing broker relistRequests (openshift#1515)
01d81e5 Follow up from openshift#1450 and openshift#1444 (openshift#1504)
5e77882 Retry failed deprovision requests (openshift#1505)
f6c891d Allow updates of instances that failed a previous update. (openshift#1502)
5107086 Use Plan ID from ExternalProperties when deprovisioning instance (openshift#1501)
d897c60 Adding message builder for expected event strings  (openshift#1465)
9f6152e update osb client and freeze gorilla context (openshift#1496)
c63277e Add unit tests verifying deleting a resource with an on-going operation or in orphan mitigation. (openshift#1490)
4ecca16 Pretty logging for controller_instance.go (openshift#1472)
c232db4 Register qemu statics when building non-amd64 images (openshift#1494)
0e54c57 fix 'Spring Cloud Services -> Configuring with Vault' link in docs (openshift#1495)
REVERT: 892b0368f0 origin build: add origin tooling

git-subtree-dir: cmd/service-catalog/go/src/github.com/kubernetes-incubator/service-catalog
git-subtree-split: 3064247d0544aa43f976d93caea0a11771434ef7
jpeeler pushed a commit to jpeeler/origin that referenced this pull request Feb 1, 2018
* Refactor how we create unit test expected event messages.

* Fixing unit tests and some bugs in the message builder.

* Expand the comments for MessageBuilder.

* Reverting the helper methods in the helper builder, now more clean.

* Adding a helper method to shortcut some of the construction of the message builder.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants