Skip to content

Issue #1457 - Allow Example Resource to Load#1460

Merged
wojtek0806 merged 2 commits intoF5Networks:developmentfrom
jasonrahm:fix.examplefiles
Jun 20, 2018
Merged

Issue #1457 - Allow Example Resource to Load#1460
wojtek0806 merged 2 commits intoF5Networks:developmentfrom
jasonrahm:fix.examplefiles

Conversation

@jasonrahm
Copy link
Copy Markdown

Problem: the example file is a collection kind, which is a kind
mismatch on a resource

Solution: Check for example at end of selflink in the _produce_instance
def and don't error on kind mismatch if present

Files Changed:

  • f5/bigip/resource.py

@jasonrahm jasonrahm closed this Jun 19, 2018
@jasonrahm jasonrahm reopened this Jun 19, 2018
@jasonrahm
Copy link
Copy Markdown
Author

don't merge yet, even if the build passes I want to add a unit test for a positive match for an example

Problem: the example file is a collection kind, which is a kind
         mismatch on a resource

Solution: Check for example at end of selflink in the _produce_instance
          def and don't error on kind mismatch if present

Files Changed:

 - f5/bigip/resource.py
 - f5/bigip/test/functional/test_resource.py
@jasonrahm
Copy link
Copy Markdown
Author

ready for review, @caphrim007 or @wojtek0806

@caphrim007
Copy link
Copy Markdown
Contributor

this looks fine

@wojtek0806
Copy link
Copy Markdown
Contributor

approved

@wojtek0806 wojtek0806 merged commit df296ad into F5Networks:development Jun 20, 2018
Copy link
Copy Markdown
Contributor

@wojtek0806 wojtek0806 left a comment

Choose a reason for hiding this comment

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

approved

@jasonrahm jasonrahm deleted the fix.examplefiles branch June 20, 2018 16:29
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