Skip to content
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

Bug 1498185 - Adjust versioning check so that it is done in the registry package #468

Merged
merged 2 commits into from
Oct 4, 2017

Conversation

dymurray
Copy link
Member

@dymurray dymurray commented Oct 3, 2017

This PR adjusts for the change of moving the APB spec version from an image label into the Spec itself. It also moves the version check out of the adapter package so that it is now obfuscated out of each individual adapter.

Changes proposed in this pull request

  • Move APB spec version check to spec validation in registry package
  • Adds unit tests for specs without versions

This PR fixes https://bugzilla.redhat.com/show_bug.cgi?id=1498185

@dymurray
Copy link
Member Author

dymurray commented Oct 3, 2017

FYI: This will fail travis until I update the upstream images so that they pass bootstrapping. Once I get approved reviews I will update the APBs and rerun travis.

Copy link
Contributor

@shawn-hurley shawn-hurley left a comment

Choose a reason for hiding this comment

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

I like the test updates and moving this to the registry section.

Copy link
Member

@djzager djzager left a comment

Choose a reason for hiding this comment

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

LGTM

@jmontleon jmontleon merged commit 379b67b into openshift:master Oct 4, 2017
jianzhangbjz pushed a commit to jianzhangbjz/ansible-service-broker that referenced this pull request May 17, 2018
…try package (openshift#468)

* Bug 1498185 - Move versioning check to spec validation within registry package

* Update and add tests
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.

None yet

4 participants