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

(PDK-1143) Allow SimpleProvider to handle multiple namevars #125

Merged
merged 1 commit into from
Sep 25, 2018
Merged

(PDK-1143) Allow SimpleProvider to handle multiple namevars #125

merged 1 commit into from
Sep 25, 2018

Conversation

da-ar
Copy link

@da-ar da-ar commented Sep 25, 2018

Create, Update and Delete now receive a hash containing all the namevar values and the composite title if the type has multiple namevars

Create, Update and Delete now receive a hash containing all the namevar values and the composite title if the type has multiple namevars
Thomas-Franklin pushed a commit to Thomas-Franklin/puppetlabs-panos that referenced this pull request Sep 25, 2018
PDK-1143 allows the resource_api to work nicely with composite providers, this PR is a refactor of the `path_monitor` and `static_routes` which work with composite namevars.

This PR's `gemfile` changes should be reverted after puppetlabs/puppet-resource_api#125 is merged.
Copy link
Contributor

@Thomas-Franklin Thomas-Franklin left a comment

Choose a reason for hiding this comment

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

I've tested these changes with PANOS and they function as intended. 👍

@da-ar da-ar changed the title WIP (PDK-1143) Allow SimpleProvider to handle multiple namevars (PDK-1143) Allow SimpleProvider to handle multiple namevars Sep 25, 2018
Thomas-Franklin pushed a commit to Thomas-Franklin/puppetlabs-panos that referenced this pull request Sep 25, 2018
PDK-1143 allows the resource_api to work nicely with composite providers, this PR is a refactor of the `path_monitor` and `static_routes` which work with composite namevars.

This PR's `gemfile` changes should be reverted after puppetlabs/puppet-resource_api#125 is merged.
Thomas-Franklin pushed a commit to Thomas-Franklin/puppetlabs-panos that referenced this pull request Sep 25, 2018
PDK-1143 allows the resource_api to work nicely with composite providers, this PR is a refactor of the `path_monitor` and `static_routes` which work with composite namevars.

This PR's `gemfile` changes should be reverted after puppetlabs/puppet-resource_api#125 is merged.
@da-ar da-ar merged commit cecd90b into puppetlabs:master Sep 25, 2018
@da-ar da-ar deleted the simpleprovider_composite branch May 14, 2019 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants